***@***.**** commented on this pull request. Хорошая работа, молодец! ------------------------------ In src/main/java/Car.java <#1 (comment)> : > + String name; + int speed; Поля лучше пометить final, тем самым исключив возможность их модификации извне ------------------------------ In src/main/java/Car.java <#1 (comment)> : > + Car(){ + this(null,0); + } Пустой конструктор можно удалить, т.к. смысла в объекте, у которого name = null, а speed = 0, - мало ------------------------------ In src/main/java/Main.java <#1 (comment)> : > public class Main { public static void main(String[] args) { - System.out.println("Hello world!"); + Scanner input = new Scanner(System.in); + + ArrayList<Car> cars = new ArrayList<>(); От хранения массива машин и лишнего цикла при определении победителя можно избавиться, если при вводе данных сразу вычислять победителя и хранить его в отдельной переменной, тогда программа будет требовать меньше памяти и работать быстрее ------------------------------ In src/main/java/Main.java <#1 (comment)> : > + + ArrayList<Car> cars = new ArrayList<>(); + cars.add(new Car()); + cars.add(new Car()); + cars.add(new Car()); + + System.out.println("Привет! Сейчас тебе надо ввести данные о трех машинах, просто следуй инструкциям :)"); + + for (Car car : cars) { + System.out.println("Введите имя машины:"); + car.name = input.next(); + while (true) { + System.out.println("Введите скорость машины - просто напишите целое число от 1 до 250 включительно:"); + if (input.hasNextInt()) { + car.speed = input.nextInt(); + if (car.speed > 0 && car.speed <= 250) { Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода — Reply to this email directly, view it on GitHub <#1 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BQLWMGOMLGFTFQVR7JYUCV335KHNFAVCNFSM6AAAAACMM457VOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTINZVGMZDSNBVGQ> . You are receiving this because you authored the thread.Message ID: ***@***.***>