-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
...7 на 28.08) спутали все планы. Я сотрудник IT AFL. Сейчас уже стало попроще, следующий спринт очень постараюсь задержать не больше чем на 1-2 дня а может и успею вовремя))). Спасибо!!!
...7 на 28.07) спутали все планы. Я сотрудник IT AFL. Сейчас уже стало попроще, следующий спринт очень постараюсь задержать не больше чем на 1-2 дня а может и успею вовремя))). Спасибо!!!
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.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
2nd try...