Conversation
| public String name; | ||
| public int velocity; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| // каждый из которых имеет два параметра: | ||
| // название и скорость. | ||
| Scanner scanner = new Scanner(System.in); | ||
| private ArrayList<Auto> raceAutos = new ArrayList<>(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| break; | ||
| } | ||
|
|
||
| while (true) { |
There was a problem hiding this comment.
Не рекомендую писать бесконечные циклы через while (true) - лучше всегда явно прописывать условие выхода из цикла, чтобы уменьшить вероятность ошибиться и повысить читабельность кода
| while (true) { | ||
| System.out.printf("Введите название авто учаcтника №%d: ", index); | ||
| name = scanner.next(); | ||
| if (!isEmptyValue(name)) { |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
2 participants
Добавил классы для авто, рейса, интерфейса ввода данных от пользователя. Выполняется проверка валидности входных данных.