Conversation
| String name; | ||
| int speed; | ||
|
|
||
| int distance; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
|
|
||
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| ArrayList<Car> carList = new ArrayList<>(); |
There was a problem hiding this comment.
Можно не хранить весь список машин, а победителя гонки сразу вычислять при вводе данных. Таким образом, программа будет требовать меньше памяти и работать быстрее
| carList.sort(Comparator.comparingInt(Car::getDistance).reversed()); | ||
|
|
||
| return carList.get(0); |
There was a problem hiding this comment.
Чтобы определить максимальную distance сортировать весь список излишне, можно просто в цикле for сразу определять максимальную distance и записывать соответствующую машину в переменную, в таком случае будет гораздо меньше операций
src/main/java/Main.java
Outdated
| int speed = scanner.nextInt(); | ||
| if(speed<0 || speed>250){ | ||
| while (speed<0 || speed>250){ | ||
| System.out.println("Вы указали неправильную скорость"); | ||
| System.out.println("Введите скорость машины № " + i); | ||
| speed = scanner.nextInt(); | ||
| } | ||
| } |
There was a problem hiding this comment.
Такую конструкцию удобнее написать в do-while цикле, либо сделать просто while(true) и выходить из этого цикла через break оператор
|
Добавил проверку корректного значения для скорости. И я снова сделал неправильный ПР. В этот раз вроде правильный ПР и я просто добавил новый коммит, где исправлен код. |
Привет, с ПРом все правильно сделал. На первую проверку спринта присылаешь новый ПР, а правки, если требуются, вносишь в существующий |
Не совсем понял что с чем сравнивать.
Как я понял текущий вариант ПР правильный.