Add database by dendzim · Pull Request #3 · dendzim/java-filmorate

dendzim · 2025-12-19T19:30:39Z

Пришлось пересоздать ветку и залить по новой схему БД с промежуточной peer-to-peer проверки. В самом проекте в БД нет поля статус в таблице User_friends и переименованы некоторые поля из-за синтаксиса тестов постмана, остальное осталось неизменным. По ФЗ12 реализован функционал ФЗ12 Добавлена H2 БД которая записывается в файл в проекте и в память для интеграционных тестов соответственно. Добавлены интеграционные тесты. В проекте осталось два рудиментарных класса реализовывающих хранение данных в памяти. В pom файл добавлен check style плагин.

VadimZharkov · 2025-12-20T05:52:53Z

Пришлось пересоздать ветку и залить по новой схему БД с промежуточной peer-to-peer проверки. В самом проекте в БД нет поля статус в таблице User_friends и переименованы некоторые поля из-за синтаксиса тестов постмана, остальное осталось неизменным. По ФЗ12 реализован функционал ФЗ12 Добавлена H2 БД которая записывается в файл в проекте и в память для интеграционных тестов соответственно. Добавлены интеграционные тесты. В проекте осталось два рудиментарных класса реализовывающих хранение данных в памяти. В pom файл добавлен check style плагин.

Было бы лучше привести в соответствие текущей схемы БД

VadimZharkov · 2025-12-20T05:55:42Z

src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java

-    private final FilmService filmService;
+    private final FilmService service;

    @Autowired


@Autowired перед конструктором можно не писать.
Да и весь конструктор с рутинным кодом можно не писать, если воспользоваться библиотекой lombok. Советую ее изучить и применять.
https://habr.com/ru/companies/piter/articles/676394/

VadimZharkov · 2025-12-20T06:10:29Z

src/main/java/ru/yandex/practicum/filmorate/storage/InMemoryFilmStorage.java

InMemory реализации из проекта нужно убрать, больше не понадобятся.

VadimZharkov · 2025-12-20T06:10:50Z

src/main/java/ru/yandex/practicum/filmorate/storage/InMemoryUserStorage.java

InMemory реализации из проекта нужно убрать, больше не понадобятся.

VadimZharkov · 2025-12-20T06:12:44Z

src/main/java/ru/yandex/practicum/filmorate/exception/ErrorHandler.java

    @ExceptionHandler
    @ResponseStatus(HttpStatus.BAD_REQUEST)
    public ErrorResponse handleValidationException(final ValidationException e) {
        return new ErrorResponse(e.getMessage());


Лучше добавить логирование ошибок log.error(...), это позволит легче с ними разбираться.

VadimZharkov · 2025-12-20T06:18:28Z

src/main/java/ru/yandex/practicum/filmorate/service/FilmService.java

    }

-    public static final Comparator<Film> comparator = Comparator.comparingInt(Film::getRating).reversed();
+    private void validateFilm(Film film) {


Для валидации можно также использовать аннотации валидации на полях классов (@NotNull, @Size и т.д.) и параметрах методов (@Valid, @Validated). Советую применить, т.к. в дальнейшем этот подход будет использоваться часто.

VadimZharkov · 2025-12-20T06:20:25Z

src/main/java/ru/yandex/practicum/filmorate/dao/BaseDao.java

+import java.util.List;
+
+@Repository
+public abstract class BaseDao<T> {


Отлично, что сделал базовый абстрактный класс, @Repository лишний на нем, т.к. экземпляр не будет никогда создаваться.

VadimZharkov · 2025-12-20T06:34:23Z

src/main/java/ru/yandex/practicum/filmorate/model/Film.java

-    }
+    private Rating mpa;
+    @JsonProperty(access = JsonProperty.Access.READ_ONLY)
+    private Integer likes = 0;


Можно не хранить в поле и соответственно обновлять и удалять кол-во лайков, а рассчитывать с помощью SQL в getPopular

VadimZharkov · 2025-12-20T06:37:12Z

src/main/java/ru/yandex/practicum/filmorate/dao/FilmDbStorage.java

+
+    @Override
+    public Film create(Film film) {
+        if (ratingStorage.findRatingById(film.getMpa().getId()).isEmpty()) {


Проверки на существование рейтинга и жанров можно не делать, т.к. в схеме БД сделаны внешние ключи и выбросится исключение, если вставляемых рейтингов и жанров не будет в БД

VadimZharkov · 2025-12-20T06:40:08Z

src/main/java/ru/yandex/practicum/filmorate/dao/GenreDbStorage.java

+                .map(genre -> new Object[]{filmId, genre.getId()})
+                .toList();
+
+        jdbc.batchUpdate(ADD_GENRE_TO_FILM_QUERY, batchArgs);


Отлично, что используешь batchUpdate

VadimZharkov

Хорошая работа в целом, можно улучшить.

dendzim · 2025-12-23T14:14:31Z

Внес правки согласно замечаниям:
-Вложил диаграмму БД с актуальными данными проекта
-Убрал InMemory реализации и их тесты
-Убрал поле с лайками из модели Film
-Переделал валидацию на валидацию аннотациями
-Убрал лишние проверки и методы старой валидации
-Добавил кастомную валидацию Даты выпуска фильма
-Спрятал конструкторы контроллеров под аннтоации

VadimZharkov · 2025-12-24T06:11:47Z

src/main/java/ru/yandex/practicum/filmorate/model/ErrorResponse.java

+
+@Getter
 public class ErrorResponse {
    String error;


private final

VadimZharkov · 2025-12-24T06:18:33Z

src/main/resources/schema.sql

+);
+
+CREATE TABLE IF NOT EXISTS films_genres (
+	film_id INTEGER,


VadimZharkov · 2025-12-24T06:18:40Z

src/main/resources/schema.sql

+
+CREATE TABLE IF NOT EXISTS films_genres (
+	film_id INTEGER,
+	genre_id INTEGER,


VadimZharkov

Доработки увидел, все хорошо. Нужно еще пару моментов поправить.

dendzim added 10 commits November 26, 2025 22:12

Добавлено создание встроенной БД и размечена архитектура проекта

dc001ed

Исправлены контроллеры и сервисы а также добавлены новые

d8b1f83

Исправлены контроллеры и сервисы а также добавлены новые

06ea5e1

Исправлены контроллеры и сервисы а также добавлены новые

6a26a3d

Реализован функционал согласно ФЗ

979fdc6

Реализован функционал согласно ФЗ

4dac193

Реализован функционал согласно ФЗ и тестам в постман

1ab6fde

Исправлена работа лайков и добавлены интеграционные тесты

0add97e

Исправлена работа лайков и добавлены интеграционные тесты

9b3774a

Исправлен check style

5fce3a0

VadimZharkov reviewed Dec 20, 2025

View reviewed changes

VadimZharkov suggested changes Dec 20, 2025

View reviewed changes

Внесены правки согласно замечаниям

b5c1c23

VadimZharkov reviewed Dec 24, 2025

View reviewed changes

src/main/resources/schema.sql Outdated

);

CREATE TABLE IF NOT EXISTS films_genres (

film_id INTEGER,

Copy link

VadimZharkov Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOT NULL

VadimZharkov reviewed Dec 24, 2025

View reviewed changes

src/main/resources/schema.sql Outdated

CREATE TABLE IF NOT EXISTS films_genres (

film_id INTEGER,

genre_id INTEGER,

Copy link

VadimZharkov Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOT NULL

VadimZharkov suggested changes Dec 24, 2025

View reviewed changes

Внесены правки согласно замечаниям

4a4fe4b

VadimZharkov approved these changes Dec 25, 2025

View reviewed changes

dendzim merged commit 87113ce into main Dec 25, 2025
2 checks passed

dendzim deleted the add-database branch December 25, 2025 17:27

Conversation

dendzim commented Dec 19, 2025

Uh oh!

VadimZharkov commented Dec 20, 2025

Uh oh!

VadimZharkov Dec 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

VadimZharkov left a comment

Choose a reason for hiding this comment

Uh oh!

dendzim commented Dec 23, 2025

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!

VadimZharkov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

VadimZharkov Dec 20, 2025 •

edited

Loading