Создал новую ветку controllers-films-users. #1

ViktorSalk · 2025-01-01T10:28:04Z

Создал пакет model. Реализовал в нем классы: Film и User. Это классы — модели данных приложения. Создал два класса-контроллера. FilmController для фильмов, и UserController для пользователей. Создал UserNotFoundException, ValidationException, GlobalExceptionHandler для валидации. Создал тесты.

afomkina

Очень рекоммендую попробовать сделать дополнительное задание, так как сприговая валидация очень популярна в коммерческой разработке и никто не пишет валидацию руками. Она очень сильно сокращает код и избавляет от ошибок "невнимательности". Вот отлчиная статься про аннотации валидации

afomkina · 2025-01-02T13:19:58Z

src/main/java/ru/yandex/practicum/filmorate/FilmorateApplication.java


+		final Gson gson = new Gson();
+		final Scanner scanner = new Scanner(System.in);
+		System.out.print("Введите JSON => ");


Код со строк 17-27 нужно удалить

в FilmorateApplication удалил лишний код.

afomkina · 2025-01-02T13:20:09Z

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

+    }
+
+    @GetMapping
+    public Collection<Film> findAll() {


Лучше возвращать не Collection менее абстрактный интерфейс - List, так мы точно будем знать какой тип коллекции ожидать в ответе

Использовал более конкретный тип List вместо Collection, сделав API более понятным и защитив его от возможных изменений извне.

afomkina · 2025-01-02T13:20:37Z

src/main/java/ru/yandex/practicum/filmorate/controller/UserController.java

+    }
+
+    @GetMapping
+    public Collection<User> findAll() {


Лучше возвращать не Collection менее абстрактный интерфейс - List, так мы точно будем знать какой тип коллекции ожидать в ответе

Использовал более конкретный тип List вместо Collection, сделав API более понятным и защитив его от возможных изменений извне.

afomkina · 2025-01-02T13:21:07Z

src/main/java/ru/yandex/practicum/filmorate/exceptions/GlobalExceptionHandler.java

+public class GlobalExceptionHandler {
+
+    @ExceptionHandler(ValidationException.class)
+    public ResponseEntity<Map<String, String>> handleValidationException(ValidationException ex) {


Вместо Map<String, String> лучше взвращать объект ErrorResponse, в котором будет поле error, куда нужно будет записывать причину ошибки

Можно сделать код еще лаконичнее, избавившись от обертки ResponseEntity. Для этого нужно просто добавить аннотацию @ResponseStatus(HttpStatus. BAD_REQUEST) над методом, а в самом методе возвращать ErrorResponse.

Вместо Map<String, String> использовал @ResponseStatus(HttpStatus. BAD_REQUEST), создав ErrorResponse.

afomkina · 2025-01-02T13:23:21Z

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

+@Data
 public class Film {
+
+    Long id;


У всех полей класса должен быть модификатор доступа

Указал модификатор доступа для полей класса, для геттеров и сеттеров при аннотации использовал приватный.

afomkina · 2025-01-02T13:23:25Z

src/main/java/ru/yandex/practicum/filmorate/model/User.java

+
+@Data
+public class User {
+


У всех полей класса должен быть модификатор доступа

Указал модификатор доступа для полей класса, для геттеров и сеттеров при аннотации использовал приватный.

@RequestBody

…ms-users 1) в FilmorateApplication удалил лишний код. 2) в FilmController и UserController использовал более конкретный тип List вместо Collection сделав API более понятным и защищая от возможных изменений извне. 3) в GlobalExceptionHandler вместо Map<String, String> использовал @ResponseStatus(HttpStatus. BAD_REQUEST) создав ErrorResponse. 4) в Film и User указал модификатор доступа для полей класса, для геттеров и сеттеров при аннотации, использовал приватный. 5) Реализовал дополнительное задание используя спринговую валидацию, С помощью аннотаций задав ограничения, которые будут проверяться автоматически: подредактировал Film и User, FilmController и UserController, FilmControllerTests и UserControllerTests. Чтобы Spring не только преобразовал тело запроса в соответствующий класс, но и проверил корректность переданных данных, вместе с аннотацией @RequestBody использовал аннотацию @Valid.

…онтроля стиля.

…я стиля.

…tests.

ViktorSalk · 2025-01-02T21:37:01Z

Очень рекомендую попробовать выполнить дополнительное задание, так как сприговая валидация очень популярна в коммерческой разработке, и никто не пишет валидацию вручную. Она значительно сокращает код и избавляет от «невнимательных» ошибок. Вот отличная статья об аннотациях валидации

Спасибо) Реализовал дополнительное задание, используя проверку Spring, с помощью аннотаций задав ограничения, которые будут проверяться автоматически: подредактировал классы Film и User, контроллеры FilmController и UserController, тесты FilmControllerTests и UserControllerTests. Чтобы Spring не только преобразовывал тело запроса в соответствующий класс, но и проверял корректность переданных данных, вместе с аннотацией @RequestBody использовал аннотацию @Valid.

afomkina · 2025-01-03T15:38:37Z

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

+    }
+
+    @PostMapping
+    public ResponseEntity<Film> createFilm(@Valid @RequestBody Film film) {


Здесь лучше тоже использовать @ResponseStatus(HttpStatus.CREATED), чтобы избавиться от дополнительной обертки ResponseEntity и просто возвращать Film

В FilmController в createFilm использовал @ResponseStatus(HttpStatus.CREATED), чтобы избавиться от дополнительной обертки.

afomkina · 2025-01-03T15:38:54Z

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

+    }
+
+    @PutMapping
+    public ResponseEntity<Film> updateFilm(@Valid @RequestBody Film film) {


Нужно использовать @ResponseStatus

В FilmController в updateFilm использовал @ResponseStatus(HttpStatus.ОК).

afomkina · 2025-01-03T15:41:26Z

src/main/java/ru/yandex/practicum/filmorate/exceptions/ErrorResponse.java

+public class ErrorResponse {
+    private final String error;
+
+    public ErrorResponse(String error) {


@Data включает в себя аннотацию @RequiredArgsConstructor, поэтому явно его объявлять не нужно

В ErrorResponse убрал явное объявление.

afomkina · 2025-01-03T15:42:21Z

src/main/java/ru/yandex/practicum/filmorate/exceptions/GlobalExceptionHandler.java

+
+    @ExceptionHandler(UserNotFoundException.class)
+    @ResponseStatus(HttpStatus.NOT_FOUND)
+    public ErrorResponse handleUserNotFoundException(UserNotFoundException ex) {


Лучше явно прописать название переменной, как exception. Без сокращений код выглядит более читаемо

В GlobalExceptionHandler убрал сокращения.

afomkina · 2025-01-03T15:47:59Z

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

+    private Long id;
+
+    @NotNull(message = "Название фильма не может быть пустым")
+    @NotBlank(message = "Название фильма не может быть пустым")


На самом деле @NotBlank содержет проверку на null, поэтому добавлять дополнительно @NotNull не нужно

Спасибо) В Film убрал лишние @NotNull.

afomkina · 2025-01-03T15:51:28Z

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

+
+    @NotNull(message = "Описание фильма не может быть пустым")
+    @NotBlank(message = "Описание фильма не может быть пустым")
+    private String description;


Чтобы полностью избавиться от ручной валидации, нужно добавить @Size

В Film description добавил @SiZe на 200 символов.

afomkina · 2025-01-03T15:53:06Z

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

+    private String description;
+
+    @NotNull(message = "Дата выхода не может быть пустой")
+    private LocalDate releaseDate;


Можно реализовать кастомную аннотацию валидации, чтобы полностью отказаться от ручной валидации.
Вот отличная статья, в которой подробно описано, как это можно сделать

Спасибо) Реализовал кастомную аннотацию валидации ReleaseDateValidation для Film.

afomkina · 2025-01-03T15:54:13Z

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

+    private LocalDate releaseDate;
+
+    @NotNull(message = "Длительность не может быть пустой")
+    private Integer duration;


@PositiveOrZero проверяет, что число больше либо равно нулю

В Film duration добавил @PositiveOrZero на длительность фильма.

afomkina · 2025-01-03T15:57:12Z

src/main/java/ru/yandex/practicum/filmorate/controller/UserController.java

+        user.setId(getNextId());
+
+        // Проверка на существующий логин
+        if (users.values().stream().anyMatch(existingUser -> existingUser.getLogin().equals(user.getLogin()))) {


Дополнительные оптимизации
Чтобы проверка почты проходила за O(1), нужно завести Set, куда при добавлении нового пользователя мы будем класть его email. Соответственно, перед тем как добавить пользователя, нужно будет посмотреть, есть ли уже в сете подобный email, если есть - то пользователь с таким email уже существует.

Так как поиск и вставка в сете работают в среднем за O(1), то наш алгоритм поиска одинаковых email тоже будет работать в среднем за O(1), а это гораздо быстрее, чем пройтись по всем элементам списка O(n)

В UserController провел дополнительную оптимизацию алгоритма поиска, чтобы проверка почты проходила быстрее, за O(1).

afomkina · 2025-01-03T15:58:09Z

src/main/java/ru/yandex/practicum/filmorate/controller/UserController.java

+            throw new ValidationException("Ошибка. Попытка добавить логин, который уже существует");
+        }
+
+        if (user.getName() == null || user.getName().isBlank()) {


Эту проверку можно перенести в конструктор класса User, тогда поле будет автоматически проставляться сразу при создании экземпляра класса

Перенес проверку из UserController в конструктор класса User, создав метод setLogin.

@SiZe

…us.CREATED), чтобы избавиться от дополнительной обертки. 2) В FilmController в updateFilm использовал @ResponseStatus(HttpStatus.ОК). 3) В ErrorResponse убрал явное объявление. 4) В GlobalExceptionHandler убрал сокращения. 5) В Film убрал лишние @NotNull. 6) В Film description добавил @SiZe на 200 символов. 7) Реализовал кастомную аннотацию валидации ReleaseDateValidation для Film. 8) В Film duration добавил @PositiveOrZero на длительность фильма. 9) В UserController провел дополнительную оптимизацию алгоритма поиска, чтобы проверка почты проходила быстрее, за O(1). 10) Перенес проверку init из UserController в конструктор класса User. 11) В User убрал лишние @NotNull.

afomkina · 2025-01-04T15:50:42Z

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

+        throw new UserNotFoundException("Фильм с id = " + film.getId() + " не найден");
+    }
+
+    private void filmValidator(@RequestBody Film film) {


Этот метод нужно удалить, так как валидация теперь работает с помощью аннотаций

Удалил filmValidator, дублирующий валидацию.

Viktor3642 added 2 commits January 1, 2025 14:25

Изменил обработку исключений, чтобы пройти проверку POSTMAN test

5ef5d20

afomkina suggested changes Jan 2, 2025

View reviewed changes

Viktor3642 added 4 commits January 2, 2025 23:52

Убрал неиспользуемые импорты в FilmorateApplication для прохождения к…

5358a7b

…онтроля стиля.

Убрал неиспользуемые импорты в UserController для прохождения контрол…

2a42e69

…я стиля.

Обновил FilmControllerTests и FilmController для прохождения POSTMAN …

e8ff563

…tests.

afomkina suggested changes Jan 3, 2025

View reviewed changes

Viktor3642 added 2 commits January 3, 2025 23:36

1) В классе User изменил аннотацию @PostConstruct, на метод setLogin.

906795c

afomkina suggested changes Jan 4, 2025

View reviewed changes

Удалил filmValidator, дублирующий валидацию.

d014dd8

afomkina approved these changes Jan 5, 2025

View reviewed changes

ViktorSalk merged commit 9a4a51d into main Jan 5, 2025
2 checks passed

Создал новую ветку controllers-films-users. #1

Создал новую ветку controllers-films-users. #1

Uh oh!

Conversation

ViktorSalk commented Jan 1, 2025

Uh oh!

afomkina left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ViktorSalk commented Jan 2, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!