Hw solution #2153
Hw solution #2153
Conversation
… and public abstract void stopWork(). Create classes Truck, Bulldozer, and Excavator that will inherit from Machine.
… and public abstract void stopWork(). Create classes Truck, Bulldozer, and Excavator that will inherit from Machine.
There was a problem hiding this comment.
Great job on implementing the Machine class as an abstract entity and ensuring that polymorphism is used effectively through the MainApp.java. Keep in mind the suggestions about making the log messages more informative by specifying the machine type. This will enhance clarity and maintainability. Keep up the good work, and remember to address the feedback in future iterations for continued improvement! 👍🚀
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
|
|
||
| public class Bulldozer extends Machine { | ||
| public void doWork() { | ||
| System.out.println("Bulldozer started its engine and began to work."); |
There was a problem hiding this comment.
The message should indicate what type of 'Machine' is working right now. It's better to specify that it's a 'Bulldozer' to make the message more informative.
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Bulldozer turned off its engine and stopped working."); |
There was a problem hiding this comment.
Similar to the doWork() method, the message in stopWork() should be more informative and indicate that it's a 'Bulldozer' that has stopped working.
|
|
||
| public class Excavator extends Machine { | ||
| public void doWork() { | ||
| System.out.println("Excavator started its engine and began to work."); |
There was a problem hiding this comment.
The message should be more informative and indicate that it is an 'Excavator' that has started working. For example: "Excavator is working."
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Excavator turned off its engine and stopped working."); |
There was a problem hiding this comment.
The stop message should also be more informative and specific to the 'Excavator'. For example: "Excavator has stopped working."
| public class Truck extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Truck started its engine and began to work."); |
There was a problem hiding this comment.
The message 'Truck started its engine and began to work.' could be more informative by specifying that it's a Truck that is working. Consider rephrasing it to include the type of Machine, for example: 'Truck is working.'
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Truck turned off its engine and stopped working."); |
There was a problem hiding this comment.
Similarly, the message 'Truck turned off its engine and stopped working.' could be more informative. It's recommended to rephrase it to clearly indicate the type of Machine, for example: 'Truck has stopped working.'
No description provided.