Repository navigation
Conversation
There was a problem hiding this comment.
Сами скомпилированные файлы .class не нужно копировать в source директории и пушить на гит
| System.out.println("Hello world!"); | ||
| //Ввод данных | ||
| InputData input = new InputData(); | ||
| Car[] cars = input.getCars(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| name = scanner.nextLine(); | ||
|
|
||
| // Проверяем, что имя не пустое | ||
| if (name.trim().isEmpty()) { |
There was a problem hiding this comment.
Вместо .trim().isEmpty() можно сделать isBlank() - тогда не будет создаваться новый объект для этой проверки
| speed = Integer.parseInt(input); | ||
|
|
||
| //Проверка диапазона | ||
| if (speed > 0 && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| String name; // Название | ||
| int speed; // Скорость (км/ч) | ||
| int distance; // Расстояние за 24 часа езды (км) |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
|
|
||
| do { | ||
| System.out.print("Введите имя машины " + (i + 1) + ": "); | ||
| name = scanner.nextLine(); |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
2 participants
No description provided.