-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
...ынес дефолтные параметры.
Поправил Readme
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Молодец, что вынес эти значения в параметры, но лучше их сделать настоящими константами, для этого нужно добавить модификатор final и переименовать в соответствии со SCREAMING_SNAKE_CASE, например так:
public static final int TOTAL_OF_CARS = 3;
В проекте 4 основных класса:
Main - точка входа в программу и организацию взаимодействия между классами;
InteracterWithUserViaConsole - класс отвечает за взаимодействие с пользователем и валидацию данных;
Race - класс для выбора победителя;
Car - данные по участникам гонки;