-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
solution #683
base: master
Are you sure you want to change the base?
solution #683
Conversation
app/main.py
Outdated
class Carnivore(Animal): | ||
|
||
@classmethod | ||
def bite(cls, victim: "Herbivore") -> None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If your method takes only Herbivore, then you wouldn't need the next check
Also no need to use quotes here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately without "if not isinstance(victim, Carnivore)" check I receive error "FAILED tests/test_main.py::test_carnivore_bite_carnivore - assert 50 == 100"
With this check code working fine
quotes I will remove for shoure
app/main.py
Outdated
self.health = health | ||
self.hidden = hidden | ||
|
||
self.alive.append(self) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should access class attribute through class name not self
|
||
def __str__(self) -> str: | ||
|
||
return (f"{{Name: {self.name}, " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if __repr__
is defined, and __str__
is not, the object will behave as though __str__=__repr__
So you can just change repr method here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is interesting. Thank you very much)
No description provided.