Проектная работа №1 - #1

Pull request сделан правильно

Еще есть вопрос по самой работе. По заданию предлагают создать 3 класса: 1. Автомобиль. В моем случае это vehicle. В строке 8 я объявил переменную vehicle типа ArrayList и в строке 21 я заполняю массив данными, которые пользователь вводит с клавиатуры. Но вычисления для победителя делаются в строке 22 в методе класса race, без использования данных из класса vehicle. Я могу полностью убрать класс Автомобиль, строки в main где я его заполняю и тд, и все равно задача будет решена. Следовательно вопрос, зачем нам класс Автомобиль?)
Спасибо!

Класс Автомобиль нужен для абстракции физического объекта автомобиль. Сейчас в рамках учебного задания достаточно 2х свойств реального автомобиля: скорости и названия. Но что, если программа будет более сложной , и для ее функционирования нужно будет 10-20-30-100 свойств? Поэтому в класс Гонка нужно передавать не 2 свойства автомобиля, а полностью экземпляр Автомобиля

Я сначала исправлял ошибки по комментариям здесь и на почту, потом полез в практикум, а там еще) поэтому 3 коммита) Не понял как удалять не нужные...
Для каждой новой проверки необходимо закрывать и создавать новый ПР или все скидывать сюда и ссылка в практикуме каждый раз одна?

Вопрос, на сайте практикума указаны "Выполнение требований задания:" и "Критические замечания:", я должен устранить только то, что прописано в критических замечаниях или еще отдельно искать в "Выполнение требований задания:"?

Критические замечания:

• "можно ввести пустое имя автомобиля" - исправил
• "& - побитовая операция. Здесь нужно использовать &&" - исправил, но есть вопрос. Погуглив понял, что при & проверяются в любом случае все условия, в && при первом false дальнейшие условия проверяться не будут. Проверял в своем варианте с & тоже верно, разве нет? Скинул скрин
  

Следовательно вопрос, это критическое замечание касается и того, что написано в "Выполнение требований задания: ❌ Приложение умеет обрабатывать неправильно введённую скорость (правильная — от 0 до 250). Оно сообщает об этом пользователю и запрашивает скорость заново." Или это отдельная ошибка? Если отдельно, то у меня стоит отлов на ввод символов или вне диапазона.

Замечания, которые были в рекомендациях, также исправил. Правильно ли я понял, что необходимо было изменить метод класса Race?

Для каждой новой проверки необходимо закрывать и создавать новый ПР или все скидывать сюда и ссылка в практикуме каждый раз одна?

на каждый спринт 1 ПР. В случае, если нужно исправить замечания, изменения с помощью комита добавляются в ту же ветку и автоматом попадают в тот же ПР.

Вопрос, на сайте практикума указаны "Выполнение требований задания:" и "Критические замечания:", я должен устранить только то, что прописано в критических замечаниях или еще отдельно искать в "Выполнение требований задания:"?

Обязательно к исправлению то, что помечено , как критическое( обычно прописывают в критических замечаниях, но иногда могут и в комментариях к ПР. Остальные замечания- не обязательны, но очень желательны. Как правило они отображают подходы к реальной разработке

& - побитовая операция. Здесь нужно использовать &&" - исправил, но есть вопрос. Погуглив понял, что при & проверяются в любом случае все условия, в && при первом false дальнейшие условия проверяться не будут. Проверял в своем варианте с & тоже верно, разве нет? Скинул скрин

Да, результат будет верным. здесь & применен к булевым типам. И будут выполнены обе проверки. Правильнее использовать && . Подробно написано здесь: https://sky.pro/wiki/java/otlichie-operatorov-and-i-and-and-v-java-kogda-i-kak-ispolzovat/

Следовательно вопрос, это критическое замечание касается и того, что написано в "Выполнение требований задания: ❌ Приложение умеет обрабатывать неправильно введённую скорость (правильная — от 0 до 250). Оно сообщает об этом пользователю и запрашивает скорость заново." Или это отдельная ошибка? Если отдельно, то у меня стоит отлов на ввод символов или вне диапазона.

Это замечание относится к ✅❌ Приложение успешно компилируется и выполняется без ошибок.