Repository navigation
Проектная работа №1 - #1
ilinhustla wants to merge 1 commit into
Conversation
| } No newline at end of file | ||
|
|
||
|
|
||
| private static Car[] input2() { |
There was a problem hiding this comment.
Имя метода лучше делать говорящим - из него должно быть понятно, что метод делает и что возвращает. Цифра в конце или сокращение обычно остаются от черновика, и потом при чтении кода приходится каждый раз заглядывать внутрь, чтобы вспомнить, что там происходит.
| private static Car[] input2() { | ||
| Scanner scanner = new Scanner(System.in); | ||
| Car[] cars = new Car[3]; | ||
| for (int i = 0; i < 3; i++) { |
There was a problem hiding this comment.
Верхняя граница цикла захардкожена числом, хотя длина массива у тебя под рукой - стоит поменять количество элементов, и цикл либо пропустит часть данных, либо упадёт с ArrayIndexOutOfBoundsException. Надёжнее опираться на сам массив:
for (int i = 0; i < cars.length; i++) {| int speed = 0; | ||
|
|
||
|
|
||
| while (true) { |
There was a problem hiding this comment.
Код для считывания скорости с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| } | ||
|
|
||
| class Car { | ||
| String name; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| Car winner = null; | ||
| int maxDistance = 0; | ||
| for (Car current : cars) { | ||
| int distance = current.speed * 24; |
There was a problem hiding this comment.
Число 24 в формуле - магическая константа: из кода не видно, что это длительность гонки в часах. Такие значения лучше выносить в именованную константу, тогда формула читается сама по себе, а значение можно поменять в одном месте:
private static final int RACE_DURATION_HOURS = 24;
...
int currentDistance = RACE_DURATION_HOURS * car.getSpeed();2 participants
No description provided.