Feat : Admin 통계 기능 추가#106
Merged
Merged
Conversation
|
Caution Review failedThe pull request is closed. ## Walkthrough
이번 변경 사항에서는 주문 및 매출 요약 기능을 위한 새로운 API 엔드포인트와 서비스 메서드가 추가되었습니다. QueryDSL 및 Redis를 위한 설정 및 의존성이 도입되었으며, 매장별/부서별 매출 집계와 랭킹 조회를 위한 DTO, 레포지토리, 엔티티가 새로 생성되었습니다. 일부 패키지명 표기 일관성도 정비되었습니다.
## Changes
| 파일/경로 그룹 | 변경 요약 |
|---|---|
| OrderController, OrderService | 매장 오늘 매출 요약 및 TOP 5 매장 매출 조회용 GET 엔드포인트 및 서비스 메서드 추가 |
| .../order/dto/OrderSalesSumDetail.java<br>.../order/dto/TopSalesStoresDetail.java | 매출 요약, TOP 매장 랭킹 정보를 위한 DTO 클래스 추가 |
| .../order/repository/OrderCustomRepository.java<br>.../order/repository/OrderCustomRepositoryImpl.java | 매출 요약, TOP 매장 랭킹 조회용 커스텀 레포지토리 및 구현체 추가 |
| .../order/repository/OrderRepository.java | OrderCustomRepository 상속 추가 |
| .../department/entity/Department.java | 부서 엔티티 신규 추가 |
| .../storePayment/controller, dto, service | 패키지명을 storePayment → storepayment로 소문자 일관화 |
| nowait-common/src/main/java/com/nowait/common/api/ApiResult.java | getError() 메서드 제거(주석 처리) |
| nowait-domain/domain-core-rdb/build.gradle | QueryDSL(Jakarta) 관련 의존성 및 annotation processor 추가, 포맷팅 정비 |
| .../config/QueryDslConfig.java | QueryDSL JPAQueryFactory 빈 등록용 설정 클래스 추가 |
| nowait-domain/domain-redis/build.gradle | Redis 모듈 신규 Gradle 빌드 스크립트 추가 |
| .../config/RedisConfig.java | Redis 연결 및 직렬화 설정용 Spring Config 클래스 추가 |
| .../service/StoreRankCacheService.java | RedisTemplate<String, String> 필드만 가진 서비스 클래스 추가 |
| settings.gradle | :nowait-domain:domain-redis 모듈 빌드에 포함 |
## Sequence Diagram(s)
```mermaid
sequenceDiagram
participant Client
participant OrderController
participant OrderService
participant OrderRepository
participant OrderCustomRepository
Client->>OrderController: GET /sales (오늘 매출 요약)
OrderController->>OrderService: getSaleSumByStoreId(memberDetails)
OrderService->>OrderRepository: findSalesSumByStoreId(storeId)
OrderRepository->>OrderCustomRepository: findSalesSumByStoreId(storeId)
OrderCustomRepository-->>OrderRepository: OrderSalesSumDetail
OrderRepository-->>OrderService: OrderSalesSumDetail
OrderService-->>OrderController: OrderSalesSumDetail
OrderController-->>Client: ApiResult.success(OrderSalesSumDetail)
Client->>OrderController: GET /top-sales (TOP 5 매장 매출)
OrderController->>OrderService: getTop5StoresBySalesToday(memberDetails)
OrderService->>OrderRepository: getTop4PlusMine(storeId)
OrderRepository->>OrderCustomRepository: getTop4PlusMine(storeId)
OrderCustomRepository-->>OrderRepository: List<TopSalesStoresDetail>
OrderRepository-->>OrderService: List<TopSalesStoresDetail>
OrderService-->>OrderController: List<TopSalesStoresDetail>
OrderController-->>Client: ApiResult.success(List<TopSalesStoresDetail>)sequenceDiagram
participant Admin
participant StorePaymentController
participant StorePaymentService
Admin->>StorePaymentController: (기존 엔드포인트 호출)
StorePaymentController->>StorePaymentService: (기존 서비스 호출)
Note right of StorePaymentController: 패키지명만 변경, 로직 동일
sequenceDiagram
participant Spring
participant QueryDslConfig
participant RedisConfig
Spring->>QueryDslConfig: 컨테이너 초기화 시
QueryDslConfig->>Spring: JPAQueryFactory 빈 등록
Spring->>RedisConfig: 컨테이너 초기화 시
RedisConfig->>Spring: RedisConnectionFactory, RedisTemplate 빈 등록
Suggested reviewers
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 14
🧹 Nitpick comments (3)
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/dto/OrderSalesSumDetail.java (2)
14-20: @builder와 명시적 생성자 중복 사용Lombok의
@Builder어노테이션과 명시적 생성자를 동시에 사용하고 있습니다. 일반적으로@Builder만 사용하거나, 특별한 이유가 있다면@AllArgsConstructor를 추가하는 것이 더 명확합니다.+@AllArgsConstructor @Getter @Builder public class OrderSalesSumDetail { - public OrderSalesSumDetail(Long storeId, Integer todaySalesSum, Integer yesterdaySalesSum, - Integer previousDaySales) { - this.storeId = storeId; - this.todaySalesSum = todaySalesSum; - this.yesterdaySalesSum = yesterdaySalesSum; - this.previousDaySales = previousDaySales; - }
8-21: Java 14+ record 클래스 사용 고려불변 데이터 전송을 위한 DTO의 경우 Java 14+의 record 클래스를 사용하면 더 간결하고 안전한 코드를 작성할 수 있습니다.
-@Getter -@Builder -public class OrderSalesSumDetail { - Long storeId; - Integer todaySalesSum; - Integer yesterdaySalesSum; - Integer previousDaySales; - - public OrderSalesSumDetail(Long storeId, Integer todaySalesSum, Integer yesterdaySalesSum, - Integer previousDaySales) { - this.storeId = storeId; - this.todaySalesSum = todaySalesSum; - this.yesterdaySalesSum = yesterdaySalesSum; - this.previousDaySales = previousDaySales; - } -} +public record OrderSalesSumDetail( + Long storeId, + Integer todaySalesSum, + Integer yesterdaySalesSum, + Integer previousDaySales +) {}nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/dto/TopSalesStoresDetail.java (1)
16-24: 명시적 생성자와 @builder 중복 사용
OrderSalesSumDetail과 동일한 이슈로,@Builder와 명시적 생성자를 동시에 사용하고 있습니다. 일관성을 위해@AllArgsConstructor를 사용하는 것을 권장합니다.+@AllArgsConstructor @Getter @Builder public class TopSalesStoresDetail { - public TopSalesStoresDetail(Long storeId, String storeName, Long departmentId, String departmentName, Integer totalSales, - Long storeRank) { - this.storeId = storeId; - this.storeName = storeName; - this.departmentId = departmentId; - this.departmentName = departmentName; - this.totalSales = totalSales; - this.storeRank = storeRank; - }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (22)
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/controller/OrderController.java(2 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java(2 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/controller/StorePaymentController.java(2 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentCreateRequest.java(1 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentCreateResponse.java(1 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentReadDto.java(1 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentUpdateRequest.java(1 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentService.java(1 hunks)nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentServiceImpl.java(1 hunks)nowait-common/src/main/java/com/nowait/common/api/ApiResult.java(1 hunks)nowait-domain/domain-core-rdb/build.gradle(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/config/QueryDslConfig.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/department/entity/Department.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/dto/OrderSalesSumDetail.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/dto/TopSalesStoresDetail.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/repository/OrderCustomRepository.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/repository/OrderCustomRepositoryImpl.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/repository/OrderRepository.java(1 hunks)nowait-domain/domain-redis/build.gradle(1 hunks)nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/config/RedisConfig.java(1 hunks)nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/service/StoreRankCacheService.java(1 hunks)settings.gradle(1 hunks)
🔇 Additional comments (21)
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentServiceImpl.java (1)
1-1: 패키지 네이밍 일관성 개선 👍
CamelCase → 소문자로 통일하여 자바 관례에 맞췄습니다. 모듈 전반의 리팩터링 방향과도 일치합니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/controller/StorePaymentController.java (2)
1-1: 패키지 경로 소문자화 확인
컨트롤러도 동일하게 수정되어 모듈 간 의존성이 깨지지 않습니다.
13-16: import 경로 동기화 OK
DTO·Service 모두 새 패키지로 맞춰져 있습니다. 남은 레거시 import 가 없는지 위 검증 스크립트로 추가 확인을 권장합니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentService.java (1)
1-6: 인터페이스 패키지 정정 완료
서비스 인터페이스도 소문자 패키지로 일관화되어 구현체·컨트롤러와 매칭됩니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentCreateRequest.java (1)
1-2: DTO 패키지 네이밍 통일 완료
엔티티 매핑 로직에 영향 없는 단순 경로 변경이며 문제 없습니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentCreateResponse.java (1)
1-1: DTO 패키지 경로 소문자화 적용 확인
다른 DTO·서비스 파일들과 동일하게 수정되었습니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentReadDto.java (1)
1-1: 패키지명 소문자 통일 완료 및 레거시 경로 없음 확인storePayment → storepayment로 변경이 전 프로젝트에 일관되게 적용되었으며, legacy 경로가 남아있지 않음을 검증 스크립트로 확인했습니다.
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/dto/StorePaymentUpdateRequest.java (1)
1-1: 패키지명 정비 확인 필요
StorePaymentUpdateRequest역시 소문자 패키지로 변경되었습니다. 빌드·IDE 캐시가 간혹 이전 경로를 기억해 충돌을 일으키므로, 프로젝트 클린-리빌드 또는 Gradleclean실행을 권장드립니다.nowait-common/src/main/java/com/nowait/common/api/ApiResult.java (1)
22-22: ApiResult.getError() 사용처 없음 확인 — 안전하게 제거 가능
- 리포지토리 전체에서
.getError()호출을 검색한 결과, 어떠한 참조도 발견되지 않았습니다.nowait-domain/domain-core-rdb/build.gradle (1)
34-39: QueryDSL 설정이 올바르게 추가되었습니다.Jakarta EE 지원 버전과 필요한 어노테이션 프로세서가 모두 포함되어 있어 QueryDSL 기능을 정상적으로 사용할 수 있습니다.
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/repository/OrderRepository.java (1)
12-12: 커스텀 리포지토리 확장이 올바르게 구현되었습니다.Spring Data JPA의 표준 패턴을 따라 OrderCustomRepository를 확장하여 커스텀 쿼리 기능을 추가했습니다.
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/config/QueryDslConfig.java (1)
11-20: QueryDSL 설정이 올바르게 구현되었습니다.표준적인 QueryDSL 설정 패턴을 따라 JPAQueryFactory 빈을 생성하고 EntityManager를 주입받아 사용하고 있습니다.
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/repository/OrderCustomRepository.java (1)
8-13: 잘 설계된 커스텀 리포지토리 인터페이스메서드 이름이 명확하고 적절한 DTO 타입을 반환하는 깔끔한 인터페이스 설계입니다. Spring Data JPA의 커스텀 리포지토리 패턴을 올바르게 따르고 있습니다.
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/dto/TopSalesStoresDetail.java (1)
9-14: 필드 가시성 설정 우수필드들이 적절히
private로 선언되어 있어 캡슐화가 잘 되어 있습니다. 다른 DTO 클래스들과 일관성을 위해 이 패턴을 따르는 것이 좋습니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/controller/OrderController.java (1)
69-82: API 문서화 완성도 우수매출 조회 엔드포인트가 적절한
@Operation어노테이션과@ApiResponse로 잘 문서화되어 있습니다. 보안과 응답 패턴도 일관성 있게 구현되었습니다.nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/order/service/OrderService.java (3)
61-71: 트랜잭션 어노테이션 사용 우수읽기 전용 트랜잭션 어노테이션을 적절히 사용하여 성능 최적화를 고려했습니다.
73-83: 일관된 서비스 구조기존 메서드들과 동일한 패턴을 따르는 일관된 구조로 구현되어 있습니다. 사용자 조회, 권한 검사, 리포지토리 호출의 흐름이 명확합니다.
78-80: 동일한 불필요한 권한 검사 로직
getSaleSumByStoreId메서드와 동일한 불필요한 권한 검사가 반복되고 있습니다.- if (!Role.SUPER_ADMIN.equals(user.getRole()) && !user.getStoreId().equals(storeId)) { - throw new OrderViewUnauthorizedException(); - } + // storeId는 user.getStoreId()에서 가져온 값이므로 추가 검사 불필요Likely an incorrect or invalid review comment.
nowait-domain/domain-redis/build.gradle (1)
1-36: 빌드 설정이 적절합니다.Redis 모듈을 위한 Gradle 빌드 설정이 올바르게 구성되었습니다. 필요한 의존성들이 포함되어 있고, Java 17 설정과 프로젝트 구조가 적절합니다.
settings.gradle (1)
9-9: 새로운 Redis 모듈이 올바르게 추가되었습니다.프로젝트 설정에
nowait-domain:domain-redis모듈이 적절하게 추가되었습니다.nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/config/RedisConfig.java (1)
13-17: Redis 설정 프로퍼티 확인 필요현재 코드베이스에서
spring.data.redis.host및spring.data.redis.port설정이*.yml/*.yaml/*.properties파일에서 검색되지 않습니다.
아래 항목을 직접 확인해주세요.
- application.yml 또는 application.properties 파일에
spring.data.redis.hostspring.data.redis.port
설정이 정의되어 있는지
Comment on lines
+6
to
+9
| import com.nowait.applicationadmin.storepayment.dto.StorePaymentCreateRequest; | ||
| import com.nowait.applicationadmin.storepayment.dto.StorePaymentCreateResponse; | ||
| import com.nowait.applicationadmin.storepayment.dto.StorePaymentReadDto; | ||
| import com.nowait.applicationadmin.storepayment.dto.StorePaymentUpdateRequest; |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
레거시 패키지(storePayment) 참조가 남아 있지 않은지 확인 바랍니다
IDE 리팩터링으로 누락된 클래스가 있을 수 있습니다. 아래 스크립트로 잔존 패키지명을 검색해 보세요.
🏁 Script executed:
#!/bin/bash
# repo 내 남은 'storePayment' 패키지 경로 탐색
rg --fixed-strings 'applicationadmin.storePayment' -nLength of output: 53
추가 검증을 위해 다양한 케이스로 검색을 재시도합니다.
🏁 Script executed:
#!/bin/bash
# 케이스에 관계없이 'storePayment' 문자열 검색
rg -n 'storePayment'
rg -n 'storepayment'
# 파일 및 디렉토리 이름에 'storePayment' 포함 여부 확인
find . -type f -iname '*storePayment*'
find . -type d -iname '*storePayment*'Length of output: 19778
레거시 패키지명(storePayment)이 여전히 남아 있습니다
camel-case 디렉토리/패키지명을 모두 소문자(storepayment)로 통일해주세요.
아래 경로들을 점검 및 수정해야 합니다:
- nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/ (controller, dto, service)
- nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/storePayment/ (entity, exception, repository)
🤖 Prompt for AI Agents
In
nowait-app-admin-api/src/main/java/com/nowait/applicationadmin/storePayment/service/StorePaymentServiceImpl.java
lines 6 to 9, the import statements use the legacy camel-case package name
'storePayment'. Update all package names and directory paths from 'storePayment'
to lowercase 'storepayment' consistently across the project, including
controller, dto, service, entity, exception, and repository packages as
specified, to unify the package naming convention.
Comment on lines
+15
to
+16
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | ||
| @AllArgsConstructor(access = AccessLevel.PRIVATE) |
There was a problem hiding this comment.
🛠️ Refactor suggestion
엔티티 생성을 위한 팩토리 메서드 추가를 고려하세요.
현재 AllArgsConstructor가 private으로 설정되어 있어 외부에서 엔티티를 직접 생성할 수 없습니다. 엔티티 생성을 위한 정적 팩토리 메서드를 추가하는 것을 권장합니다.
@AllArgsConstructor(access = AccessLevel.PRIVATE)
public class Department {
+
+ public static Department create(String name) {
+ return new Department(null, name);
+ }📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | |
| @AllArgsConstructor(access = AccessLevel.PRIVATE) | |
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | |
| @AllArgsConstructor(access = AccessLevel.PRIVATE) | |
| public class Department { | |
| public static Department create(String name) { | |
| return new Department(null, name); | |
| } | |
| // … rest of the entity’s fields and methods … | |
| } |
🤖 Prompt for AI Agents
In
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/department/entity/Department.java
around lines 15 to 16, the class uses a private AllArgsConstructor which
prevents external instantiation. To fix this, add a public static factory method
that internally calls the private constructor to create and return a new
Department instance, enabling controlled and clear entity creation.
Comment on lines
+19
to
+20
| @Id | ||
| private Long Id; |
There was a problem hiding this comment.
ID 필드에 자동 생성 전략과 네이밍 컨벤션을 적용하세요.
@GeneratedValue어노테이션이 없어서 ID 값을 수동으로 설정해야 합니다.- 필드명이 Java 네이밍 컨벤션을 따르지 않습니다 (
Id→id).
다음과 같이 수정하세요:
@Id
-private Long Id;
+@GeneratedValue(strategy = GenerationType.IDENTITY)
+private Long id;📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| @Id | |
| private Long Id; | |
| @Id | |
| @GeneratedValue(strategy = GenerationType.IDENTITY) | |
| private Long id; |
🤖 Prompt for AI Agents
In
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/department/entity/Department.java
at lines 19-20, the ID field lacks the @GeneratedValue annotation for automatic
ID generation and uses incorrect naming convention with 'Id'. Rename the field
to 'id' following Java conventions and add the @GeneratedValue annotation to
enable automatic ID generation.
Comment on lines
+104
to
+119
| Integer userRank = rankMap.get(userStoreId); | ||
|
|
||
| // 3. 상위 5개 주점 조회 (내 주점 포함 여부는 나중에 확인) | ||
| List<TopSalesStoresDetail> result = getTopNStores(allStores, rankMap, 5); | ||
| Set<Long> already = result.stream() | ||
| .map(TopSalesStoresDetail::getStoreId) | ||
| .collect(Collectors.toSet()); | ||
|
|
||
| // 4. 내 주점 추가 (내 주점이 포함되지 않으면 추가) | ||
| TopSalesStoresDetail myStore = getMyStoreDetail(allStores, rankMap, userStoreId); | ||
| if (myStore == null) | ||
| myStore = getZeroSalesStore(userStoreId); | ||
|
|
||
| if (myStore != null && !already.contains(userStoreId)) { | ||
| // 내 주점이 5위 이하로 밀린 경우 | ||
| if (userRank > 5) { |
There was a problem hiding this comment.
userRank가 null일 때 NullPointerException이 발생할 수 있습니다.
매출이 없는 store의 경우 rankMap에 존재하지 않아 userRank가 null이 될 수 있습니다.
다음과 같이 null 체크를 추가하세요:
// 4. 내 주점 추가 (내 주점이 포함되지 않으면 추가)
TopSalesStoresDetail myStore = getMyStoreDetail(allStores, rankMap, userStoreId);
if (myStore == null)
myStore = getZeroSalesStore(userStoreId);
if (myStore != null && !already.contains(userStoreId)) {
// 내 주점이 5위 이하로 밀린 경우
- if (userRank > 5) {
+ if (userRank != null && userRank > 5) {📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| Integer userRank = rankMap.get(userStoreId); | |
| // 3. 상위 5개 주점 조회 (내 주점 포함 여부는 나중에 확인) | |
| List<TopSalesStoresDetail> result = getTopNStores(allStores, rankMap, 5); | |
| Set<Long> already = result.stream() | |
| .map(TopSalesStoresDetail::getStoreId) | |
| .collect(Collectors.toSet()); | |
| // 4. 내 주점 추가 (내 주점이 포함되지 않으면 추가) | |
| TopSalesStoresDetail myStore = getMyStoreDetail(allStores, rankMap, userStoreId); | |
| if (myStore == null) | |
| myStore = getZeroSalesStore(userStoreId); | |
| if (myStore != null && !already.contains(userStoreId)) { | |
| // 내 주점이 5위 이하로 밀린 경우 | |
| if (userRank > 5) { | |
| Integer userRank = rankMap.get(userStoreId); | |
| // 3. 상위 5개 주점 조회 (내 주점 포함 여부는 나중에 확인) | |
| List<TopSalesStoresDetail> result = getTopNStores(allStores, rankMap, 5); | |
| Set<Long> already = result.stream() | |
| .map(TopSalesStoresDetail::getStoreId) | |
| .collect(Collectors.toSet()); | |
| // 4. 내 주점 추가 (내 주점이 포함되지 않으면 추가) | |
| TopSalesStoresDetail myStore = getMyStoreDetail(allStores, rankMap, userStoreId); | |
| if (myStore == null) | |
| myStore = getZeroSalesStore(userStoreId); | |
| if (myStore != null && !already.contains(userStoreId)) { | |
| // 내 주점이 5위 이하로 밀린 경우 | |
| if (userRank != null && userRank > 5) { |
🤖 Prompt for AI Agents
In
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/order/repository/OrderCustomRepositoryImpl.java
around lines 104 to 119, userRank retrieved from rankMap can be null if the
userStoreId has no sales, causing a NullPointerException when compared. Add a
null check for userRank before using it in comparisons or logic, and handle the
null case appropriately to avoid exceptions.
| @Service | ||
| public class StoreRankCacheService { | ||
|
|
||
| private RedisTemplate<String, String> redisTemplate; |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
RedisTemplate 타입 불일치를 확인하세요.
StoreRankCacheService는 RedisTemplate<String, String>을 사용하지만, RedisConfig에서는 RedisTemplate<String, Object>를 제공합니다. 타입 일치를 위해 설정을 수정하거나 별도의 Bean을 정의해야 합니다.
다음 스크립트를 실행하여 RedisConfig의 Bean 정의를 확인하세요:
🏁 Script executed:
#!/bin/bash
# RedisConfig에서 RedisTemplate Bean 정의 확인
ast-grep --pattern 'RedisTemplate<$_, $_> $_() {
$$$
}'Length of output: 59
아래 스크립트를 실행해 RedisConfig.java 파일 위치를 찾고, 해당 파일에서 RedisTemplate Bean 정의를 확인해 주세요.
🏁 Script executed:
#!/bin/bash
# 1. RedisConfig.java 위치 찾기
echo "== RedisConfig.java 파일 위치 =="
fd -e java | grep -i RedisConfig.java
# 2. 찾은 파일에서 RedisTemplate Bean 정의 검색
config_file=$(fd -e java | grep -i RedisConfig.java | head -n1)
if [[ -n "$config_file" ]]; then
echo "== $config_file 내용 중 @Bean 및 RedisTemplate 검색 =="
grep -R "@Bean" -n "$config_file" | grep -C3 "RedisTemplate"
echo "== $config_file 내용 중 Generic 타입으로 RedisTemplate 검색 =="
grep -R "RedisTemplate<" -n "$config_file"
else
echo "RedisConfig.java 파일을 찾을 수 없습니다."
fiLength of output: 1392
RedisTemplate 제네릭 타입 불일치
StoreRankCacheService에서는 RedisTemplate<String, String>을 사용하지만, RedisConfig에서 제공하는 Bean은 RedisTemplate<String, Object>입니다. 이대로 두면 스프링 빈 주입 시 타입 불일치로 예외가 발생할 수 있으니, 아래 중 하나를 적용해 주세요.
- Service 필드 타입을
RedisTemplate<String, Object>로 변경 RedisConfig에RedisTemplate<String, String>전용 Bean(@bean) 정의
검토 파일:
- nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/service/StoreRankCacheService.java
- nowait-domain/domain-redis/src/main/java/com/nowait.domaincoreredis/config/RedisConfig.java
🤖 Prompt for AI Agents
In
nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/service/StoreRankCacheService.java
at line 9, the RedisTemplate field uses RedisTemplate<String, String> but the
RedisConfig provides RedisTemplate<String, Object>, causing a type mismatch on
Spring bean injection. To fix this, either change the field type in
StoreRankCacheService to RedisTemplate<String, Object> or define a new
RedisTemplate<String, String> bean in RedisConfig and inject that instead.
RedisTemplate 의존성 주입이 누락되었습니다.
redisTemplate 필드가 초기화되지 않았습니다. 이는 서비스 사용 시 NullPointerException을 발생시킬 수 있습니다.
+ public StoreRankCacheService(RedisTemplate<String, String> redisTemplate) {
+ this.redisTemplate = redisTemplate;
+ }
+
private RedisTemplate<String, String> redisTemplate;또는 필드 주입을 사용하는 경우:
+ @Autowired
private RedisTemplate<String, String> redisTemplate;📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| private RedisTemplate<String, String> redisTemplate; | |
| public StoreRankCacheService(RedisTemplate<String, String> redisTemplate) { | |
| this.redisTemplate = redisTemplate; | |
| } | |
| private RedisTemplate<String, String> redisTemplate; |
Suggested change
| private RedisTemplate<String, String> redisTemplate; | |
| @Autowired | |
| private RedisTemplate<String, String> redisTemplate; |
🤖 Prompt for AI Agents
In
nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/service/StoreRankCacheService.java
at line 9, the redisTemplate field is declared but not injected, which can cause
NullPointerException when used. Fix this by adding dependency injection for
redisTemplate, either by annotating the field with @Autowired or by adding a
constructor that accepts RedisTemplate<String, String> and assigns it to the
field.
Comment on lines
+25
to
+31
| public RedisTemplate<String, Object> redisTemplate() { | ||
| RedisTemplate<String, Object> redisTemplate = new RedisTemplate<>(); | ||
|
|
||
| redisTemplate.setConnectionFactory(redisConnectionFactory()); | ||
|
|
||
| redisTemplate.setKeySerializer(new StringRedisSerializer()); | ||
| redisTemplate.setValueSerializer(new StringRedisSerializer()); |
There was a problem hiding this comment.
🛠️ Refactor suggestion
RedisTemplate 타입과 시리얼라이저 불일치를 수정하세요.
RedisTemplate<String, Object>로 타입을 선언했지만 모든 시리얼라이저가 StringRedisSerializer로 설정되어 있습니다. 이는 타입 불일치를 야기합니다.
다음 중 하나를 선택하여 수정하세요:
옵션 1: 타입을 String으로 변경 (권장)
@Bean
-public RedisTemplate<String, Object> redisTemplate() {
- RedisTemplate<String, Object> redisTemplate = new RedisTemplate<>();
+public RedisTemplate<String, String> redisTemplate() {
+ RedisTemplate<String, String> redisTemplate = new RedisTemplate<>();옵션 2: Object 타입에 적합한 시리얼라이저 사용
- redisTemplate.setValueSerializer(new StringRedisSerializer());
+ redisTemplate.setValueSerializer(new GenericJackson2JsonRedisSerializer());
- redisTemplate.setHashValueSerializer(new StringRedisSerializer());
+ redisTemplate.setHashValueSerializer(new GenericJackson2JsonRedisSerializer());🤖 Prompt for AI Agents
In
nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/config/RedisConfig.java
around lines 25 to 31, the RedisTemplate is declared as RedisTemplate<String,
Object> but all serializers are set to StringRedisSerializer, causing a type
mismatch. To fix this, either change the RedisTemplate type to
RedisTemplate<String, String> to match the serializers or keep
RedisTemplate<String, Object> and replace the value serializer with one suitable
for Object types, such as GenericJackson2JsonRedisSerializer.
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
작업 요약
Issue Link
#103
문제점 및 어려움
주점 순위 변동 기능 추가 예정
해결 방안
Reference
Summary by CodeRabbit
신규 기능
버그 수정
빌드/환경
기타