fix : PR코드 수정

adminhelper · adminhelper · commit c5754ffe7e7c · 2025-11-18T01:02:46.000+09:00
diff --git a/apps/commerce-api/src/main/java/com/loopers/application/like/LikeFacade.java b/apps/commerce-api/src/main/java/com/loopers/application/like/LikeFacade.java
@@ -3,7 +3,6 @@
 import com.loopers.domain.like.LikeService;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Component;
-import org.springframework.transaction.annotation.Transactional;
 
 /**
  * packageName : com.loopers.application.like
@@ -18,7 +17,6 @@
  */
 @Component
 @RequiredArgsConstructor
-@Transactional
 public class LikeFacade {
 
     private final LikeService likeService;
diff --git a/apps/commerce-api/src/main/java/com/loopers/application/order/OrderFacade.java b/apps/commerce-api/src/main/java/com/loopers/application/order/OrderFacade.java
@@ -4,7 +4,6 @@
 import com.loopers.domain.order.OrderItem;
 import com.loopers.domain.order.OrderService;
 import com.loopers.domain.order.OrderStatus;
-import com.loopers.domain.point.Point;
 import com.loopers.domain.point.PointService;
 import com.loopers.domain.product.Product;
 import com.loopers.domain.product.ProductService;
@@ -40,7 +39,7 @@ public class OrderFacade {
     public OrderInfo createOrder(CreateOrderCommand command) {
 
         if (command == null || command.items() == null || command.items().isEmpty()) {
-            throw new CoreException(ErrorType.NOT_FOUND, "상품 정보가 비어있습니다");
+            throw new CoreException(ErrorType.BAD_REQUEST, "상품 정보가 비어있습니다");
         }
 
         Order order = Order.create(command.userId());
@@ -71,8 +70,7 @@ public OrderInfo createOrder(CreateOrderCommand command) {
 
         order.updateTotalAmount(totalAmount);
 
-        Point point = pointService.findPointByUserId(command.userId());
-        point.use(totalAmount);
+        pointService.usePoint(command.userId(), totalAmount);
 
         //저장
         Order saved = orderService.createOrder(order);
diff --git a/apps/commerce-api/src/main/java/com/loopers/application/product/ProductFacade.java b/apps/commerce-api/src/main/java/com/loopers/application/product/ProductFacade.java
@@ -31,10 +31,10 @@ public class ProductFacade {
     private final LikeService likeService;
     private final ProductDomainService productDomainService;
 
-    public Page<ProductInfo> getProducts(Pageable pageable) {
-        return productService.getProducts(pageable)
+    public Page<ProductInfo> getProducts(String sort, Pageable pageable) {
+        return productService.getProducts(sort ,pageable)
                 .map(product ->  {
-                    Brand brand = brandService.getBrand(product.getId());
+                    Brand brand = brandService.getBrand(product.getBrandId());
                     long likeCount = likeService.countByProductId(product.getId());
                     return ProductInfo.of(product, brand, likeCount);
                 });
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/brand/Brand.java b/apps/commerce-api/src/main/java/com/loopers/domain/brand/Brand.java
@@ -40,7 +40,7 @@ public static Brand create(String name) {
 
     private String requireValidName(String name) {
         if (name == null || name.isEmpty()) {
-            throw new CoreException(ErrorType.BAD_REQUEST, "상품명 비어 있을수 없습니다.");
+            throw new CoreException(ErrorType.BAD_REQUEST, "브랜드명은 비어 있을 수 없습니다.");
         }
         return name.trim();
     }
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/brand/BrandService.java b/apps/commerce-api/src/main/java/com/loopers/domain/brand/BrandService.java
@@ -23,7 +23,6 @@ public class BrandService {
 
     private final BrandRepository brandRepository;
 
-    @Transactional
     public void save(Brand brand) {
         brandRepository.save(brand);
     }
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/order/Order.java b/apps/commerce-api/src/main/java/com/loopers/domain/order/Order.java
@@ -58,6 +58,7 @@ public static Order create(String userId) {
     }
 
     public void addOrderItem(OrderItem orderItem) {
+        orderItem.setOrder(this);
         this.orderItems.add(orderItem);
     }
 
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/point/Point.java b/apps/commerce-api/src/main/java/com/loopers/domain/point/Point.java
@@ -42,7 +42,6 @@ public void charge(Long chargeAmount) {
             throw new CoreException(ErrorType.BAD_REQUEST, "0원 이하로 포인트를 충전 할수 없습니다.");
         }
         this.balance += chargeAmount;
-        new Point(this.userId, this.balance);
     }
 
     public void use(Long useAmount) {
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/point/PointService.java b/apps/commerce-api/src/main/java/com/loopers/domain/point/PointService.java
@@ -30,11 +30,11 @@ public Point usePoint(String userId, Long useAmount) {
                 .orElseThrow(() -> new CoreException(ErrorType.NOT_FOUND, "포인트 정보를 찾을 수 없습니다."));
 
         if (useAmount == null || useAmount <= 0) {
-            throw new CoreException(ErrorType.NOT_FOUND, "차감할 포인트는 1 이상이어야 합니다.");
+            throw new CoreException(ErrorType.BAD_REQUEST, "차감할 포인트는 1 이상이어야 합니다.");
         }
 
         if (point.getBalance() < useAmount) {
-            throw new CoreException(ErrorType.NOT_FOUND, "포인트가 부족합니다.");
+            throw new CoreException(ErrorType.BAD_REQUEST, "포인트가 부족합니다.");
         }
 
         point.use(useAmount);
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/product/Product.java b/apps/commerce-api/src/main/java/com/loopers/domain/product/Product.java
@@ -83,14 +83,17 @@ private Long requireValidPrice(Long price) {
         return price;
     }
 
-    public Long requireValidLikeCount(Long likeCount) {
+    private Long requireValidLikeCount(Long likeCount) {
         if (likeCount == null || likeCount < 0) {
             throw new CoreException(ErrorType.BAD_REQUEST, "좋아요 개수는 0개 미만으로 설정할 수 없습니다.");
         }
         return likeCount;
     }
 
     private Long requireValidStock(Long stock) {
+        if (stock == null) {
+            throw new CoreException(ErrorType.BAD_REQUEST, "상품 재고는 필수입니다.");
+        }
         if (stock < 0) {
             throw new CoreException(ErrorType.BAD_REQUEST, "상품 재고는 0 미만으로 설정할 수 없습니다.");
         }
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/product/ProductDomainService.java b/apps/commerce-api/src/main/java/com/loopers/domain/product/ProductDomainService.java
@@ -7,6 +7,7 @@
 import com.loopers.support.error.ErrorType;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Component;
+import org.springframework.transaction.annotation.Transactional;
 
 /**
  * packageName : com.loopers.domain.product
@@ -27,6 +28,7 @@ public class ProductDomainService {
     private final BrandRepository brandRepository;
     private final LikeRepository likeRepository;
 
+    @Transactional(readOnly = true)
     public ProductDetail getProductDetail(Long id) {
         Product product = productRepository.findById(id)
                 .orElseThrow(() -> new CoreException(ErrorType.NOT_FOUND, "상품을 찾을 수 없습니다."));
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/product/ProductService.java b/apps/commerce-api/src/main/java/com/loopers/domain/product/ProductService.java
@@ -4,7 +4,9 @@
 import com.loopers.support.error.ErrorType;
 import lombok.RequiredArgsConstructor;
 import org.springframework.data.domain.Page;
+import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
+import org.springframework.data.domain.Sort;
 import org.springframework.stereotype.Component;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -27,8 +29,20 @@ public class ProductService {
     private final ProductRepository productRepository;
 
     @Transactional(readOnly = true)
-    public Page<Product> getProducts(Pageable pageable) {
-        return productRepository.findAll(pageable);
+    public Page<Product> getProducts(String sort, Pageable pageable) {
+        Sort sortOption = switch (sort) {
+            case "price_asc" -> Sort.by("price").ascending();
+            case "likes_desc" -> Sort.by("likeCount").descending();
+            default -> Sort.by("createdAt").descending(); // latest
+        };
+
+        Pageable sortedPageable = PageRequest.of(
+                pageable.getPageNumber(),
+                pageable.getPageSize(),
+                sortOption
+        );
+
+        return productRepository.findAll(sortedPageable);
     }
 
     public Product getProduct(Long productId) {
diff --git a/apps/commerce-api/src/main/java/com/loopers/domain/user/UserService.java b/apps/commerce-api/src/main/java/com/loopers/domain/user/UserService.java
@@ -15,7 +15,7 @@ public class UserService {
     @Transactional
     public User register(String userId, String email, String birth, String gender) {
         userRepository.findByUserId(userId).ifPresent(user -> {
-            throw new CoreException(ErrorType.CONFLICT, "이미 가입된 사용자ID 입니다.");
+            throw new CoreException(ErrorType.CONFLICT, "이미 가입된 사용자 ID 입니다.");
         });
 
         User user = new User(userId, email, birth, gender);
diff --git a/apps/commerce-api/src/main/java/com/loopers/infrastructure/brand/BrandJpaRepository.java b/apps/commerce-api/src/main/java/com/loopers/infrastructure/brand/BrandJpaRepository.java
@@ -3,8 +3,6 @@
 import com.loopers.domain.brand.Brand;
 import org.springframework.data.jpa.repository.JpaRepository;
 
-import java.util.Optional;
-
 /**
  * packageName : com.loopers.infrastructure.brand
  * fileName     : BrandJpaRepository
@@ -17,5 +15,4 @@
  * 2025. 11. 12.     byeonsungmun       최초 생성
  */
 public interface BrandJpaRepository extends JpaRepository<Brand, Long> {
-    Optional<Brand> findById(Long id);
 }
diff --git a/apps/commerce-api/src/main/java/com/loopers/infrastructure/product/ProductRepositoryImpl.java b/apps/commerce-api/src/main/java/com/loopers/infrastructure/product/ProductRepositoryImpl.java
@@ -2,6 +2,8 @@
 
 import com.loopers.domain.product.Product;
 import com.loopers.domain.product.ProductRepository;
+import com.loopers.support.error.CoreException;
+import com.loopers.support.error.ErrorType;
 import lombok.RequiredArgsConstructor;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
@@ -38,13 +40,15 @@ public Optional<Product> findById(Long id) {
 
     @Override
     public void incrementLikeCount(Long productId) {
-        Product product = productJpaRepository.findById(productId).get();
+        Product product = productJpaRepository.findById(productId)
+                .orElseThrow(() -> new CoreException(ErrorType.NOT_FOUND, "상품을 찾을수 없습니다."));
         product.increaseLikeCount();
     }
 
     @Override
     public void decrementLikeCount(Long productId) {
-        Product product = productJpaRepository.findById(productId).get();
+        Product product = productJpaRepository.findById(productId)
+                .orElseThrow(() -> new CoreException(ErrorType.NOT_FOUND, "상품을 찾을수 없습니다."));
         product.decreaseLikeCount();
     }
 
diff --git a/apps/commerce-api/src/main/java/com/loopers/infrastructure/user/UserRepositoryImpl.java b/apps/commerce-api/src/main/java/com/loopers/infrastructure/user/UserRepositoryImpl.java
@@ -20,8 +20,7 @@ public Optional<User> findByUserId(String userId) {
 
     @Override
     public User save(User user) {
-        userJpaRepository.save(user);
-        return user;
+        return userJpaRepository.save(user);
     }
 }
 
diff --git a/apps/commerce-api/src/main/java/com/loopers/interfaces/api/point/PointV1ApiSpec.java b/apps/commerce-api/src/main/java/com/loopers/interfaces/api/point/PointV1ApiSpec.java
@@ -22,7 +22,7 @@ ApiResponse<PointV1Dto.PointResponse> getPoint(
             description = "회원의 포인트를 충전한다."
     )
     ApiResponse<PointV1Dto.PointResponse> chargePoint(
-            @Schema(name = "포인트 충전 요청", description = "조회할 회원 ID")
+            @Schema(name = "포인트 충전 요청", description = "충전할 포인트 정보를 포함한 요청")
             PointV1Dto.ChargePointRequest request
     );
 }
diff --git a/apps/commerce-api/src/test/java/com/loopers/domain/user/UserTest.java b/apps/commerce-api/src/test/java/com/loopers/domain/user/UserTest.java
@@ -42,12 +42,12 @@ void throwsException_whenInvalidEmailFormat() {
         void throwsException_whenInvalidBirthFormat() {
             // given
             String userId = "yh45g";
-            String invalidEmail = "invalid-email-format"; // '@' 없음
-            String birth = "1994-12-05";
+            String email = "valid@loopers.com";
+            String invalidBirth = "19941205"; // 형식 오류: 하이픈 없음
             String gender = "MALE";
 
             // when & then
-            assertThrows(CoreException.class, () -> new User(userId, invalidEmail, birth, gender));
+            assertThrows(CoreException.class, () -> new User(userId, email, invalidBirth, gender));
         }
     }
 }
diff --git a/docs/2round/03-class-diagram.md b/docs/2round/03-class-diagram.md
@@ -32,11 +32,6 @@ class Product {
   Long stock
 }
 
-class Stock {
-  Long productId
-  int quantity
-}
-
 class Like {
   Long id
   String userId
@@ -73,7 +68,6 @@ class Payment {
 %% 관계 설정
 User --> Point
 Brand --> Product
-Product --> Stock
 Product --> Like
 User --> Like 
 User --> Order 
diff --git a/docs/3round/3round.md b/docs/3round/3round.md
@@ -24,7 +24,7 @@
 ## ✅ Checklist
 
 - [x]  상품 정보 객체는 브랜드 정보, 좋아요 수를 포함한다.
-- [ ]  상품의 정렬 조건(`latest`, `price_asc`, `likes_desc`) 을 고려한 조회 기능을 설계했다
+- [x]  상품의 정렬 조건(`latest`, `price_asc`, `likes_desc`) 을 고려한 조회 기능을 설계했다
 - [x]  상품은 재고를 가지고 있고, 주문 시 차감할 수 있어야 한다
 - [x]  재고는 감소만 가능하며 음수 방지는 도메인 레벨에서 처리된다
 
@@ -33,28 +33,28 @@
 - [x]  좋아요는 유저와 상품 간의 관계로 별도 도메인으로 분리했다
 - [x]  중복 좋아요 방지를 위한 멱등성 처리가 구현되었다
 - [x]  상품의 좋아요 수는 상품 상세/목록 조회에서 함께 제공된다
-- [ ]  단위 테스트에서 좋아요 등록/취소/중복 방지 흐름을 검증했다
+- [x]  단위 테스트에서 좋아요 등록/취소/중복 방지 흐름을 검증했다
 
 ### 🛒 Order 도메인
 
-- [ ]  주문은 여러 상품을 포함할 수 있으며, 각 상품의 수량을 명시한다
-- [ ]  주문 시 상품의 재고 차감, 유저 포인트 차감 등을 수행한다
-- [ ]  재고 부족, 포인트 부족 등 예외 흐름을 고려해 설계되었다
-- [ ]  단위 테스트에서 정상 주문 / 예외 주문 흐름을 모두 검증했다
+- [x]  주문은 여러 상품을 포함할 수 있으며, 각 상품의 수량을 명시한다
+- [x]  주문 시 상품의 재고 차감, 유저 포인트 차감 등을 수행한다
+- [x]  재고 부족, 포인트 부족 등 예외 흐름을 고려해 설계되었다
+- [x]  단위 테스트에서 정상 주문 / 예외 주문 흐름을 모두 검증했다
 
 ### 🧩 도메인 서비스
 
-- [ ]  도메인 간 협력 로직은 Domain Service에 위치시켰다
-- [ ]  상품 상세 조회 시 Product + Brand 정보 조합은 도메인 서비스에서 처리했다
-- [ ]  복합 유스케이스는 Application Layer에 존재하고, 도메인 로직은 위임되었다
-- [ ]  도메인 서비스는 상태 없이, 도메인 객체의 협력 중심으로 설계되었다
+- [x]  도메인 간 협력 로직은 Domain Service에 위치시켰다
+- [x]  상품 상세 조회 시 Product + Brand 정보 조합은 도메인 서비스에서 처리했다
+- [x]  복합 유스케이스는 Application Layer에 존재하고, 도메인 로직은 위임되었다
+- [x]  도메인 서비스는 상태 없이, 도메인 객체의 협력 중심으로 설계되었다
 
 ### **🧱 소프트웨어 아키텍처 & 설계**
 
-- [ ]  전체 프로젝트의 구성은 아래 아키텍처를 기반으로 구성되었다
+- [x]  전체 프로젝트의 구성은 아래 아키텍처를 기반으로 구성되었다
     - Application → **Domain** ← Infrastructure
-- [ ]  Application Layer는 도메인 객체를 조합해 흐름을 orchestration 했다
-- [ ]  핵심 비즈니스 로직은 Entity, VO, Domain Service 에 위치한다
-- [ ]  Repository Interface는 Domain Layer 에 정의되고, 구현체는 Infra에 위치한다
-- [ ]  패키지는 계층 + 도메인 기준으로 구성되었다 (`/domain/order`, `/application/like` 등)
-- [ ]  테스트는 외부 의존성을 분리하고, Fake/Stub 등을 사용해 단위 테스트가 가능하게 구성되었다
+- [x]  Application Layer는 도메인 객체를 조합해 흐름을 orchestration 했다
+- [x]  핵심 비즈니스 로직은 Entity, VO, Domain Service 에 위치한다
+- [x]  Repository Interface는 Domain Layer 에 정의되고, 구현체는 Infra에 위치한다
+- [x]  패키지는 계층 + 도메인 기준으로 구성되었다 (`/domain/order`, `/application/like` 등)
+- [x]  테스트는 외부 의존성을 분리하고, Fake/Stub 등을 사용해 단위 테스트가 가능하게 구성되었다

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ public static Brand create(String name) {`
`40`	`40`
`41`	`41`	`private String requireValidName(String name) {`
`42`	`42`	`if (name == null \|\| name.isEmpty()) {`
`43`		`- throw new CoreException(ErrorType.BAD_REQUEST, "상품명 비어 있을수 없습니다.");`
	`43`	`+ throw new CoreException(ErrorType.BAD_REQUEST, "브랜드명은 비어 있을 수 없습니다.");`
`44`	`44`	`}`
`45`	`45`	`return name.trim();`
`46`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,6 @@ public class BrandService {`
`23`	`23`
`24`	`24`	`private final BrandRepository brandRepository;`
`25`	`25`
`26`		`- @Transactional`
`27`	`26`	`public void save(Brand brand) {`
`28`	`27`	`brandRepository.save(brand);`
`29`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ public static Order create(String userId) {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`public void addOrderItem(OrderItem orderItem) {`
	`61`	`+ orderItem.setOrder(this);`
`61`	`62`	`this.orderItems.add(orderItem);`
`62`	`63`	`}`
`63`	`64`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,6 @@ public void charge(Long chargeAmount) {`
`42`	`42`	`throw new CoreException(ErrorType.BAD_REQUEST, "0원 이하로 포인트를 충전 할수 없습니다.");`
`43`	`43`	`}`
`44`	`44`	`this.balance += chargeAmount;`
`45`		`- new Point(this.userId, this.balance);`
`46`	`45`	`}`
`47`	`46`
`48`	`47`	`public void use(Long useAmount) {`
Original file line number	Diff line number	Diff line change
`@@ -30,11 +30,11 @@ public Point usePoint(String userId, Long useAmount) {`
`30`	`30`	`.orElseThrow(() -> new CoreException(ErrorType.NOT_FOUND, "포인트 정보를 찾을 수 없습니다."));`
`31`	`31`
`32`	`32`	`if (useAmount == null \|\| useAmount <= 0) {`
`33`		`- throw new CoreException(ErrorType.NOT_FOUND, "차감할 포인트는 1 이상이어야 합니다.");`
	`33`	`+ throw new CoreException(ErrorType.BAD_REQUEST, "차감할 포인트는 1 이상이어야 합니다.");`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`if (point.getBalance() < useAmount) {`
`37`		`- throw new CoreException(ErrorType.NOT_FOUND, "포인트가 부족합니다.");`
	`37`	`+ throw new CoreException(ErrorType.BAD_REQUEST, "포인트가 부족합니다.");`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`point.use(useAmount);`
Original file line number	Diff line number	Diff line change
`@@ -83,14 +83,17 @@ private Long requireValidPrice(Long price) {`
`83`	`83`	`return price;`
`84`	`84`	`}`
`85`	`85`
`86`		`- public Long requireValidLikeCount(Long likeCount) {`
	`86`	`+ private Long requireValidLikeCount(Long likeCount) {`
`87`	`87`	`if (likeCount == null \|\| likeCount < 0) {`
`88`	`88`	`throw new CoreException(ErrorType.BAD_REQUEST, "좋아요 개수는 0개 미만으로 설정할 수 없습니다.");`
`89`	`89`	`}`
`90`	`90`	`return likeCount;`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`private Long requireValidStock(Long stock) {`
	`94`	`+ if (stock == null) {`
	`95`	`+ throw new CoreException(ErrorType.BAD_REQUEST, "상품 재고는 필수입니다.");`
	`96`	`+ }`
`94`	`97`	`if (stock < 0) {`
`95`	`98`	`throw new CoreException(ErrorType.BAD_REQUEST, "상품 재고는 0 미만으로 설정할 수 없습니다.");`
`96`	`99`	`}`