Repository navigation
Conversation
| public class Main { | ||
| public static void main(String[] args) { | ||
| System.out.println("Hello world!"); | ||
| ArrayList<Cars> automobile = new ArrayList<>(); |
There was a problem hiding this comment.
От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее
| Scanner scanner = new Scanner(System.in); | ||
| String name; | ||
| Cars car; | ||
| while(true) { |
There was a problem hiding this comment.
Не рекомендую писать бесконечные циклы через while (true) - лучше всегда явно прописывать условие выхода из цикла, чтобы уменьшить вероятность ошибиться и повысить читабельность кода
| while(true) { | ||
| System.out.println("Введите название " + i + "-го автомобиля:"); | ||
| name = scanner.nextLine(); | ||
| if (!name.isEmpty()) { |
There was a problem hiding this comment.
Код для считывания непустой строки с ввода лучше вынести в отдельную функцию - код, разделённый на небольшие функции, легче читать, поддерживать и переиспользовать
| Cars firstCar = new Cars(inputName(1), inputSpeed(1)); | ||
| arrival.setWiner(firstCar); | ||
| Cars secondCar = new Cars(inputName(2), inputSpeed(2)); | ||
| arrival.setWiner(secondCar); | ||
| Cars thirdCar = new Cars(inputName(3), inputSpeed(3)); | ||
| arrival.setWiner(thirdCar); |
There was a problem hiding this comment.
От хранения массива ты избавился, молодец, однако лучше все таки чтение данных и определение победителя делать в цикле, тут явно прослеживается копирование одного куска кода три раза - повод вынести это в цикл
2 participants
No description provided.