Conversation
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Серьёзное замечание только одно: одна таблица базы данных - один класс для выполнения базовых операций с этой таблицей.
Ну и тесты, конечно.
|
|
||
| @Override | ||
| public void addLike(Integer userId, Integer filmId) { | ||
| final String sqlQuery = "INSERT INTO LIKE_FILMS (USER_ID, FILM_ID) values(?, ?)"; |
There was a problem hiding this comment.
Это не очень удачное решение-прямой доступ к чужой таблице. По идее, данный класс должен работать только и исключительно со своей таблицей. Лучше было бы выделить методы изменения таблицы "Фильмы-Лайки" в отдельный класс.
модификация данных в таблице должны происходить исключительно и только в соответствующем классе.
Принцип такой: одна таблица- один класс для добавления, удаления, изменения данных в этой таблице.
| } | ||
|
|
||
|
|
||
| /*public List<Integer> findAllId() { |
There was a problem hiding this comment.
Если код больше не нужен, то его лучше было бы удалить.
| } | ||
|
|
||
| public void addFriend(Integer userId, Integer friendId) { | ||
| String sqlQuery = "INSERT INTO USER_FRIEND (user_id, friend_id)" + |
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день!
Собственно, по коду замечаний нет.
Но с тестированием нужно обязательно разобраться.
Вы Базы чистите?
# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/model/mapper/LikeFilmsRowMapper.java # src/main/java/ru/yandex/practicum/filmorate/service/FilmService.java # src/main/java/ru/yandex/practicum/filmorate/sortage/LikeFilmDbStorage.java # src/test/java/ru/yandex/practicum/filmorate/controllerTest/FilmControllerTest.java
|
Прошу прощения, что долго, уезжал на неделю без доступа к компьютеру, приехал только вчера |
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день!
Замечаний нет.
Работа принята.
2 participants
Не дописал интеграционное тестирование, кажется, что еще с основной частью не все хорошо