Conversation
.idea/.gitignore
Outdated
There was a problem hiding this comment.
Рекомендация: файлы директорий .idea и build стоит вынести в .gitignore
There was a problem hiding this comment.
Рекомендация: файлы директорий .idea и build стоит вынести в .gitignore
В связи с тем что мне дали заготовку проекта в котором уже был заполненный .gitignore - я не стал его трогать.
Вопрос к разработчикам заготовки - почему там вырезана только часть папок в idea? :) но я учту конечно на будущее, спасибо.
There was a problem hiding this comment.
Тут скорее суть комментария в том, чтобы в PR только нужные файлы. В нашем случае это классы, реализующие гонку. Разработчики заготовки оставили эти файлы для упрощения студентам запуска проекта в Android Studio.
src/main/java/Main.java
Outdated
| while (true) { | ||
| System.out.format("Введите скорость автомобиля '%s'(от 0 до 250км/ч):", name); | ||
| speed = Integer.parseInt(scanner.nextLine()); | ||
| if (speed <= 250 && speed >= 0) { |
There was a problem hiding this comment.
У меня же в условии speed <= 250. я проверил 250 можно ввести.
| public boolean isExist(String name) { | ||
| for (Car existingCar : cars) { | ||
| if (existingCar.name.equals(name)) { | ||
| return true; |
There was a problem hiding this comment.
Рекомендация: если нашли авто с таким имененм, значит, можно дальше по циклу не идти и сделать break
There was a problem hiding this comment.
если мы возвращаем true - то цикл автоматически должен прерваться, верно? тогда команда break зачем?
There was a problem hiding this comment.
да, согласна. Здесь цикл прерывается
src/main/java/Main.java
Outdated
| leader = car; | ||
| } | ||
| } | ||
| System.out.format("Финиш\nПобедил автомобиль '%s'. Он проехал трассу за %d ч.", leader.name, time /leader.speed); |
There was a problem hiding this comment.
Некорректный вывод: время гонки одно и то же, все ехали 24 часа. А вот расстояние все проехали разное
There was a problem hiding this comment.
Некорректный вывод: время гонки одно и то же, все ехали 24 часа. А вот расстояние все проехали разное
Поторопился на последнем шаге, каюсь :)
1. обработка пустого названия авто. 2. обработка при вводе скорости символов отличных от цифр. 3. вывод правильных результатов в пройденных километрах. 4. учет оставленных комментариев в PR 5. Доработка основного блока, вынос всей математики в классы.
.idea/.gitignore
Outdated
There was a problem hiding this comment.
Тут скорее суть комментария в том, чтобы в PR только нужные файлы. В нашем случае это классы, реализующие гонку. Разработчики заготовки оставили эти файлы для упрощения студентам запуска проекта в Android Studio.
| public boolean isExist(String name) { | ||
| for (Car existingCar : cars) { | ||
| if (existingCar.name.equals(name)) { | ||
| return true; |
There was a problem hiding this comment.
да, согласна. Здесь цикл прерывается
| *.iml | ||
| .gradle | ||
| /local.properties | ||
| /.idea/caches |
There was a problem hiding this comment.
Комментарий про "файлы директорий .idea и build стоит вынести в .gitignore" ты понял немного не так. Наоборот , /.idea/ и /build должны обязательно присутствовать в файле .gitignore для того, чтобы вынести их из-под версионного контроля. И что бы они каждый раз при перестроении проекта не попадали в PR
пока что без комментариев.