HW done #2134
HW done #2134
Conversation
There was a problem hiding this comment.
Great job on your code! 🌟 It looks like you've followed the guidelines and implemented the necessary functionality effectively. Keep up the good work, and always aim to provide clear and informative messages in your methods for better readability and maintenance. Happy coding! 👍
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| public class Bulldozer extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Bulldozer starts work"); |
There was a problem hiding this comment.
The message should indicate the specific type of machine that is working. Consider including 'Bulldozer' in the message for clarity, e.g., 'Bulldozer is starting its work.'
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Bulldozer stopped working"); |
There was a problem hiding this comment.
Similarly, the stop message should clearly indicate that it's the 'Bulldozer' that stopped working. For example, 'Bulldozer has stopped working.'
| public class Excavator extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Excavator starts work"); |
There was a problem hiding this comment.
The message in the doWork method should be more informative. According to the checklist, it should indicate the type of Machine that is working. Consider rephrasing it to include the word 'Excavator' explicitly, for example, 'Excavator is doing its work.'
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Excavator stopped working"); |
There was a problem hiding this comment.
Similarly, the message in the stopWork method should be more informative and indicate the specific type of Machine. You could change it to 'Excavator has stopped working.' to make it clearer.
| public class Truck extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Truck starts work"); |
There was a problem hiding this comment.
The message 'Truck starts work' could be more informative. According to the checklist, it should indicate what type of Machine is working, which is correct, but consider including more details about the work the Truck is doing.
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Truck stopped working"); |
There was a problem hiding this comment.
Similarly, the message 'Truck stopped working' could provide more context about the state of the Truck when it stops working. More informative messages would improve the clarity of the code behavior.
No description provided.