First review #1
First review #1
Conversation
src/main/java/Car.java
Outdated
| return speed; | ||
| } | ||
|
|
||
| public void setName(String name) { |
There was a problem hiding this comment.
Сеттеры по идее здесь лишние, так как поля передаются при создании объекта в конструкторе
src/main/java/Main.java
Outdated
| name = scanner.next(); | ||
| while (true) { | ||
| System.out.println("— Введите скорость машины №" + i +":"); | ||
| speed = scanner.nextInt(); |
There was a problem hiding this comment.
Здесь при вводе нецелочисленных значений будет креш, который нигде не обрабатывается. Необходимо добавить обработку и попросить пользователя ввести данные заново
| private Car leader; | ||
| private Integer time = 24; | ||
|
|
||
| private Map<Car, Integer> participantsResults; |
There was a problem hiding this comment.
Можно обойтись без хранения всех машин в мапе, если сразу при вводе данных определять победителя, таким образом программа будет работать быстрее и требовать меньше памяти
There was a problem hiding this comment.
Ну мы не знаем как это приложение будет жить дальше. Может потом надо будет выводить список всех машин или первое второе и третье место. (имеется в виду условно). Поэтому я выбрал конструкцию с заделом на расширение
No description provided.