Merge pull request #48 from team-nabi/NABI-250--beommo-feat--famous-c…

…ards

NABI-250--feat : 인기상품 조회 기능 추가

src/main/java/org/prgrms/nabimarketbe/domain/card/api/CardController.java

@@ -1,17 +1,20 @@
 package org.prgrms.nabimarketbe.domain.card.api;
 
-import java.util.List;
-
+import io.swagger.v3.oas.annotations.Parameter;
+import io.swagger.v3.oas.annotations.enums.ParameterIn;
+import io.swagger.v3.oas.annotations.media.Schema;
+import lombok.RequiredArgsConstructor;
 import org.prgrms.nabimarketbe.domain.card.dto.request.CardCreateRequestDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.request.CardStatusUpdateRequestDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.request.CardUpdateRequestDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.CardCreateResponseDTO;
-import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardPagingResponseDTO;
+import org.prgrms.nabimarketbe.domain.card.dto.response.CardUpdateResponseDTO;
+import org.prgrms.nabimarketbe.domain.card.dto.response.projection.CardFamousResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardListResponseDTO;
+import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardPagingResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardResponseDTO;
-import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardUserResponseDTO;
-import org.prgrms.nabimarketbe.domain.card.dto.response.CardUpdateResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardSuggestionResponseDTO;
+import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardUserResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.entity.CardStatus;
 import org.prgrms.nabimarketbe.domain.card.service.CardService;
 import org.prgrms.nabimarketbe.domain.category.entity.CategoryEnum;
@@ -31,10 +34,7 @@
 import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.bind.annotation.RestController;
 
-import io.swagger.v3.oas.annotations.Parameter;
-import io.swagger.v3.oas.annotations.enums.ParameterIn;
-import io.swagger.v3.oas.annotations.media.Schema;
-import lombok.RequiredArgsConstructor;
+import java.util.List;
 
 @RestController
 @RequestMapping("/api/v1/cards")
@@ -58,43 +58,43 @@ public ResponseEntity<SingleResult<CardResponseDTO<CardCreateResponseDTO>>> crea
         return ResponseEntity.ok(ResponseFactory.getSingleResult(card));
     }
 
-   @GetMapping("/{cardId}")
-   public ResponseEntity<SingleResult<CardUserResponseDTO>> getCardById(
-       @RequestHeader(value = "Authorization", required = false) String token,
-       @PathVariable Long cardId) {
-       CardUserResponseDTO cardSingleReadResponseDTO = cardService.getCardById(token, cardId);
+    @GetMapping("/{cardId}")
+    public ResponseEntity<SingleResult<CardUserResponseDTO>> getCardById(
+        @RequestHeader(value = "Authorization", required = false) String token,
+        @PathVariable Long cardId) {
+        CardUserResponseDTO cardSingleReadResponseDTO = cardService.getCardById(token, cardId);
 
-       return ResponseEntity.ok(ResponseFactory.getSingleResult(cardSingleReadResponseDTO));
-   }
+        return ResponseEntity.ok(ResponseFactory.getSingleResult(cardSingleReadResponseDTO));
+    }
 
     @GetMapping
     public ResponseEntity<SingleResult<CardPagingResponseDTO>> getCardsByCondition(
-            @RequestParam(required = false) CategoryEnum category,
-            @RequestParam(required = false) PriceRange priceRange,
-            @RequestParam(required = false) List<CardStatus> status,
-            @RequestParam(required = false) String cardTitle,
-            @RequestParam(required = false) String cursorId,
-            @RequestParam Integer size
+        @RequestParam(required = false) CategoryEnum category,
+        @RequestParam(required = false) PriceRange priceRange,
+        @RequestParam(required = false) List<CardStatus> status,
+        @RequestParam(required = false) String cardTitle,
+        @RequestParam(required = false) String cursorId,
+        @RequestParam Integer size
     ) {
         CardPagingResponseDTO cardListReadPagingResponseDTO = cardService.getCardsByCondition(
-                category,
-                priceRange,
-                status,
-                cardTitle,
-                cursorId,
-                size
+            category,
+            priceRange,
+            status,
+            cardTitle,
+            cursorId,
+            size
         );
 
         return ResponseEntity.ok(ResponseFactory.getSingleResult(cardListReadPagingResponseDTO));
     }
 
     @GetMapping("/{cardId}/available-cards")
     public ResponseEntity<SingleResult<CardListResponseDTO<CardSuggestionResponseDTO>>> getSuggestionAvailableCards(
-            @RequestHeader(name = "Authorization") String token,
-            @PathVariable(name = "cardId") Long targetCardId
+        @RequestHeader(name = "Authorization") String token,
+        @PathVariable(name = "cardId") Long targetCardId
     ) {
         CardListResponseDTO<CardSuggestionResponseDTO> cardListResponseDTO
-                = cardService.getSuggestionAvailableCards(token, targetCardId);
+            = cardService.getSuggestionAvailableCards(token, targetCardId);
 
         return ResponseEntity.ok(ResponseFactory.getSingleResult(cardListResponseDTO));
     }
@@ -116,9 +116,9 @@ public ResponseEntity<SingleResult<CardResponseDTO<CardUpdateResponseDTO>>> upda
 
     @PutMapping("/status/{cardId}")
     public ResponseEntity<CommonResult> updateCardStatusById(
-            @RequestHeader(name = "authorization") String token,
-            @PathVariable Long cardId,
-            @RequestBody CardStatusUpdateRequestDTO cardStatusUpdateRequestDTO
+        @RequestHeader(name = "authorization") String token,
+        @PathVariable Long cardId,
+        @RequestBody CardStatusUpdateRequestDTO cardStatusUpdateRequestDTO
     ) {
         cardService.updateCardStatusById(
             token,
@@ -131,28 +131,35 @@ public ResponseEntity<CommonResult> updateCardStatusById(
 
     @GetMapping("/{status}/my-cards")
     public ResponseEntity<SingleResult<CardPagingResponseDTO>> getMyCardsByStatus(
-            @RequestHeader(name = "authorization") String token,
-            @PathVariable CardStatus status,
-            @RequestParam(required = false) String cursorId,
-            @RequestParam Integer size
+        @RequestHeader(name = "authorization") String token,
+        @PathVariable CardStatus status,
+        @RequestParam(required = false) String cursorId,
+        @RequestParam Integer size
     ) {
         CardPagingResponseDTO cardListReadPagingResponseDTO = cardService.getMyCardsByStatus(
-                token,
-                status,
-                cursorId,
-                size
+            token,
+            status,
+            cursorId,
+            size
         );
 
         return ResponseEntity.ok(ResponseFactory.getSingleResult(cardListReadPagingResponseDTO));
     }
 
     @DeleteMapping("/{cardId}")
     public ResponseEntity<CommonResult> deleteCardById(
-            @RequestHeader(name = "authorization") String token,
-            @PathVariable Long cardId
+        @RequestHeader(name = "authorization") String token,
+        @PathVariable Long cardId
     ) {
         cardService.deleteCardById(token, cardId);
 
         return ResponseEntity.ok(ResponseFactory.getSuccessResult());
     }
+
+    @GetMapping("/popular")
+    public ResponseEntity<SingleResult<CardListResponseDTO<CardFamousResponseDTO>>> getCardsByPopularity() {
+        CardListResponseDTO<CardFamousResponseDTO> cardList = cardService.getCardsByPopularity();
+
+        return ResponseEntity.ok(ResponseFactory.getSingleResult(cardList));
+    }
 }

src/main/java/org/prgrms/nabimarketbe/domain/card/dto/response/projection/CardFamousResponseDTO.java

@@ -0,0 +1,15 @@
+package org.prgrms.nabimarketbe.domain.card.dto.response.projection;
+
+import lombok.Getter;
+import org.prgrms.nabimarketbe.domain.item.entity.PriceRange;
+
+@Getter
+public class CardFamousResponseDTO {
+    private Long cardId;
+
+    private String itemName;
+
+    private PriceRange priceRange;
+
+    private String thumbnail;
+}

src/main/java/org/prgrms/nabimarketbe/domain/card/repository/CardRepositoryCustom.java

@@ -1,33 +1,36 @@
 package org.prgrms.nabimarketbe.domain.card.repository;
 
-import java.util.List;
-
+import org.prgrms.nabimarketbe.domain.card.dto.response.projection.CardFamousResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardPagingResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardSuggestionResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.entity.CardStatus;
 import org.prgrms.nabimarketbe.domain.category.entity.CategoryEnum;
 import org.prgrms.nabimarketbe.domain.item.entity.PriceRange;
 import org.prgrms.nabimarketbe.domain.user.entity.User;
 
+import java.util.List;
+
 public interface CardRepositoryCustom {
     CardPagingResponseDTO getCardsByCondition(
-            CategoryEnum category,
-            PriceRange priceRange,
-            List<CardStatus> status,
-            String title,
-            String cursorId,
-            Integer size
+        CategoryEnum category,
+        PriceRange priceRange,
+        List<CardStatus> status,
+        String title,
+        String cursorId,
+        Integer size
     );
 
     CardPagingResponseDTO getMyCardsByStatus(
-            User user,
-            CardStatus status,
-            String cursorId,
-            Integer size
+        User user,
+        CardStatus status,
+        String cursorId,
+        Integer size
     );
 
     List<CardSuggestionResponseDTO> getSuggestionAvailableCards(
-            Long userId,
-            Long targetCardId
+        Long userId,
+        Long targetCardId
     );
+
+    List<CardFamousResponseDTO> getCardsByPopularity();
 }

src/main/java/org/prgrms/nabimarketbe/domain/card/repository/CardRepositoryImpl.java

@@ -10,6 +10,7 @@
 import com.querydsl.core.types.dsl.StringTemplate;
 import com.querydsl.jpa.impl.JPAQueryFactory;
 import lombok.RequiredArgsConstructor;
+import org.prgrms.nabimarketbe.domain.card.dto.response.projection.CardFamousResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.projection.CardInfoResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.projection.CardListReadResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardPagingResponseDTO;
@@ -31,6 +32,8 @@
 
 @RequiredArgsConstructor
 public class CardRepositoryImpl implements CardRepositoryCustom {
+    private static final int FAMOUS_CARD_SIZE = 5;
+
     private final JPAQueryFactory jpaQueryFactory;
 
     @Override
@@ -65,8 +68,8 @@ public CardPagingResponseDTO getCardsByCondition(
                 titleEquals(cardTitle)
             )
             .orderBy(getOrderSpecifier(Sort.by(
-                    Sort.Order.desc("createdDate"),
-                    Sort.Order.desc("cardId")
+                Sort.Order.desc("createdDate"),
+                Sort.Order.desc("cardId")
             )))
             .limit(size)
             .fetch();
@@ -146,6 +149,35 @@ public List<CardSuggestionResponseDTO> getSuggestionAvailableCards(
         return cardList;
     }
 
+    @Override
+    public List<CardFamousResponseDTO> getCardsByPopularity() {
+        List<CardFamousResponseDTO> cardList = jpaQueryFactory
+            .select(
+                Projections.fields(
+                    CardFamousResponseDTO.class,
+                    card.cardId,
+                    card.item.itemName,
+                    card.item.priceRange,
+                    card.thumbnail
+                )
+            )
+            .from(card)
+            .where(statusEquals(CardStatus.TRADE_AVAILABLE))
+            .orderBy(card.viewCount.desc(), card.dibCount.desc())
+            .limit(FAMOUS_CARD_SIZE)
+            .fetch();
+
+        return cardList;
+    }
+
+    private BooleanExpression statusEquals(CardStatus status) {
+        if (status == null) {
+            return null;
+        }
+
+        return card.status.eq(status);
+    }
+
     private BooleanExpression cursorId(String cursorId) {
         if (cursorId == null) {
             return null;

src/main/java/org/prgrms/nabimarketbe/domain/card/service/CardService.java

@@ -7,6 +7,7 @@
 import org.prgrms.nabimarketbe.domain.card.dto.response.CardCreateResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.CardDetailResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.CardUpdateResponseDTO;
+import org.prgrms.nabimarketbe.domain.card.dto.response.projection.CardFamousResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardListResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardPagingResponseDTO;
 import org.prgrms.nabimarketbe.domain.card.dto.response.wrapper.CardResponseDTO;
@@ -161,7 +162,7 @@ public CardResponseDTO<CardUpdateResponseDTO> updateCardById(
         return new CardResponseDTO<>(cardUpdateResponseDTO);
     }
 
-    @Transactional(readOnly = true)
+    @Transactional
     public CardUserResponseDTO getCardById(
         String token,
         Long cardId
@@ -317,7 +318,7 @@ public List<CardSuggestionResponseDTO> getSuggestionResultCardList(
     ) {
         Card targetCard = cardRepository.findById(targetId)
             .orElseThrow(() -> new BaseException(ErrorCode.CARD_NOT_FOUND));
-        
+
         Boolean pokeAvailable = targetCard.getPokeAvailable();
         PriceRange priceRange = targetCard.getItem().getPriceRange();
 
@@ -327,6 +328,14 @@ public List<CardSuggestionResponseDTO> getSuggestionResultCardList(
         return parseCardListWithOnlyOffer(cardList, priceRange);
     }
 
+    @Transactional(readOnly = true)
+    public CardListResponseDTO<CardFamousResponseDTO> getCardsByPopularity() {
+        List<CardFamousResponseDTO> cardList = cardRepository.getCardsByPopularity();
+        CardListResponseDTO<CardFamousResponseDTO> response = new CardListResponseDTO<>(cardList);
+
+        return response;
+    }
+
     private List<CardSuggestionResponseDTO> parseCardListWithPokeAndOffer(
         List<CardSuggestionResponseDTO> cardList,
         PriceRange targetPriceRange

src/main/java/org/prgrms/nabimarketbe/global/error/GlobalExceptionHandler.java

@@ -17,39 +17,43 @@ public class GlobalExceptionHandler {
     @ExceptionHandler(Exception.class)
     protected ResponseEntity<CommonResult> defaultException(Exception e) {
         log.error(e.getMessage());
-      
+
         return new ResponseEntity<>(
             ResponseFactory.getFailResult(
-                    ErrorCode.UNKNOWN.getCode(),
-                    ErrorCode.UNKNOWN.getMessage()
+                ErrorCode.UNKNOWN.getCode(),
+                ErrorCode.UNKNOWN.getMessage()
             ),
             HttpStatus.INTERNAL_SERVER_ERROR
         );
     }
 
     @ExceptionHandler(BaseException.class)
     protected ResponseEntity<CommonResult> handleCustomException(BaseException e) {
+        log.error(e.getErrorCode().getMessage());
+
         ErrorCode errorCode = e.getErrorCode();
 
         return new ResponseEntity<>(
             ResponseFactory.getFailResult(
-                    errorCode.getCode(),
-                    errorCode.getMessage()
+                errorCode.getCode(),
+                errorCode.getMessage()
             ),
             errorCode.getStatus()
         );
     }
 
     @ExceptionHandler(MethodArgumentNotValidException.class)
     public ResponseEntity<CommonResult> handleValidationExceptions(MethodArgumentNotValidException e) {
+        log.error(e.getMessage());
+
         BindingResult bindingResult = e.getBindingResult();
 
         String message = bindingResult.getFieldError().getDefaultMessage();
 
         return new ResponseEntity<>(
             ResponseFactory.getFailResult(
-                    ErrorCode.INVALID_REQUEST.getCode(),
-                    message),
+                ErrorCode.INVALID_REQUEST.getCode(),
+                message),
             HttpStatus.BAD_REQUEST
         );
     }