Repository navigation
Conversation
| String name; | ||
| int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| int carSpeed; | ||
| String cancelInput = "Неправильный ввод. Попробуйте еще раз"; | ||
|
|
||
| String checkInputName() { |
There was a problem hiding this comment.
Здесь check не самый подходящий глагол для нейминга функции, т.к. функция не только проверяет, но считывает и возвращает считанное значение, гарантируя его корректность. Лучше назвать функцию например readName
| } | ||
| } | ||
|
|
||
| int checkInputSpeed() { |
There was a problem hiding this comment.
С этой функцией аналогично комментарию выше
| Race race; | ||
|
|
||
| System.out.println("Давайте отправимся на автогонку \"24 часа Ле-Мана\" - и понаблюдаем за исходом событий!"); | ||
| cars = new Car[3]; |
There was a problem hiding this comment.
В программе можно избавиться от хранения массива и лишней итерации цикла по массиву машин, определяя победителя и сохраняя только его при вводе значений, таким образом программа будет работать быстрее и требовать меньше памяти
| int bestDistance; | ||
| String nameOfWinner; | ||
| int distance; |
There was a problem hiding this comment.
Здесь эти переменные лучше спрятать за модификатором видимости private, а для получения машины-победителя использовать возвращаемое значение функции calculate
There was a problem hiding this comment.
Большое спасибо за подробный фидбек!
2 participants
«24 часа Ле-Мана» — консольное приложение, которое будет задавать вопросы в консоли, считывать пользовательский ввод и в результате выводить победителя гонки.