После "свежего взгляда" на работу заметил косяки, так что исправил их сам. "Итоговым" вариантом я считаю коммит "(с исправлениями)", так что проверять нужно его. #1
Conversation
| String name; | ||
| int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне. Таким образом, геттеры можно будет удалить и получать значения полей по прямому доступу к ним
| int speed = scanner.nextInt(); | ||
| scanner.nextLine(); // Очистка буфера | ||
|
|
||
| if (speed > 0 && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы с говорящими названиями для повышения читабельности кода
| public class Race { | ||
| String currentLeaderName = "Лидер отсутствует"; | ||
| int currentLeaderDistance = 0; | ||
| ArrayList<Auto> cars = new ArrayList<>(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| String currentLeaderName = "Лидер отсутствует"; | ||
| int currentLeaderDistance = 0; |
There was a problem hiding this comment.
Данные поля можно сделать приватными (пометив их модификатором private), так как в данном классе для получения имени победителя уже есть геттер, а сами поля лучше всегда скрывать, чтобы не было возможности снаружи этого класса сломать ему логику работы
2 participants
No description provided.