Add friends likes#2
Conversation
…nds-likes # Conflicts: # src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java
…nds-likes # Conflicts: # src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Константин!
Практически всё сделано, осталось чуть поправить.
| private final Map<Long, Film> films = new HashMap<>(); | ||
| private final Logger log = LoggerFactory.getLogger(FilmController.class); | ||
| private final Logger logger = LoggerFactory.getLogger(FilmController.class); | ||
| FilmService filmService; |
There was a problem hiding this comment.
Для соблюдения инкапсуляции переменную лучше сделать приватной.
|
|
||
| @GetMapping("/{id}") | ||
| @ResponseStatus(HttpStatus.OK) | ||
| public Film getFilmId(@PathVariable long id) { |
There was a problem hiding this comment.
PathVariable long id
Здесь и дальше.
К таким параметрам можно было бы добавить аннотацию Positive.
Конечно, на Ваше усмотрение.
Обратите внимание, что необходимо будет добавить @validated в контроллер на уровне класса, чтобы проверять параметры метода. В этом случае аннотация @validated устанавливается на уровне класса, даже если она присутствует на методах.
| import java.util.HashSet; | ||
| import java.util.Set; | ||
|
|
||
| @Component |
There was a problem hiding this comment.
Вместо Component можно использовать Repository
Аннотация @repository показывает, что класс применяется для работы с поиском, а также для получения и хранения данных. На практике @repository может применяться при реализации шаблона DAO.
Необязательно, на Ваше усмотрение.
Различие между аннотациями component, @repository и @service в Spring
https://sky.pro/media/razlichie-mezhdu-annotacziyami-component-repository-i-service-v-spring/
| public class InMemoryFilmStorage extends Storage<Film> implements FilmStorage<Film> { | ||
|
|
||
|
|
||
| public Film create(Film film) { |
There was a problem hiding this comment.
Над методами лучше расставлять аннотацию
Override
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Константин!
Замечаний нет.
Работа принята.
2 participants
No description provided.