Repository navigation
Conversation
| private String name; | ||
| private int speed; |
There was a problem hiding this comment.
Поля лучше пометить final, тогда можно будет убрать private модификатор и удалить геттеры - будет напрямую доступ к полям только на чтение
| Scanner scanner = new Scanner(System.in); | ||
| for (int i = 1; i < 4; i++) { | ||
|
|
||
| while (true) { |
There was a problem hiding this comment.
Не рекомендую писать бесконечные циклы через while (true) - лучше всегда явно прописывать условие выхода из цикла, чтобы уменьшить вероятность ошибиться и повысить читабельность кода
| name = scanner.nextLine().trim(); | ||
| if (name.isEmpty()) { |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| } | ||
| } | ||
|
|
||
| while (true) { |
There was a problem hiding this comment.
Код для считывания скорости с ввода аналогично лучше вынести в отдельную функцию
| showErrorInvalidCharacters(); | ||
| } else { | ||
| speed = Integer.parseInt(inputSpeed); | ||
| if (speed > 0 && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести вконстанты для повышения читабельности кода
2 participants
No description provided.