Добавлены 8 фич. by Deathriot · Pull Request #11 · Deathriot/java-filmorate-group-project

Deathriot · 2024-04-10T19:18:35Z

Все тесты пройдены, заливаем все в мейн!

Feat: Удаление фильмов и пользователей

Add most populars

1. controller - DirectorController 2. model - Director 3. service - DirectorService и DirectorServiceImpl 4. storage - DirectorStorage и DirectorDbStorage 5. validator - DateValidator и IsAfterDate Добавлен основной функционал: 1. Создание режиссёра 2. Получение режиссёра по id 3. Вывод списка всех режиссёров 4. Изменение режиссёра 5. Удаление режиссёра Дополнителный функционал: 1. Добавлена аннотация @IsAfterDate, которая решает часть проблем с тестами, связанными с датой. 2. Добавлены таблицы DIRECTOR и FILM_DIRECTOR в schema.sql, а также немного изменен порядок создания таблиц.

1. Создание режиссёра 2. Получение режиссёра по id 3. Вывод списка всех режиссёров 4. Изменение режиссёра 5. Удаление режиссёра 6. Возвращает список фильмов режиссера отсортированных по количеству лайков или году выпуска. Дополнителный функционал: 1. Добавлена аннотация @IsAfterDate, которая решает часть проблем с тестами, связанными с датой. 2. Добавлены таблицы DIRECTOR и FILM_DIRECTOR в schema.sql, а также немного изменен порядок создания таблиц.

# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/controller/ReviewController.java # src/main/java/ru/yandex/practicum/filmorate/model/Review.java # src/main/java/ru/yandex/practicum/filmorate/service/review/ReviewService.java # src/main/java/ru/yandex/practicum/filmorate/storage/review/ReviewStorage.java # src/main/java/ru/yandex/practicum/filmorate/storage/review/dao/ReviewDbStorage.java # src/test/java/ru/yandex/practicum/filmorate/controller/ReviewDbStorageTest.java

1. Создание режиссёра 2. Получение режиссёра по id 3. Вывод списка всех режиссёров 4. Изменение режиссёра 5. Удаление режиссёра 6. Возвращает список фильмов режиссера отсортированных по количеству лайков или году выпуска. Дополнителный функционал: 1. Добавлена аннотация @IsAfterDate, которая решает часть проблем с тестами, связанными с датой. 2. Добавлены таблицы DIRECTOR и FILM_DIRECTOR в schema.sql, а также немного изменен порядок создания таблиц.

Добавление режиссёров в фильмы. add-director

…ля getUserRecommendations

…ользователя getUserRecommendations

…ода getUserRecommendations. Добавить сам метод getUserRecommendations для получения рекомендаций фильмов для пользователя

…ия рекомендаций фильмов для пользователя

…и создания объектов путем создания копии через билдер.

…даций пользователей.

# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/model/Review.java # src/main/resources/schema.sql

…просе

Add reviews

fix: восстановить потерянный при мердже эндпоинт getUserRecommendations

Нарисовал обновленную диаграмму БД

Расширен список таблиц, а также примеры запросов к ним

Чето все строчки поплыли, пытаюсь сделать красиво

"Примеры запросов" Теперь пишутся большими жирными буквами :)

Deathriot readme updates

Add feed v2

LexLippi

Привет!

Хорошая работа!

Есть некоторое количество замечаний, которые необходимо исправить!

Желаю удачи!

LexLippi · 2024-04-11T08:12:00Z

+
+@Data
+@Builder
+public class Director {


Классам с данными лучше всегда добавлять модификатор final, чтобы ограничивать возможность наследования

LexLippi · 2024-04-11T08:12:26Z

+
+@Data
+@Builder
+public class Review {


Аналогично про модификатор final

Сделано

LexLippi · 2024-04-11T08:12:46Z

+
+@Data
+@Builder
+public class Event {


Аналогично про модификатор final

LexLippi · 2024-04-11T08:17:56Z

+    public Collection<Film> getPopular(Integer count, Integer genreId, String year) {
+        if (year == null) {
+            return filmStorage.getPopularByGenre(count, genreId);
+        } else if (genreId == null) {


else не нужен, так как везде в ветках ранее происходит return. То есть вот эта запись эквивалентна тому, что написано:

if (year == null) { return filmStorage.getPopularByGenre(count, genreId); } if (genreId == null) { return filmStorage.getPopularByYear(count, year); } return filmStorage.getPopularByGenreAndYear(count, genreId, year);

Однако цикломатическая сложность у такого подхода ниже, что повышает читаемость кода

LexLippi · 2024-04-11T08:59:21Z

+        if (sortBy.equals("year")) {
+            return filmStorage.getByDirectorSortByYear(id);
+        } else if (sortBy.equals("likes")) {
+            return filmStorage.getByDirectorSortByLikes(id);
+        } else {
+            throw new IllegalArgumentException("Wrong sort format");
+        }


Аналогично можно переписать без else

LexLippi · 2024-04-11T09:15:03Z

+                .usingGeneratedKeyColumns("REVIEW_ID");
+
+        Integer reviewId = simpleJdbcInsert.executeAndReturnKey(toMap(review)).intValue();
+        return getReviewById(reviewId);


Точно ли нам при выходе надо делать запрос на получение объекта?
Его можно сконструировать из только что вставленного + того идентификатора, который нам вернул jdbcTemplate, таким образом мы сэкономим на одном запросе к СУБД, иногда это может быть очень важно с точки зрения производительности

Сделано

LexLippi · 2024-04-11T09:19:06Z

+                review.getIsPositive(),
+                review.getReviewId());
+
+        return getReviewById(review.getReviewId());


Аналогично тому, что написал выше

К сожалению, здесь так не получится. Так или иначе придется выгружать из БД ревью с этим айди, тк не всем полям разрешено обновляться через апдейт (userId, filmId, useful). Так же, нельзя запрещать менять эти поля, так как тесты постмана ждут ответ 200.

LexLippi · 2024-04-11T09:20:43Z

+
+    @Override
+    public Collection<Review> getAllReviews(Integer count) {
+        String sql = "SELECT * FROM REVIEWS ORDER BY USEFUL DESC LIMIT ?;";


Лучше fetch first ? rows only, так как оператор limit не определен в sql-стандарте и при переходе на другую СУБД запрос может поломаться

сделано

LexLippi · 2024-04-11T09:20:57Z

+    @Override
+    public Collection<Review> getAllReviewsOfFilm(Integer filmId, Integer count) {
+        filmStorage.checkFilmExist(filmId);
+        String sql = "SELECT * FROM REVIEWS WHERE FILM_ID=? ORDER BY USEFUL DESC LIMIT ?;";


Про limit уже писал

сделано

LexLippi · 2024-04-11T09:23:38Z

+        String sql = "SELECT IS_POSITIVE FROM USER_REVIEW_RATE WHERE REVIEW_ID=? AND USER_ID=?";
+        SqlRowSet userRate = jdbcTemplate.queryForRowSet(sql, reviewId, userId);
+
+        if (!userRate.next())


Советую всегда использовать фигурные скобки в соответствии с
Google Java Code Style

сделано

Исправление недочетов ревьюера

LexLippi · 2024-04-12T07:29:26Z

Привет!

Хорошая работа!

Поздравляю всех с окончанием группового проекта и желаю удачи!

MayFoxes and others added 30 commits April 4, 2024 02:47

Feat: Удаление фильмов и пользователей

10ac2ef

Отзывы. Пока что без тестов, пусть лежат

e3268af

Отзывы. Пока что без тестов, пусть лежат

7e22e07

fix: check username

d324fc6

fix: schema fix

d2bf2ed

Merge pull request #1 from Deathriot/add-remove-endpoint

7d48263

Feat: Удаление фильмов и пользователей

Почти готово!

aedf5c3

feat: добавить возможность выводить топ-N фильмов по количеству лайков

d06f48b

feat: добавить тесты

eea3369

Merge pull request #7 from Deathriot/add-most-populars

af30c28

Add most populars

Feat: Добавлены отзывы. Ветка готова к PR.

7e190be

Отзывы. Пока что без тестов, пусть лежат

56ae68c

Почти готово!

c2ec47f

Feat: Добавлены отзывы. Ветка готова к PR.

998b568

Feat: add reviews. С ребейзом develop. Готова к мерджу!

0afdaf4

Merge remote-tracking branch 'origin/add-reviews' into add-reviews

7728f3d

Merge pull request #4 from Deathriot/add-director

7e9852d

Добавление режиссёров в фильмы. add-director

feat: добавить эндпоинт получения рекомендаций фильмов для пользовате…

0760d6f

…ля getUserRecommendations

feat: добавить в интерфейс метод получения рекомендаций фильмов для п…

03ca46f

…ользователя getUserRecommendations

feat: добавить и реализовать метод getUserRecommendations для получен…

d5017a9

…ия рекомендаций фильмов для пользователя

feat: добавить в аннотацию @builder параметр toBuilder для возможност…

c25cf1f

…и создания объектов путем создания копии через билдер.

feat: добавить метода теста userRecommendations для проверки рекоммен…

2ffed3a

…даций пользователей.

Deathriot and others added 23 commits April 9, 2024 21:07

rebase add-search

776a0a1

Feat: Добавлены отзывы. Ветка готова к PR.

84d751e

Feat: add reviews. С ребейзом develop. Готова к мерджу!

efe7c3c

Feat: Добавлены отзывы. Ветка готова к PR.

62f99f9

Feat: Я нашел директора!

6fd9585

FIX: Исправлен кодстайл. Произведена декомпозиция

140f9b3

FIX: Добавлен PK для таблицы USER_REVIEW_RATE

e994d33

Merge remote-tracking branch 'origin/add-reviews' into add-reviews

7cb1d89

# Conflicts: # src/main/java/ru/yandex/practicum/filmorate/model/Review.java # src/main/resources/schema.sql

FIX: Теперь сортировка для получения всех отзывов происходит в sql-за…

00ae6dc

…просе

Merge pull request #8 from Deathriot/add-reviews

557ff4b

Add reviews

fix: восстановить потерянный при мердже эндпоинт getUserRecommendations

f028c22

Merge pull request #9 from Deathriot/hotfix-add-recommendations

c04b67d

fix: восстановить потерянный при мердже эндпоинт getUserRecommendations

rebase

9ae341e

Feat: лента событий

704f41a

fix: some fixed

7b9cad4

Update README.md, update ER-Diagram

6f293f4

Нарисовал обновленную диаграмму БД

Update README, discription

2262134

Расширен список таблиц, а также примеры запросов к ним

FIX README

5cebf5b

Чето все строчки поплыли, пытаюсь сделать красиво

fix README part 2

43e8a51

"Примеры запросов" Теперь пишутся большими жирными буквами :)

updated feeds

6ec960a

some minor fixes

6cba7b9

Merge pull request #10 from Deathriot/Deathriot-README-updates

1ea6b9e

Deathriot readme updates

Merge pull request #3 from Deathriot/add-feed

baeebaa

Add feed v2

LexLippi suggested changes Apr 11, 2024

View reviewed changes

Deathriot and others added 5 commits April 11, 2024 14:37

FIX: исправлена аннотация на валидацию фильмов.

2e1da67

FIX: Исправлены недочеты от ревьюера (отзывы)

311bea8

FIX: Исправлены ВСЕ недочеты от ревьюера

8528772

FIX: Удален дроп базы.

03d7f0b

Merge pull request #12 from Deathriot/add-reviews

564abbd

Исправление недочетов ревьюера

Conversation

Deathriot commented Apr 10, 2024

Uh oh!

LexLippi left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LexLippi commented Apr 12, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants