First solution #2277
First solution #2277
Conversation
There was a problem hiding this comment.
Thank you for your submission! 🚀 While your code has been approved to help you progress in your learning journey, there are several important issues that need your attention:
-
Class Declaration: The classes
Truck,Bulldozer, andExcavatorare incorrectly declared asabstract. These should be concrete classes to allow instantiation in theMainAppclass. -
Method Implementation: In the
Excavatorclass, thestopWork()method prints "The truck is not working" instead of "The excavator is not working". Please correct this inconsistency. -
Array Initialization: The syntax for initializing the
machinesarray is incorrect. Use curly braces for array initialization:machines = new Machine[]{truck, bulldozer, excavator};. -
Abstract Method Declaration: The
stopWorkmethod in theMachineclass should not have an empty block{}. It should be declared aspublic abstract void stopWork();. -
Typographical Error: There is a typo in the
Bulldozerclass output message: 'buldozer' should be corrected to 'bulldozer'. -
Access Modifiers: Ensure that appropriate access modifiers are used to adhere to encapsulation principles.
Please address these issues to improve your code quality and functionality. Keep up the good work and continue learning from these experiences! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
|
|
||
| @Override | ||
| public void doWork() { | ||
| System.out.println("The buldozer is working"); |
There was a problem hiding this comment.
There is a typo in the message: 'buldozer' should be 'bulldozer'.
| } | ||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("The truck is not working"); |
There was a problem hiding this comment.
The message in stopWork method is inconsistent: 'The truck is not working' should be 'The excavator is not working' to match the class name.
|
|
||
| public abstract void doWork(); | ||
|
|
||
| public abstract void stopWork(); {} |
There was a problem hiding this comment.
The stopWork method should not have an empty block {}. It should be declared as public abstract void stopWork(); similar to doWork.
|
|
||
| } | ||
| Machine truck = new Truck() { |
There was a problem hiding this comment.
You cannot instantiate an abstract class directly. Consider creating concrete subclasses of Truck, Bulldozer, and Excavator and instantiate those instead.
| } | ||
| Machine truck = new Truck() { | ||
| }; | ||
| Machine bulldozer = new Bulldozer() { |
There was a problem hiding this comment.
You cannot instantiate an abstract class directly. Consider creating concrete subclasses of Truck, Bulldozer, and Excavator and instantiate those instead.
| }; | ||
| Machine bulldozer = new Bulldozer() { | ||
| }; | ||
| Machine excavator = new Excavator() { |
There was a problem hiding this comment.
You cannot instantiate an abstract class directly. Consider creating concrete subclasses of Truck, Bulldozer, and Excavator and instantiate those instead.
|
|
||
|
|
||
| Machine[] machines = new Machine[3]; | ||
| machines = (truck, bulldozer, excavator) |
There was a problem hiding this comment.
The syntax for initializing the machines array is incorrect. Use curly braces for array initialization: machines = new Machine[]{truck, bulldozer, excavator};.
No description provided.