Skip to content

Попытка написать гонки номер раз#1

Open
PolinaKoko wants to merge 1 commit into
mainfrom
dev
Open

Попытка написать гонки номер раз#1
PolinaKoko wants to merge 1 commit into
mainfrom
dev

Conversation

@PolinaKoko
Copy link
Copy Markdown
Owner

Я реально очень не понимаю что делаю

Copy link
Copy Markdown

@ArturNurtdinov ArturNurtdinov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хорошая работа, молодец!

Comment thread src/main/java/Car.java
Comment on lines +2 to +3
private String name;
private int speed;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поля лучше пометить final, тем самым исключив возможность их модификации извне. Тогда геттеры можно будет удалить и безопасно получать доступ напрямую по названию поля

Comment thread src/main/java/Main.java
Comment on lines +15 to +19
while (carName.trim().isEmpty()) {
System.out.println("Ошибка: название не может быть пустым!");
System.out.println("Введите название машины:");
carName = scanner.nextLine();
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать

Comment thread src/main/java/Main.java
carSpeed = scanner.nextInt();
scanner.nextLine();

if (carSpeed > 0 && carSpeed <= 250) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants