Repository navigation
Conversation
| String name; | ||
| short speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| continue; | ||
| } | ||
|
|
||
| if (speed <= 0 || speed > 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| } | ||
|
|
||
| break; | ||
| } while (true); |
There was a problem hiding this comment.
Не рекомендую писать бесконечные циклы через while (true) - лучше всегда явно прописывать условие выхода из цикла, чтобы уменьшить вероятность ошибиться и повысить читабельность кода
| Car car1 = createCar(1); | ||
| Car car2 = createCar(2); | ||
| Car car3 = createCar(3); |
There was a problem hiding this comment.
Этот код с созданием трёх машин лучше вынести в цикл, чтобы не копировать блоки кода друг за другом
| @@ -0,0 +1,23 @@ | |||
| public class Race { | |||
| Car car1, car2, car3; | |||
There was a problem hiding this comment.
От хранения всех машин можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
2 participants
Сделана первая работа в соответствии с требованиями.