Repository navigation
Проектная работа №1 спринта №2 - #1
sapphire-hydrangea wants to merge 1 commit into
Conversation
| String name; | ||
| int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| Scanner scanner = new Scanner(System.in); | ||
| ArrayList<Car> cars = new ArrayList<>(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| carName = carName.trim(); | ||
| while (carName.isEmpty()) { | ||
| System.out.println("Введена пустая строка. Введите название автомобиля №" + (i + 1) + " повторно:"); | ||
| carName = scanner.nextLine(); | ||
| carName = carName.trim(); | ||
| } |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| System.out.println("Введено не число. Введите скорость автомобиля №" + (i + 1) + " повторно:"); | ||
| continue; | ||
| } | ||
| if (speed <= 0 || speed > 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| } | ||
|
|
||
| System.out.println("Введите скорость автомобиля №" + (i + 1) + ":"); | ||
| int speed; |
There was a problem hiding this comment.
Аналогично код для считывания скорости лучше вынести в отдельную функцию
2 participants
No description provided.