Repository navigation
Conversation
| public String name; | ||
| public Integer speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| public Car() { | ||
| name = null; | ||
| speed = null; | ||
| } |
There was a problem hiding this comment.
Лучше передать значения этим полям в конструкторе, чтобы сделать два поля неизменяемыми - это упростит работу с ними
| @@ -0,0 +1,9 @@ | |||
| public class Leader { | |||
There was a problem hiding this comment.
Думаю, что для победителя гонки не стоит делать отдельный класс, т.к. по сути уже есть класс Car и именно объект этого класса считается победителем в гонке
| public class Race { | ||
| private static final int timeInHours = 24; | ||
| private final int maxCarsCounter; | ||
| HashSet<Car> cars; |
There was a problem hiding this comment.
От хранения сета машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| } | ||
|
|
||
| break; | ||
| } catch (java.util.InputMismatchException e) { |
There was a problem hiding this comment.
Лучше импортировать класс, чтобы не писать имя пакета в коде
2 participants
Проектная работа №1