Repository navigation
Conversation
| String name; | ||
| double speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| while (true) { | ||
| System.out.println("Введите название машины №" + (i + 1) + ":"); | ||
| name = scanner.nextLine().trim(); | ||
| if (!name.isEmpty()) { | ||
| break; | ||
| } else { | ||
| System.out.println("Ошибка! Название не может быть пустым"); | ||
| } | ||
| } |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| } | ||
|
|
||
| double speed; | ||
| while (true) { |
There was a problem hiding this comment.
Аналогично код для считывания скорости лучше вынести в отдельную функцию
| String input = scanner.nextLine().trim(); | ||
| try { | ||
| speed = Double.parseDouble(input); | ||
| if (speed > 0 && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| double winnerDistance = 0; | ||
|
|
||
| void updateWinner(Car car) { | ||
| double distance = car.getSpeed() * 24; |
There was a problem hiding this comment.
24 лучше вынести в константу private static final int RACE_DURATION_HOURS =24; для повышения читабельности кода и чтобы избавиться от захардкоженных чисел
2 participants
No description provided.