-
Notifications
You must be signed in to change notification settings - Fork 1
Реализация функционала 13 спринта (групповой проект) #27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Feature: Add remove endpoint - "Удаление фильмов и пользователей"
2. add methods: checkUserId, checkFilmId, checkReviewId for class ReviewServiceImpl
2. unification methods: deleteLike, deleteDislike in deleteReaction for class ReviewServiceImpl, class deleteReaction, interface ReviewRepository, interface ReviewService
Add director - Добавление режиссёров в фильмы.
# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java # src/main/java/ru/yandex/practicum/filmorate/dal/FilmRepository.java # src/main/java/ru/yandex/practicum/filmorate/dal/JdbcFilmRepository.java # src/main/java/ru/yandex/practicum/filmorate/service/FilmService.java # src/main/java/ru/yandex/practicum/filmorate/service/FilmServiceImpl.java
Ветка для реализации задачи add-reviews.
Добавил функциональность "Рекомендации"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Добрый вечер, Надежда!
Вы отлично поработали над проектом. Функционал задач реализован в соответствии с техническим заданием, грамотно разобрались с взаимодействием слоев Controller - service - repository. Несмотря на то, что это первый групповой проект, результат можно считать очень хорошим.
От меня только несколько мелких уточнений, часть из них на ваше усмотрение.
src/main/java/ru/yandex/practicum/filmorate/controller/DirectorController.java
Show resolved
Hide resolved
src/main/java/ru/yandex/practicum/filmorate/controller/DirectorController.java
Show resolved
Hide resolved
src/main/java/ru/yandex/practicum/filmorate/dal/JdbcReviewRepository.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/yandex/practicum/filmorate/controller/DirectorController.java
Show resolved
Hide resolved
src/main/java/ru/yandex/practicum/filmorate/dal/JdbcDirectorRepository.java
Show resolved
Hide resolved
Develop fix
Update application.properties
avfyodorov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Добрый день, Надежда!
Замечаний нет.
Групповой проект принят.
Поздравляю с удачным опытом групповой разработки, успехов!
Уверен, что данный опыт пригодится вам в дальнейшем.
@Just-Roman:
Оценка — полезно/бесполезно, Тип отзыва — негативный/положительный.
POST /reviews -Добавление нового отзыва.
PUT /reviews -Редактирование уже имеющегося отзыва.
DELETE /reviews/{id} -Удаление уже имеющегося отзыва.
GET /reviews/{id} - Получение отзыва по идентификатору.
GET /reviews?filmId={filmId}&count={count} -Получение всех отзывов по идентификатору фильма, если фильм не
указан то все. Если кол-во не указано то 10.
PUT /reviews/{id}/like/{userId} — пользователь ставит лайк отзыву.
PUT /reviews/{id}/dislike/{userId} — пользователь ставит дизлайк отзыву.
DELETE /reviews/{id}/like/{userId} — пользователь удаляет лайк/дизлайк отзыву.
DELETE /reviews/{id}/dislike/{userId} — пользователь удаляет дизлайк отзыву.
лайки на фильмы и отзывы (написание, обновление удаление), которые оставлял пользователь.
enum EventType, enum OperationType.
таблицы activity.
@Max-Browckin:
Найти пользователей с максимальным количеством пересечения по лайкам.
Определить фильмы, которые один пролайкал, а другой нет.
Рекомендовать фильмы, которым поставил лайк пользователь с похожими вкусами, а тот, для кого составляется рекомендация, ещё не поставил.
Более подробно подобный алгоритм описан в этой [статье](https://www.baeldung.com/java-collaborative-filtering-recommendations).
@slavakorg:
Вывод всех фильмов режиссёра, отсортированных по количеству лайков.
Вывод всех фильмов режиссёра, отсортированных по годам.
query — текст для поиска
by — может принимать значения director (поиск по режиссёру), title (поиск по названию), либо оба значения через запятую при поиске одновременно и по режиссеру и по названию.
GET /films/search?query=крад&by=director,title
@Kotezh:
По жанру.
За указанный год.
userId — идентификатор пользователя, запрашивающего информацию;
friendId — идентификатор пользователя, с которым необходимо сравнить список фильмов