Repository navigation
Практическая работа №1 - #1
oshmeleva88 wants to merge 2 commits into
Conversation
| System.out.println("Ошибка! Введите целое число!"); | ||
| } | ||
| else { | ||
| speed = Integer.parseInt(input); //как парсить тоже спросила в Яндексе |
There was a problem hiding this comment.
Надо исправить: Integer.parseInt бросает NumberFormatException, если число не помещается в int (например, 9999999999 - regex такой ввод пропускает). Приложение упадёт со стектрейсом вместо повторного запроса, а по условию задания любое невалидное значение скорости должно переспрашиваться. Самое простое решение которое, так же закроет ошибку с парсингом нечислового значения - try/catch, но тогда необходимо добавить проверку на пустую строку
There was a problem hiding this comment.
Сделала try catch для проверки корректности ввода числа, на пустую строку у меня выше была проверка, я ее не меняла
| } | ||
| } No newline at end of file | ||
| public static class Car { | ||
| String name; //название автомобиля |
There was a problem hiding this comment.
Можно лучше: поля name/speed доступны снаружи и записываются напрямую (car.name = ...). Сделай их private и наполняй объект через конструктор - за данные объекта отвечает сам объект, и готовый конструктор Car(String, int) уже ждёт этого.
There was a problem hiding this comment.
С этим обязательно разберусь
| return car; | ||
| } | ||
| } No newline at end of file | ||
| public static class Car { |
There was a problem hiding this comment.
Можно лучше: Можно лучше: Car и Race вложены в Main.java. Требованию задания это соответствует, но в Java один публичный класс держат в отдельном файле (Car.java, Race.java)
There was a problem hiding this comment.
Сделала. Спасибо за подсказку, в дальнейшем буду классы помещать в отдельные файлы
2 participants
Создала PR из dev в main