Merge pull request #77 from kea-dpang/develop

Develop
kea-dpang · Feb 6, 2024 · 06e5875 · 06e5875
2 parents 1642f3f + 20d637a
commit 06e5875
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 21 deletions.
diff --git a/src/main/java/kea/dpang/item/controller/ItemController.java b/src/main/java/kea/dpang/item/controller/ItemController.java
@@ -121,7 +121,7 @@ public ResponseEntity<SuccessResponse<List<ReviewDto>>> getReviewList(
 
     @GetMapping("/popular/list")
     @Operation(summary = "인기 상품 리스트 조회", description = "인기 상품 정보를 페이지 정보에 따라 조회합니다.")
-    public ResponseEntity<SuccessResponse<List<PopularItemDto>>> getPopularItems(@RequestParam List<Long> itemIdList, Pageable pageable) {
+    public ResponseEntity<SuccessResponse<List<PopularItemDto>>> getPopularItems(Pageable pageable) {
         List<PopularItemDto> popularItems = itemService.getPopularItems();
         return new ResponseEntity<>(
                 new SuccessResponse<>(HttpStatus.OK.value(),"인기 상품 리스트가 조회되었습니다.", popularItems),

diff --git a/src/main/java/kea/dpang/item/exception/SellerNotFoundException.java b/src/main/java/kea/dpang/item/exception/SellerNotFoundException.java
@@ -0,0 +1,16 @@
+package kea.dpang.item.exception;
+
+import lombok.Getter;
+import org.springframework.http.HttpStatus;
+import org.springframework.web.bind.annotation.ResponseStatus;
+
+@Getter
+@ResponseStatus(value = HttpStatus.NOT_FOUND)
+public class SellerNotFoundException extends RuntimeException {
+    private final Long sellerId;
+
+    public SellerNotFoundException(Long sellerId) {
+        super(String.format("판매처를 찾을 수 없음: 판매처 ID - '%s'", sellerId));
+        this.sellerId = sellerId;
+    }
+}
diff --git a/src/main/java/kea/dpang/item/service/ItemServiceImpl.java b/src/main/java/kea/dpang/item/service/ItemServiceImpl.java
@@ -5,6 +5,7 @@
 import kea.dpang.item.entity.Item;
 import kea.dpang.item.entity.SubCategory;
 import kea.dpang.item.exception.ItemNotFoundException;
+import kea.dpang.item.exception.SellerNotFoundException;
 import kea.dpang.item.feign.SellerServiceFeignClient;
 import kea.dpang.item.repository.ItemRepository;
 import lombok.RequiredArgsConstructor;
@@ -119,9 +120,6 @@ public Page<ItemDetailDto> getItemList(Category category, SubCategory subCategor
         log.info("상품 리스트 조회를 시작합니다 : 카테고리 = {}, 서브카테고리 = {}, 판매자ID = {}, 최소가격 = {}, 최대가격 = {}, 키워드 = {}, 페이지 요청 정보 = {}", category, subCategory, sellerId, minPrice, maxPrice, keyword, pageable);
         Page<Item> items = filterItems(category, subCategory, sellerId, minPrice, maxPrice, keyword, pageable);
 
-//        log.info("판매자 이름 조회를 시작합니다 : 판매자ID = {}", sellerId);
-//        String sellerName = sellerServiceFeignClient.getSeller(sellerId).getBody().getData().toLowerCase();
-
         log.info("상품 리스트를 ItemResponseDto로 변환합니다.");
         return items.map(item -> {
             log.info("판매자 이름 조회를 시작합니다 : 판매자ID = {}", item.getSellerId());
@@ -224,22 +222,5 @@ public void changeStock(List<UpdateStockRequestDto> updateStockRequestDtos) {
         log.info("모든 아이템의 재고 수량 변경이 완료되었습니다.");
     }
 
-//    @Override
-//    public void updateItemDiscount(List<Long> itemIds, UpdateItemDiscountDto dto) {
-//        // 상품 조회
-//        Item item = itemRepository.findById(itemIds, dto);
-//        // 할인 정보 업데이트
-//        item.setDiscount(updateItemDiscountDto.getDiscountRate());
-//        itemRepository.save(item);
-//    }
-//
-//    @Override
-//    public void deleteItemDiscount(Long itemId) {
-//        // 상품 조회
-//        Item item = itemRepository.findById(itemId);
-//        // 할인 정보 삭제
-//        item.setDiscount(null);
-//        itemRepository.save(item);
-//    }
 
 }