home work#1
Conversation
| public String toString() { | ||
| return "Машина: " + name + ", Скорость: " + speed + " км/ч"; | ||
| } |
There was a problem hiding this comment.
Лучше не переопределять toString под определённый формат, т.к. скорость может задаваться не только в км/ч. Лучше отдельный метод написать
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| Scanner scanner = new Scanner(System.in); | ||
| List<Car> cars = new ArrayList<>(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| if (speed < 1 || speed > 250) { | ||
| System.out.println("Неправильно! Скорость должна быть от 1 до 250 км/ч. Введите заново."); | ||
| } | ||
| } while (speed < 1 || speed > 250); |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| double minTime = Double.MAX_VALUE; | ||
|
|
||
| for (Car car : cars) { | ||
| double time = car.getTimeForDistance(DISTANCE_KM); |
There was a problem hiding this comment.
По условию задания гонка длится 24 часа, поэтому в данном случае логику можно упростить, просто умножив скорость на 24 и сравнивая полученное значение
No description provided.