-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Do OOP Basic #2518
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?
Do OOP Basic #2518
Changes from all commits
bad9184
f981fa3
ea02aac
b0d3b01
d05667b
8231e64
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| package core.basesyntax; | ||
|
|
||
| public class Bulldozer extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Bulldozer is started work."); | ||
| } | ||
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Bulldozer is stopped working."); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The phrasing 'is stopped working' is also a bit awkward. A more common way to say this would be 'Bulldozer has stopped working.' or 'Bulldozer stopped working.' |
||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| package core.basesyntax; | ||
|
|
||
| public class Excavator extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Excavator is started work."); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The message "Excavator is started work." is a bit unnatural in English. For better readability, consider changing it to "Excavator has started work." or "Excavator started working." |
||
| } | ||
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Excavator is stopped work."); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The task description requires a message indicating the machine "stopped working". The current message "Excavator is stopped work." doesn't quite match. Please update it to something like "Excavator stopped working." |
||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| package core.basesyntax; | ||
|
|
||
| public abstract class Machine { | ||
|
|
||
| public abstract void doWork(); | ||
|
|
||
| public abstract void stopWork(); | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,11 @@ | ||
| package core.basesyntax; | ||
|
|
||
| public class MainApp { | ||
|
|
||
| public static void main(String[] args) { | ||
| Machine[] machines = {new Truck(), new Bulldozer(), new Excavator()}; | ||
| for (Machine machine : machines) { | ||
| machine.doWork(); | ||
| machine.stopWork(); | ||
| } | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| package core.basesyntax; | ||
|
|
||
| public class Truck extends Machine { | ||
| @Override | ||
| public void doWork() { | ||
| System.out.println("Truck is started work."); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This message is grammatically a bit awkward. A more natural phrasing in English would be "Truck started working." or "Truck has started its work." |
||
| } | ||
|
|
||
| @Override | ||
| public void stopWork() { | ||
| System.out.println("Truck is stopped work."); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similar to the |
||
| } | ||
| } | ||
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.
The phrasing 'is started work' is a bit unnatural in English. Consider rephrasing this to something like 'Bulldozer started to work.' or 'Bulldozer has started its work.' for better clarity.