-
Notifications
You must be signed in to change notification settings - Fork 1.8k
all classes and methods created. Array with method works #2485
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
base: master
Are you sure you want to change the base?
Conversation
mateacademy-ai-mentor
left a comment
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.
Review Summary
Excellent work on this task! Your solution correctly implements the class hierarchy and demonstrates a solid understanding of polymorphism. All functional requirements are met, so I am approving your submission.
I have one minor suggestion for future improvement. In several of your output messages, you've used it's when the possessive its would be grammatically correct. For example, in Bulldozer.java, the message should be "Bulldozer started its work". This is a common mistake, but correcting it will make your output more professional.
This doesn't affect the logic of your program at all. Keep up the great work!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| public class Bulldozer extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Bulldozer started it's work"); |
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.
There's a small grammatical error here. it's is a contraction for "it is". The possessive form is its. The message should be "Bulldozer started its work".
| public class Excavator extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Excavator started it's work"); |
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.
There's a small grammatical error in the message. it's is a contraction for "it is". The possessive form you're looking for is its.
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Excavator stopped it's working"); |
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.
Similar to the doWork method, the possessive pronoun here should be its, not it's.
| public class Truck extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Truck started it's work"); |
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.
There's a small grammatical error here. it's is a contraction for "it is". The possessive form is its. The message should be "Truck started its work".
No description provided.