Merge pull request #46 from hyujikoh/feat/sorted-list

refactor(cache): 캐시 키 형식 및 오류 메시지 개선

Author: hyujikoh

apps/commerce-api/src/main/java/com/loopers/application/point/PointFacade.java

@@ -1,5 +1,7 @@
 package com.loopers.application.point;
 
+import java.math.BigDecimal;
+
 import org.springframework.stereotype.Component;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -36,7 +38,7 @@ public PointV1Dtos.PointInfo getPointInfo(String username) {
 
     @Transactional
     public PointV1Dtos.PointChargeResponse chargePoint(String username, PointV1Dtos.PointChargeRequest request) {
-        java.math.BigDecimal totalAmount = pointService.charge(username, request.amount());
+        BigDecimal totalAmount = pointService.charge(username, request.amount());
 
         return new PointV1Dtos.PointChargeResponse(username, totalAmount);
     }

apps/commerce-api/src/main/java/com/loopers/support/error/ErrorType.java

@@ -29,7 +29,7 @@ public enum ErrorType {
 
     //좋아요 관련 오류
     ALREADY_LIKED_PRODUCT(HttpStatus.CONFLICT, HttpStatus.CONFLICT.getReasonPhrase(), "이미 좋아요한 상품입니다."),
-    NOT_EXIST_LIKED(HttpStatus.BAD_REQUEST, HttpStatus.BAD_REQUEST.getReasonPhrase(), "좋아요하지 않은 상품입니다."),
+    NOT_EXIST_LIKED(HttpStatus.BAD_REQUEST, HttpStatus.NOT_FOUND.getReasonPhrase(), "좋아요하지 않은 상품입니다."),
 
     // 주문 관련 오류
     NOT_FOUND_ORDER(HttpStatus.NOT_FOUND, HttpStatus.NOT_FOUND.getReasonPhrase(), "존재하지 않는 주문입니다."),

apps/commerce-api/src/main/java/com/loopers/util/ProductDataGeneratorRunner.java

@@ -2,6 +2,7 @@
 
 import net.datafaker.Faker;
 import java.math.BigDecimal;
+import java.math.RoundingMode;
 import java.util.*;
 
 import org.springframework.boot.CommandLineRunner;
@@ -219,7 +220,7 @@ private ProductEntity createRandomProduct(Long brandId) {
         BigDecimal originPrice = generateRandomPrice(10000, 500000);
         BigDecimal discountPrice = random.nextDouble() > 0.5
                 ? originPrice.multiply(BigDecimal.valueOf(random.nextDouble() * 0.8 + 0.1))
-                .setScale(0, java.math.RoundingMode.HALF_UP)
+                .setScale(0, RoundingMode.HALF_UP)
                 : null;
         int stockQuantity = random.nextInt(1000) + 1;
 

apps/commerce-api/src/test/java/com/loopers/domain/user/UserUnitTest.java

@@ -1,5 +1,7 @@
 package com.loopers.domain.user;
 
+import java.math.BigDecimal;
+
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
@@ -130,13 +132,13 @@ void can_charge_point() {
             // given
             UserDomainCreateRequest userRegisterRequest = UserTestFixture.createDefaultUserDomainRequest();
             UserEntity userEntity = UserEntity.createUserEntity(userRegisterRequest);
-            java.math.BigDecimal chargeAmount = new java.math.BigDecimal("1000");
+            BigDecimal chargeAmount = new BigDecimal("1000");
 
             // when
             userEntity.chargePoint(chargeAmount);
 
             // then
-            UserTestFixture.assertUserPointAmount(userEntity, new java.math.BigDecimal("1000.00"));
+            UserTestFixture.assertUserPointAmount(userEntity, new BigDecimal("1000.00"));
         }
 
         @Test
@@ -147,8 +149,8 @@ void charge_fails_with_zero_or_negative_amount() {
             UserEntity userEntity = UserEntity.createUserEntity(userRegisterRequest);
 
             // when & then
-            UserTestFixture.assertChargePointFails(userEntity, java.math.BigDecimal.ZERO, "충전 금액은 0보다 커야 합니다.");
-            UserTestFixture.assertChargePointFails(userEntity, new java.math.BigDecimal("-100"), "충전 금액은 0보다 커야 합니다.");
+            UserTestFixture.assertChargePointFails(userEntity, BigDecimal.ZERO, "충전 금액은 0보다 커야 합니다.");
+            UserTestFixture.assertChargePointFails(userEntity, new BigDecimal("-100"), "충전 금액은 0보다 커야 합니다.");
         }
     }
 }

apps/commerce-streamer/src/main/java/com/loopers/support/error/ErrorType.java

@@ -29,7 +29,7 @@ public enum ErrorType {
 
     //좋아요 관련 오류
     ALREADY_LIKED_PRODUCT(HttpStatus.CONFLICT, HttpStatus.CONFLICT.getReasonPhrase(), "이미 좋아요한 상품입니다."),
-    NOT_EXIST_LIKED(HttpStatus.BAD_REQUEST, HttpStatus.BAD_REQUEST.getReasonPhrase(), "좋아요하지 않은 상품입니다."),
+    NOT_EXIST_LIKED(HttpStatus.BAD_REQUEST, HttpStatus.NOT_FOUND.getReasonPhrase(), "좋아요하지 않은 상품입니다."),
 
     // 주문 관련 오류
     NOT_FOUND_ORDER(HttpStatus.NOT_FOUND, HttpStatus.NOT_FOUND.getReasonPhrase(), "존재하지 않는 주문입니다."),

modules/jpa/src/main/generated/com/loopers/domain/QBaseEntity.java

@@ -13,7 +13,7 @@
  * QBaseEntity is a Querydsl query type for BaseEntity
  */
 @Generated("com.querydsl.codegen.DefaultSupertypeSerializer")
-public class QBaseEntity extends EntityPathBase<BaseEntity> {
+public class QBaseEntity extends EntityPathBase<BaseEntity<?>> {
 
     private static final long serialVersionUID = 1030422725L;
 
@@ -27,16 +27,19 @@ public class QBaseEntity extends EntityPathBase<BaseEntity> {
 
     public final DateTimePath<java.time.ZonedDateTime> updatedAt = createDateTime("updatedAt", java.time.ZonedDateTime.class);
 
+    @SuppressWarnings({"all", "rawtypes", "unchecked"})
     public QBaseEntity(String variable) {
-        super(BaseEntity.class, forVariable(variable));
+        super((Class) BaseEntity.class, forVariable(variable));
     }
 
+    @SuppressWarnings({"all", "rawtypes", "unchecked"})
     public QBaseEntity(Path<? extends BaseEntity> path) {
-        super(path.getType(), path.getMetadata());
+        super((Class) path.getType(), path.getMetadata());
     }
 
+    @SuppressWarnings({"all", "rawtypes", "unchecked"})
     public QBaseEntity(PathMetadata metadata) {
-        super(BaseEntity.class, metadata);
+        super((Class) BaseEntity.class, metadata);
     }
 
 }

modules/redis/src/main/java/com/loopers/cache/CacheKeyGenerator.java

@@ -135,7 +135,7 @@ private String generateSortString(Sort sort) {
     }
 
     /**
-     * 일간 랭킹 키 생성: ranking:all:2025-12-23
+     * 일간 랭킹 키 생성: ranking:all:20251223
      */
     public String generateDailyRankingKey(LocalDate date) {
         return new StringJoiner(DELIMITER)

modules/redis/src/main/java/com/loopers/cache/dto/CachePayloads.java

@@ -1,5 +1,8 @@
 package com.loopers.cache.dto;
 
+import java.math.BigDecimal;
+import java.util.Objects;
+
 import lombok.Getter;
 
 /**
@@ -47,7 +50,7 @@ public java.time.LocalDate getEventDate() {
         }
 
         /**
-         * 조회 이벤트 생성 생성
+         * 조회 이벤트 생성
          */
         public static RankingScore forProductView(Long productId, long occurredAt) {
             return new RankingScore(productId, EventType.PRODUCT_VIEW, 1.0, occurredAt);
@@ -63,7 +66,8 @@ public static RankingScore forLikeAction(Long productId, long occurredAt) {
         /**
          * 주문 이벤트 생성 메소드 (가격 * 수량 기반, 로그 정규화)
          */
-        public static RankingScore forPaymentSuccess(Long productId, java.math.BigDecimal totalPrice, long occurredAt) {
+        public static RankingScore forPaymentSuccess(Long productId, BigDecimal totalPrice, long occurredAt) {
+            Objects.requireNonNull(totalPrice);
             // 로그 정규화 적용하여 극값 방지
             // Math.log(x + 1)을 사용하여 0원일 때도 안전하게 처리
             double normalizedScore = Math.log(totalPrice.doubleValue() + 1);