-
Notifications
You must be signed in to change notification settings - Fork 0
Java-Module-Project-YP-2nd-try #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…7 на 28.08) спутали все планы. Я сотрудник IT AFL. Сейчас уже стало попроще, следующий спринт очень постараюсь задержать не больше чем на 1-2 дня а может и успею вовремя))). Спасибо!!!
…7 на 28.07) спутали все планы. Я сотрудник IT AFL. Сейчас уже стало попроще, следующий спринт очень постараюсь задержать не больше чем на 1-2 дня а может и успею вовремя))). Спасибо!!!
| private String _name; // Название машинки. | ||
| private int _speed; //Скорость машинки. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Поля лучше пометить final, тем самым исключив возможность их модификации извне. Тогда можно будет убрать нижнее подчеркивание из нейминга и получать прямой доступ к полям - будет меньше кода, но также безопасно и надёжно
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| Scanner scanner = new Scanner(System.in); // Скнер для ввода данных. | ||
| Car[] cars = new Car[3]; // Массив размерностью в 3 машинки могламно условию. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| int speed; | ||
| while (true) { | ||
| System.out.println("Введите его скорость (1–250):"); | ||
| speed = scanner.nextInt(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Если тут будет введено не целочисленное значение, то тут будет выброшен эксепшн, который не обрабатывается и приводит к закрытию приложения
| speed = scanner.nextInt(); | ||
| scanner.nextLine(); //Странно что в JAVA nextInt() не очищает буффер ввода!!!!!! | ||
|
|
||
| if (speed >0 && speed <=250){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
2nd try...