Repository navigation
Conversation
| while (true) { | ||
| carName = scanner.nextLine().trim(); | ||
| if (carName.isEmpty()) { | ||
| System.out.println("Название автомобиля не может быть пустым или состоять из пробелов, повторите ввод"); | ||
| } else { | ||
| break; | ||
| } | ||
| } |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| while (true) { | ||
| if (scanner.hasNextInt()) { | ||
| carSpeed = scanner.nextInt(); | ||
| if (carSpeed < 0 || carSpeed > 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| } | ||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
Лишние пустые строчки лучше удалить, чтобы не засоряли код. Достаточно использовать 1 пустую строку для разделения логически делимых блоков в коде
| @@ -0,0 +1,14 @@ | |||
| public class Race { | |||
| String leader; | |||
There was a problem hiding this comment.
Саму переменную лучше сделать приватной, чтобы исключить возможность изменения её значения извне, а для получения результата гонки написать отдельную функцию-геттер
| String carName; | ||
| int carSpeed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
2 participants
No description provided.