1) Создал новую ветку add-friends-likes. #2

ViktorSalk · 2025-01-20T15:26:46Z

Создал интерфейсы FilmStorage и UserStorage, в которых будут определены методы добавления, удаления и модификации объектов. 3) Создал классы InMemoryFilmStorage и InMemoryUserStorage, имплементирующие новые интерфейсы, и перенес всю логику хранения, обновления и поиска объектов. 4) Создал UserService и FilmService для обеспечения возможности пользователям добавлять друг друга в друзья и ставить фильмам лайки. 5) Создал класс ValidationService для фокусировки на проверке конкретных правил валидации данных в изоляции: Проверки длины описания фильма, Валидации имени фильма, Проверки email пользователя, Валидации логина и т.д. 6) Создал тесты на новую фичу: тесты фильма, тесты пользователя, тесты лайков, тесты друзей.

2) Создал интерфейсы FilmStorage и UserStorage, в которых будут определены методы добавления, удаления и модификации объектов. 3) Создал классы InMemoryFilmStorage и InMemoryUserStorage, имплементирующие новые интерфейсы, и перенес всю логику хранения, обновления и поиска объектов. 4) Создал UserService и FilmService для обеспечения возможности пользователям добавлять друг друга в друзья и ставить фильмам лайки. 5) Создал класс ValidationService для фокусировки на проверке конкретных правил валидации данных в изоляции: Проверки длины описания фильма, Валидации имени фильма, Проверки email пользователя, Валидации логина и т.д. 6) Создал тесты на новую фичу: тесты фильма, тесты пользователя, тесты лайков, тесты друзей.

2) Добавил проверку на нулевое значение лайков 3) Упростил добавление и удаление в filmStorage и InMemoryUserStorage

2) Обновил проверку на нулевое значение лайков 3) Обновил добавление и удаление в FilmService и InMemoryUserStorage

2) Обновил проверку на нулевое значение лайков 3) Обновил добавление и удаление в FilmService и UserController

2) Обновил проверку на нулевое значение лайков 3) Обновил добавление и удаление в User, UserService и ValidationServiceTests

2) Добавил ErrorHandler 3) Обновил UserController и FilmController

2) Изменил UserController для возвращения кода 404 при попытке обновить несуществующего пользователя

afomkina · 2025-01-21T14:28:40Z

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

        }
-        if (films.containsKey(film.getId())) {
-            Film oldFilm = films.get(film.getId());
+        if (!filmService.getAllMapFilms().containsKey(film.getId())) {


Код со строк 47-50 является бизнес логикой приложения, его нужно вынести на уровень сервиса

Вынес на уровень сервиса бизнес логику из FilmController.

afomkina · 2025-01-21T14:29:37Z

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

    @PutMapping
-    @ResponseStatus(HttpStatus.OK)
    public Film updateFilm(@Valid @RequestBody Film film) {
        if (film.getId() == null) {


Лучше добавить аннотацию валидации @NotNull над полем id в классе Film

Убрал лишний код (при добавлении аннотации валидации @NotNull в класс Film срабатывают ошибки в прохождении POSTMAN теста.

afomkina · 2025-01-21T14:36:31Z

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

-        if (newUser.getId() == null) {
-            log.error("Не введен id пользователя при изменении");
+    public User updateUser(@Valid @RequestBody User user) {
+        if (user.getId() == null) {


Нужно добавить аннотацию @NotNull над полем id класса User

Убрал лишний код (при добавлении аннотации валидации @NotNull в класс User срабатывают ошибки в прохождении POSTMAN теста.

afomkina · 2025-01-21T14:36:49Z

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

+        if (user.getId() == null) {
            throw new ValidationException("id пользователя должен быть указан");
        }
+        if (!userService.getAllUserMap().containsKey(user.getId())) {


Код со строк 40-43 нужно перенести в сервис

Вынес на уровень сервиса бизнес логику из UserController.

afomkina · 2025-01-21T14:38:24Z

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

-        throw new UserNotFoundException("Пользователь с id =" + newUser.getId() + " не найден");
+    @ExceptionHandler(UserNotFoundException.class)
+    @ResponseStatus(HttpStatus.NOT_FOUND)
+    public Map<String, String> handleUserNotFound(final UserNotFoundException e) {


Есть класс GlobalExceptionHandler, где отлавливаются все ошибки приложения. Данный код нужно удалить

Расширил класс GlobalExceptionHandler, в котором отлавливаются ошибки приложения, убрал дубль из UserController.

afomkina · 2025-01-21T15:05:54Z

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

+    }
+
+    @Override
+    public Map<Long, Set<User>> getFriendsMap() {


Этот метод нужно удалить

Удалил лишние методы из InMemoryUserStorage.

afomkina · 2025-01-21T15:06:05Z

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

+    }
+
+    @Override
+    public void setCollectionAllUsers(Map<Long, User> allUsers1) {


Этот метод нужно удалить

Удалил лишние методы из InMemoryUserStorage.

afomkina · 2025-01-21T15:06:41Z

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

+    }
+
+    @Override
+    public Set<User> getUserFriends(long id) {


Этот метод нужно удалить

Удалил лишние методы из InMemoryUserStorage.

afomkina · 2025-01-21T15:06:47Z

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

+    }
+
+    @Override
+    public void updateUsersFriends(long id, Set<User> userSetFriends) {


Этот метод нужно удалить

Удалил лишние методы из InMemoryUserStorage.

afomkina · 2025-01-21T15:08:11Z

src/test/java/ru/yandex/practicum/filmorate/ValidationServiceTests.java

+import static org.junit.jupiter.api.Assertions.*;
+
+@SpringBootTest
+public class ValidationServiceTests {


Данные тесты нужно удалить, так как они тестируют не работающую валидацию. Взамен нужно написать тесты, которые будет тестировать реальную валидацию. Как это сделать написала в комментарии к ValidationService

Удалил не используемый класс ValidationService и заменил тесты на ValidationTests с проверкой валидации и ServiceTests с проверкой на работу с друзьями, с лайками, популярными фильмами.

2) Убрал лишний код (при добавлении аннотации валидации @NotNull в класс Film срабатывают ошибки в прохождении POSTMAN теста. 3) Убрал лишний код (при добавлении аннотации валидации @NotNull в класс User срабатывают ошибки в прохождении POSTMAN теста. 4) Вынес на уровень сервиса бизнес логику из UserController. 5) Расширил класс GlobalExceptionHandler, в котором отлавливаются ошибки приложения, убрал дубль из UserController. 6) Удалил лишние методы из InMemoryUserStorage. 7) Удалил не используемый класс LikeComparator. 8) Удалил лишние getFriends и setFriends из User. 9) Убрал wildcard из импортов проекта. 10) Удалил не используемый LikeComparator из FilmService. 11) Выделил в отдельный метод и переиспользовал повторяющиеся строки кода в FilmService. 12) Использовал film и filmToCompare вместо однобуквенных переменных в FilmService. 13) Переписал циклы в стиле Stream API в UserService. 14) Удалил не используемый класс ValidationService. 15) Удалил не используемый класс ValidationServicetest. 16) Создал класс Validationtest для тестирования спринговой валидации. 17) Использовал List вместо Collection в FilmStorage. 18) Сделал класс InMemoryFilmStorage final. 19) Удалил лишний метод, добавив аннотацию генерирующую конструктор для final полей класса в InMemoryFilmStorage. 20) Удалил лишние методы из InMemoryFilmStorage. 21) Сделал класс InMemoryUserStorage final. 22) Удалил лишнюю мапу из InMemoryUserStorage. 23) Удалил лишний метод, добавив аннотацию генерирующую конструктор для final полей класса в InMemoryUserStorage. 24) Удалил лишние методы из InMemoryUserStorage. 25) Использовал List вместо Collection в UserStorage.

afomkina · 2025-01-23T11:34:42Z

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

    @GetMapping
    public List<Film> findAll() {
-        return new ArrayList<>(films.values());
+        return new ArrayList<>(filmService.getAllFilms());


Чтобы не кастовать Collection в List, на уровне сервиса нужно возращать интерфейс List вместо Collection

Заменил Collection на List в FilmService.

afomkina · 2025-01-23T11:35:37Z

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

-        log.error("Фильм с id = {} не найден", film.getId());
-        throw new UserNotFoundException("Фильм с id = " + film.getId() + " не найден");
+    @GetMapping("/popular")
+    public Collection<Film> getPopularFilms(@RequestParam(required = false, defaultValue = "10") Integer count) {


Давай здесь тоже будем возвращать пользователю List

Заменил Collection на List в FilmService и FilmController.

afomkina · 2025-01-23T11:36:31Z

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

    @GetMapping
    public List<User> findAll() {
-        return new ArrayList<>(users.values());
+        return new ArrayList<>(userService.getAllUsers());


Нужно на уровне сервиса возвращать List вместо Collection

Заменил Collection на List в UserService.

afomkina · 2025-01-23T11:37:09Z

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

-        throw new UserNotFoundException("Пользователь с id =" + newUser.getId() + " не найден");
+    @GetMapping("/{id}/friends/common/{otherId}")
+    public List<User> getCommonFriends(@PathVariable Long id, @PathVariable Long otherId) {
+        return new ArrayList<>(userService.generalFriends(id, otherId));


Замени Collection на List

Заменил Collection на List в UserService

afomkina · 2025-01-23T11:38:48Z

src/main/java/ru/yandex/practicum/filmorate/exceptions/GlobalExceptionHandler.java

 @RestControllerAdvice
 public class GlobalExceptionHandler {

+    public static class FilmNotFoundException extends RuntimeException {


Эта ошибка нигде не выбрасывается, чтобы ее отлавливать в ExceptionHandler. Данный код и код со строк 29-34 - бесполезный

Удалил неиспользуемый код в GlobalExceptionHandler.

afomkina · 2025-01-23T11:43:52Z

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

+    }
+
+    private long getNextId() {
+        long currentMaxId = allUsers.keySet().stream().mapToLong(id -> id).max().orElse(0);


Стримы принято бить по строкам:

Suggested change

long currentMaxId = allUsers.keySet().stream().mapToLong(id -> id).max().orElse(0);

long currentMaxId = allUsers.keySet().stream()

.mapToLong(id -> id)

.max()

.orElse(0);

Сделал стрим более читаемым.

afomkina · 2025-01-23T11:45:17Z

src/test/java/ru/yandex/practicum/filmorate/service/ServiceTests.java

+    class FilmLikesTests {
+        @Test // Тесты для работы с лайками
+        void testLikeOperations() {
+            Film film = new Film();


Вместо того, чтобы в каждом тесте создавать одинаковые объекты, их можно вынести в блок @BeforeEach и избавиться от дублирований кода

Удалил дубли, вынеся их блоком @beforeeach в ServiceTests.

2) Заменил Collection на List в UserService. 3) Заменил возвращение пользователю getPopularFilms интерфейс List в FilmService. 4) Удалил неиспользуемый код в GlobalExceptionHandler. 5) Сделал стрим более читаемым. 6) Удалил дубли, вынеся их блоком @beforeeach в ServiceTests.java

Viktor3642 added 9 commits January 20, 2025 19:25

1) Внёс изменения для прохождения Postman теста.

e8562bb

2) Добавил проверку на нулевое значение лайков 3) Упростил добавление и удаление в filmStorage и InMemoryUserStorage

1) Внёс изменения для прохождения Postman теста.

fd99aee

2) Обновил проверку на нулевое значение лайков 3) Обновил добавление и удаление в FilmService и InMemoryUserStorage

1) Внёс изменения для прохождения Postman теста.

8990307

2) Обновил проверку на нулевое значение лайков 3) Обновил добавление и удаление в FilmService и UserController

1) Внёс изменения для прохождения Postman теста.

7e5b9b7

2) Обновил проверку на нулевое значение лайков 3) Обновил добавление и удаление в User, UserService и ValidationServiceTests

1) Внёс изменения для прохождения Postman теста.

1b9ea9e

2) Добавил ErrorHandler 3) Обновил UserController и FilmController

1) Внёс изменения для прохождения Postman теста.

2ba0149

2) Добавил ErrorHandler 3) Обновил UserController и FilmController

1) Внёс изменения для прохождения Postman теста.

2be3b60

2) Добавил ErrorHandler 3) Обновил UserController и FilmController

1) Внёс изменения для прохождения Postman теста.

ec75ffe

2) Изменил UserController для возвращения кода 404 при попытке обновить несуществующего пользователя

afomkina suggested changes Jan 21, 2025

View reviewed changes

afomkina suggested changes Jan 23, 2025

View reviewed changes

afomkina approved these changes Jan 24, 2025

View reviewed changes

ViktorSalk merged commit 052b4a1 into main Jan 24, 2025
2 checks passed

1) Создал новую ветку add-friends-likes. #2

1) Создал новую ветку add-friends-likes. #2

Uh oh!

Conversation

ViktorSalk commented Jan 20, 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!

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!

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!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone