Conversation
| final int RaceleMan = 24; | ||
| final int CarCount = 3; |
There was a problem hiding this comment.
Эти две переменные можно положить в константы на уровне класса, чтобы не объявлять их внутри метода, там же их можно назвать в соответствии с SCREAMING_SNAKE_CASE как настоящие константы)
| final int RaceleMan = 24; | ||
| final int CarCount = 3; | ||
|
|
||
| Auto[] cars = new Auto[CarCount]; |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| String input = scanner.nextLine().trim(); | ||
| try { | ||
| speed = Integer.parseInt(input); | ||
| if (speed > 0 && speed <= 250) { |
There was a problem hiding this comment.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
| private final String name; | ||
| private final int speed; |
There was a problem hiding this comment.
Так как переменные помечены модификатором final, то можно сделать их публичными и убрать геттеры, получая доступ к данным напрямую по имени поля - будет так же надёжно и безопасно
| return speed; | ||
| } | ||
|
|
||
| public int distanceЗа24Часа() { |
There was a problem hiding this comment.
В коде лучше не использовать кириллицу, так как это сильно затруднит чтение кода. Код пишется обычно только на одном языке. На русском можно написать комментарий или документацию к методу
2 participants
Привет, моя первая работа