Conversation
| while (true) { | ||
| System.out.println("Введите название автомобиля №" + i + ": "); | ||
| name = scanner.nextLine().trim(); | ||
| if (name.isEmpty()) { | ||
| System.out.println("Вы забыли ввести название автомобиля, попробуйте еще!"); | ||
| continue; | ||
| } | ||
| break; | ||
| } |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| } | ||
|
|
||
| speed = Integer.parseInt(input); | ||
| if (speed > 0 && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| System.out.println("Вы забыли ввести скорость, попробуйте еще!"); | ||
| continue; | ||
| } | ||
| if (!input.matches("\\d+")) { |
There was a problem hiding this comment.
Проверка input.matches("\\d+") не гаранитрует, что ниже parseInt не выбросит исключение - лучше обработать исключение, чтобы была гарантия отсутствия ошибок
| String name; | ||
| int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
| } | ||
| } | ||
|
|
||
| class Car { |
There was a problem hiding this comment.
Классы лучше в отдельных файлах объявлять, чтобы один файл не разрастался сильно
2 participants
Гоночки, тачки, скорость