Переработана архитектура и добалена новая логика#4
Conversation
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Максим!
Очень хорошая работа, от меня всего пара небольших уточнений.
| import java.util.Map; | ||
|
|
||
| @RestControllerAdvice | ||
| public class GlobalExceptionHandler { |
There was a problem hiding this comment.
В класс хорошо бы добавить логирование. Это полезно при разборе логов. Помогает быстрее понять когда и в каком случае возникла та или иная ошибка.
| } | ||
|
|
||
| @GetMapping("/{id}") | ||
| public Film getFilm(@PathVariable Long id) { |
There was a problem hiding this comment.
Здесь и в других аналогичных методах контроллеров.
К таким параметрам можно было бы добавить аннотацию Positive.
Конечно, на Ваше усмотрение.
Обратите внимание, что необходимо будет добавить @validated в контроллер на уровне класса, чтобы проверять параметры метода. В этом случае аннотация @validated устанавливается на уровне класса, даже если она присутствует на методах.
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Максим!
Замечаний нет.
Работа принята.
2 participants
Реализовал добавление друзей и лайки фильмов