Repository navigation
Conversation
| public String name; | ||
| public int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| } | ||
|
|
||
| System.out.println("Самая быстрая машина: " + race.leaderName); | ||
| scanner.close(); |
There was a problem hiding this comment.
Молодец, что не забыл закрыть сканнер - это очень важно)
| String leaderName = ""; | ||
| int maxDistance = 0; | ||
| int timeOfTheRace = 24; |
There was a problem hiding this comment.
Данные переменные лучше сделать приватными, чтобы извне нельзя было сломать логику работы этого класса, а для получения названия машины-победителя написать отдельную функцию геттер. Таким образом, весь алгоритм по определению победителя будет инкапсулирован в этом классе - это облегчит тестирование и поддержку этой функциональности
| System.out.print("Введите скорость машины №" + carNumber + " (от 1 до 250): "); | ||
| String input = scanner.nextLine(); | ||
|
|
||
| int speed = Integer.parseInt(input); |
There was a problem hiding this comment.
Данный вызов может выбросить креш, если вместо числа ввести символы - этот эксепшн необходимо обработать, предложив пользователю ввести данные заново
2 participants
No description provided.