Gimaldinov - Final Spring Project by DEEXORT · Pull Request #1 · DEEXORT/ReviewPlatform

DEEXORT · 2026-01-28T05:25:01Z

Финальный проект 5 модуля.
Данные для аутентификации в приложение:
Администратор:
login=admin, password=admin
Менеджер:
login=manager, password=manager
Пользователь:
login=cyber_user, password=cyber_user

…e with interfaces.

…enticationHandler for sending cookie. Added REST controllers for Product and Review.

…s adjusted. dev(test): Added IT tests for UserRestController.

dev(test): Added and fixed IT tests for controllers

demologin · 2026-01-30T15:35:43Z

Общий вывод по проекту

Представленный код демонстрирует хорошее владение экосистемой Spring Boot (Security, Data JPA, MVC) и инструментами тестирования (MockMvc, Testcontainers). Архитектура построена по классической многослойной модели, что упрощает поддержку.

Сильные стороны:

Грамотное использование Testcontainers для интеграционного тестирования.

Разделение ответственности между контроллерами, сервисами и репозиториями.

Использование Lombok для уменьшения шаблонного кода.

Области для роста:

Безопасность и конфигурация: Стоит уделить внимание защите чувствительных данных (секреты JWT) и валидации входящих DTO.

Производительность JPA: Необходимо контролировать типы загрузки (FetchType) и избегать лишних запросов к БД внутри циклов или мапперов.

Чистота кода: Переход от System.out к логированию и строгое соблюдение модификаторов доступа (private final) сделают код более профессиональным.

Проект выглядит очень достойно для продвинутого уровня обучения. Видно стремление к использованию современных подходов Java 21. Устранение мелких недочетов по SOLID и Code Convention позволит автору претендовать на позицию Middle-разработчика.

Поставил A

demologin · 2026-01-30T15:35:44Z

src/main/java/com/javarush/reviewplatform/review/ReviewService.java

+    public ReviewTo save(ReviewTo reviewTo) {
+        ReviewTo saved = super.save(reviewTo);
+        ProductTo updatedProduct = updateProduct(saved.getProductId());
+        return saved;


При пересчете рейтинга используется метод findAll(), что может привести к проблемам с производительностью при большом количестве отзывов. Рекомендуется использовать агрегатные функции SQL (AVG) через репозиторий.

demologin · 2026-01-30T15:35:45Z

src/main/java/com/javarush/reviewplatform/review/ReviewService.java

+        return saved;
+    }
+
+    private ProductTo updateProduct(Long productId) {


Использование Stream API для вычисления среднего значения — отличная практика Java 8+, но здесь лучше применить метод DoubleStream.average() для чистоты кода.

demologin · 2026-01-30T15:35:46Z

src/main/java/com/javarush/reviewplatform/user/UserService.java

+    public UserService(UserRepository repository, UserMapper mapper, PasswordEncoder passwordEncoder) {
+        super(repository, mapper);
+        this.passwordEncoder = passwordEncoder;
+    }


Поле passwordEncoder должно быть помечено как final для обеспечения иммутабельности и следования принципам внедрения зависимостей через конструктор.

demologin · 2026-01-30T15:35:47Z

src/main/java/com/javarush/reviewplatform/user/UserService.java

+                .map(mapper::mapToDto)
+                .orElseThrow(() -> new UserNotFoundException("User not found with username: " + username));
+    }
+}


Метод поиска пользователя бросает RuntimeException без описания. На собеседовании это сочтут плохим тоном; лучше использовать кастомное исключение или EntityNotFoundException.

demologin · 2026-01-30T15:35:48Z

src/main/java/com/javarush/reviewplatform/product/ProductService.java

+
+@Service
+public class ProductService extends AbstractBaseService<Product, ProductTo, ProductRepository, ProductMapper> {
+    public ProductService(ProductRepository repository, ProductMapper mapper) {


Отсутствие модификатора доступа у полей сервиса (package-private по умолчанию). Согласно Java Code Convention, поля должны быть private.

demologin · 2026-01-30T15:35:48Z

src/main/java/com/javarush/reviewplatform/auth/JwtProvider.java

+    @Getter
+    private long expiration;
+
+    public String generateToken(CustomUserDetails userDetails, List<String> roles) {


Секретный ключ захардкожен прямо в коде. В продвинутой разработке такие данные выносятся в переменные окружения или конфигурационные файлы.

demologin · 2026-01-30T15:35:49Z

src/main/java/com/javarush/reviewplatform/category/CategoryRestController.java

+
+    @GetMapping("/{id}")
+    public CategoryTo getCategory(@PathVariable Long id) {
+        return categoryService.getById(id);


В контроллере используется System.out.println для логирования. В Spring Boot проектах необходимо использовать логгеры (например, Slf4j), чтобы управлять уровнями вывода данных.

demologin · 2026-01-30T15:35:50Z

src/main/java/com/javarush/reviewplatform/review/ReviewRestController.java

+    }
+
+    @DeleteMapping("/{id}")
+    @PreAuthorize("hasAnyRole('ADMIN', 'MANAGER')")


Аннотация @PreAuthorize использует роли напрямую. Хорошей практикой считается вынос названий ролей в константы для предотвращения опечаток.

demologin · 2026-01-30T15:35:51Z

src/main/java/com/javarush/reviewplatform/user/User.java

+import jakarta.persistence.Enumerated;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;


Использование @DaTa из Lombok в JPA сущностях может привести к проблемам с производительностью и бесконечной рекурсии в методах hashCode/equals. Лучше использовать @Getter и @Setter.

demologin · 2026-01-30T15:35:52Z

src/test/java/com/javarush/reviewplatform/category/CategoryRestControllerIT.java

+
+    @Test
+    @WithMockUser // Обычный пользователь
+    void should_getAllCategories() throws Exception {


Название теста should_getAllCategories нарушает стандарт camelCase для Java. В тестах это допустимо, но лучше придерживаться единого стиля в проекте.

demologin · 2026-01-30T15:35:53Z

src/test/java/com/javarush/reviewplatform/review/ReviewServiceTest.java

+
+        // Настраиваем поведение базового сохранения (super.save)
+        when(mapper.mapToEntity(inputDto)).thenReturn(reviewEntity);
+        when(repository.save(reviewEntity)).thenReturn(reviewEntity);


Тест перегружен настройками Mockito (when/thenReturn). Это признак того, что тестируемый метод делает слишком много (нарушение Single Responsibility).

demologin · 2026-01-30T15:35:53Z

src/main/java/com/javarush/reviewplatform/product/Product.java

+import java.util.ArrayList;
+import java.util.List;
+
+@Entity


Поле category помечено как @manytoone, но не указан FetchType. По умолчанию это Eager, что может вызвать проблему N+1. Рекомендуется всегда ставить LAZY.

demologin · 2026-01-30T15:35:54Z

src/main/java/com/javarush/reviewplatform/auth/AuthRequest.java

+
+import lombok.AllArgsConstructor;
+import lombok.Data;
+import lombok.NoArgsConstructor;


Класс для DTO не является финальным и не использует преимущества неизменяемости (final поля).

demologin · 2026-01-30T15:35:55Z

src/main/java/com/javarush/reviewplatform/review/ReviewMapper.java

+import java.util.List;
+
+@Mapper(componentModel = "spring")
+public interface ReviewMapper extends BaseMapper<Review, ReviewTo> {


Логика преобразования userId в сущность User находится внутри маппера и требует обращения к репозиторию. Это делает маппер 'тяжелым' и затрудняет тестирование.

demologin · 2026-01-30T15:35:56Z

src/main/java/com/javarush/reviewplatform/util/Constant.java

+        throw new IllegalStateException("Utility class");
+    }
+    // ============ URL PATHS =============
+    public static final class Path {


Интерфейс для констант — это антипаттерн (Constant Interface). Следует использовать финальный класс с приватным конструктором.

demologin · 2026-01-30T15:35:57Z

src/main/java/com/javarush/reviewplatform/review/ReviewViewTo.java

+@Setter
+@NoArgsConstructor
+@SuperBuilder
+public class ReviewViewTo extends ReviewTo {


Наследование DTO (ReviewViewTo extends ReviewTo) усложняет структуру данных. Часто лучше использовать композицию или плоские структуры для API.

demologin · 2026-01-30T15:35:58Z

src/main/java/com/javarush/reviewplatform/user/UserTo.java

+import net.minidev.json.annotate.JsonIgnore;
+
+@Getter
+@Setter


Отсутствует валидация полей (например, @Email, @notblank). В Spring Boot важно проверять входные данные на уровне контроллера.

DEEXORT added 24 commits January 14, 2026 21:23

dev(core): Created entities and liquibase changelog for database

504ef71

dev(core): Added Category repository and service. Added common packag…

b02b855

…e with interfaces.

dev(core): Added product controller

7328c7d

dev(core): Fixed Product controller

ff3ea6c

dev(core): Added review controller and simple styles for templates

6ffe5d9

dev(core): Added user registration

4786ac5

dev(core): Added CategoryRestController and Utility class with constants

bf39f6d

dev(core): Added SecurityConfig

61d4c7f

dev(core): Added changelog with admin user. Fix UI templates

6d2f956

dev(core): Changed view templates for products.

64599a5

dev(core): Fix category and product templates

6f9c4d3

dev(core): Added JwtProvider for generate JWT token and JwtCookieAuth…

87efc18

…enticationHandler for sending cookie. Added REST controllers for Product and Review.

dev(core): Changed view templates for products.

347bf7c

dev(core): Added custom UserDetails and fixed security configuration

1ae8598

dev(core): Separated security configuration for REST and UI. Template…

3f49c7e

…s adjusted. dev(test): Added IT tests for UserRestController.

dev(core): Added endpoints for user reviews.

179dd20

dev(test): Added and fixed IT tests for controllers

dev(core): Fixed styles for app

39d8d76

Create README.md

47e3f1a

Update README.md

fe088e5

dev(core): Added docker-compose and fixed thymeleaf templates

bd46bfd

dev(core): Final version

d95a74d

Merge remote-tracking branch 'origin/master'

57721e2

dev(core): Final

175f343

Merge branch 'base' into feature/full-project

9244fa5

DEEXORT changed the title ~~Final Spring Project~~ Gimaldinov - Final Spring Project Jan 28, 2026

demologin reviewed Jan 30, 2026

View reviewed changes

Conversation

DEEXORT commented Jan 28, 2026

Uh oh!

demologin commented Jan 30, 2026

Общий вывод по проекту

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants