Develop#13
Conversation
fix: Добавил DROP TABLE всех таблиц перед запуском приложения
…ию режиссеров: У сущности Film появилось новое поле - список режиссеров, так же появилась новая сущность Director. Добавлены новы слои контроллеров, сервисов, хранилищ и Dao для корректной обработке запросов, касаемых Director (режиссеров). Добавлены новые эндпоинты в /films. А так же отредактированы методы хранилища и сервисов фильмов, по корректному добавлению фильмов с режиссерами. Добавились две новые таблицы FILM_DIRECTOR_T и DIRECTORS_T хранящие в себе информацию по режиссерам их и взаимосвязь с табл FILM_T. Добавилось новое исключение для обработки некорректно переданных данных, касаемых режиссеров. Новый функционал был прописан в файле README и добавлена новая схема БД с выделением новых таблиц для наглядности
…вы на фильмы. У отзывов есть рейтинг, оценка и тип. Оценка — полезно/бесполезно. Тип отзыва — негативный/положительный. Рейтинг отзыва при создании равен нулю. Если пользователь оценивает отзыв как полезный, это увеличивает его рейтинг на 1. Если как бесполезный, то уменьшает на 1. Отзывы сортируются по рейтингу полезности.
Add remove endpoint
Добавление нового функционала, касаемого режиссеров (Director)
добавлена функциональность "Отзывы".
feat: создать функциональность по поиску общих фильмов
…andler класс ErrorHandler style: удалить комментарии в FilmController, UserController Test-fix: Get director 1 before create в Postman sprint13/add-director
fix-test add-director
…лайков с фильтрацией по жанру и за указанный год
feat: создать поиск фильмов по названию и/или имени режиссера
feature: реализовал возможность выводить топ-N фильмов по количеству …
Add recommendations
2 теста по ленте остались
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый вечер!
У вас очень хорошая работа. Вы быстро сумели сделать грамотную реализацию задач, с разбивкой по слоям и разделением функционала. А, с учётом того, что команда у вас несыгранная, результат можно считать просто прекрасным. Похоже, что ваша команда составлена из сильных и опытных разработчиков. Особенно хочу отметить два момента: Вы очень грамотно разложили приложение по слоям контроллер - сервис - репозиторий и у вас хороший писатель запросов.
У меня есть несколько скорее вопросов и предложений, чем замечаний. Посмотрите пожалуйста. Я не стал писать одинаковые комментарии в разных местах, если решите править, просмотрите весь код пожалуйста.
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день!
Замечаний нет. Работа принята.
Поздравляю команду с завершением группового этапа. Так держать!
P.S.
По поводу запроса. Наставник что-то молчит, поэтому я на свой страх и риск этот запрос принимаю. Он мне понравился. Но, имейте в виду, что в реальной обстановке, если будет большое количество записей( А индексов тут нет, придётся перебором искать), сервер можно подвесить. А вот в учебной задаче, с двумя тремя записями, самое то. :) Поэтому пользуйтесь этим приёмом крайне осторожно.
5 participants
No description provided.