Conversation
src/main/java/Car.java
Outdated
| String name; | ||
| int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
src/main/java/Main.java
Outdated
| if (!Arrays.asList(new String[]{ "да", "lf", "y", "yes"}).contains(continueAnswer.trim().toLowerCase())) { | ||
| System.out.println("До свидания"); | ||
| break; | ||
| } |
There was a problem hiding this comment.
В необходимости считывания ответа для продолжения нет, т.к. в задании это не указано. Из улучшений здесь могу предложить вынести массив допустимых слов в отдельную константу, чтобы каждый раз его заново здесь не создавать
src/main/java/Main.java
Outdated
| System.out.println("Введите скорость машины №" + carNumber); | ||
| if (scanner.hasNextInt()) { | ||
| int speed = scanner.nextInt(); | ||
| if (0 < speed && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
src/main/java/Race.java
Outdated
| Car currentLeader; | ||
| int durationHours = 24; |
There was a problem hiding this comment.
Обе переменные лучше сделать приватными, а для получения названия машины-победителя написать отдельную функцию
src/main/java/Race.java
Outdated
| Car currentLeader; | ||
| int durationHours = 24; | ||
|
|
||
| public Car getLeader(Car car) { |
There was a problem hiding this comment.
Лучше не писать в названии функции слово get, если она ничего не возвращает, более подходящим неймингом здесь может быть evaluateLeader или updateLeader
Проектная работа №1
try 1