feat(Reservation): 예약 응답데이터 개선 #129
Merged
Conversation
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
- 병준님 요청 사항 기반 항목 추가 - DepartmentRepository 생성 - 내 모든 대기열 조회 로직 구현
|
""" Walkthrough이 변경사항은 사용자가 신청한 모든 대기열 목록을 조회하는 새로운 API 엔드포인트와 DTO를 추가하고, 이를 지원하기 위한 서비스 및 레포지토리 계층의 메서드와 JPA 리포지토리 인터페이스를 도입합니다. 일부 import 정렬 및 기존 엔드포인트 설명도 소폭 수정되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant ReservationController
participant ReservationService
participant StoreRepository
participant WaitingRedisRepository
participant DepartmentRepository
Client->>ReservationController: GET /my/waitings (with 인증정보)
ReservationController->>ReservationService: getAllMyWaitings(사용자)
ReservationService->>StoreRepository: findAllActiveStoreIds()
loop 각 StoreId
ReservationService->>WaitingRedisRepository: getRank(storeId, userId)
alt 사용자가 대기중인 경우
ReservationService->>WaitingRedisRepository: getWaitingTimestamp(storeId, userId)
ReservationService->>DepartmentRepository: findById(departmentId)
ReservationService-->>ReservationService: MyWaitingQueueDto 생성
end
end
ReservationService-->>ReservationController: List<MyWaitingQueueDto>
ReservationController-->>Client: 응답 반환
Estimated code review effort3 (90–240 minutes) Possibly related PRs
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (3)
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/department/repository/DepartmentRepository.java (1)
7-7: import 문을 사용하여 코드 가독성을 개선하세요.제네릭 타입 파라미터에서 완전한 패키지 경로를 사용하는 대신 import 문을 추가하는 것이 좋습니다.
+import com.nowait.domaincorerdb.department.entity.Department; + @Repository -public interface DepartmentRepository extends JpaRepository<com.nowait.domaincorerdb.department.entity.Department, Long> { +public interface DepartmentRepository extends JpaRepository<Department, Long> {nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/dto/MyWaitingQueueDto.java (1)
17-36: 필드 정의는 적절하나 일부 개선사항이 있습니다.전반적으로 필요한 정보들이 잘 포함되어 있습니다. 다음 사항들을 고려해보세요:
status필드가 String으로 되어 있는데, enum 타입 사용을 권장합니다teamsAhead와rank가 중복된 정보일 수 있습니다 (rank = teamsAhead + 1)- @Schema(description = "대기 상태", example = "WAITING") - private String status; + @Schema(description = "대기 상태", example = "WAITING") + private WaitingStatus status;별도의 WaitingStatus enum 생성을 고려해보세요.
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java (1)
129-129: 하드코딩된 상태값 개선이 필요합니다.상태값이 하드코딩되어 있고 TODO 주석이 남아있습니다. enum을 활용한 상태 관리를 구현해야 합니다.
WaitingStatus enum을 생성하고 실제 대기 상태를 반영하는 로직 구현을 도와드릴까요?
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
nowait-app-user-api/src/main/java/com/nowait/applicationuser/order/dto/OrderCreateResponseDto.java(1 hunks)nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/controller/ReservationController.java(4 hunks)nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/dto/MyWaitingQueueDto.java(1 hunks)nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/repository/WaitingRedisRepository.java(1 hunks)nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java(3 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/department/repository/DepartmentRepository.java(1 hunks)nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/store/repository/StoreRepository.java(2 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (1)
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/repository/WaitingRedisRepository.java (1)
nowait-domain/domain-redis/src/main/java/com/nowait/domaincoreredis/common/util/RedisKeyUtils.java (1)
RedisKeyUtils(5-43)
🔇 Additional comments (9)
nowait-app-user-api/src/main/java/com/nowait/applicationuser/order/dto/OrderCreateResponseDto.java (1)
7-7: import 정렬 개선이 좋습니다.import 문의 재정렬로 코드 정리가 잘 되었습니다.
nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/store/repository/StoreRepository.java (1)
26-28: 활성 매장 ID 조회 메서드가 잘 구현되었습니다.JPQL 쿼리가 올바르게 작성되었고, 필요한 조건들(isActive=true, deleted=false)이 적절히 적용되었습니다. 성능을 위해 ID만 선택하는 것도 좋은 접근입니다.
TODO 주석에 언급된 QueryDSL 전환 계획도 향후 타입 안전성과 유지보수성 측면에서 좋은 방향입니다.
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/repository/WaitingRedisRepository.java (1)
51-55: 대기 시간 조회 메서드가 올바르게 구현되었습니다.Redis sorted set에서 사용자의 대기 등록 시간을 조회하는 로직이 적절합니다:
- 기존 패턴을 따라 RedisKeyUtils를 사용한 키 생성
- null 처리로 사용자가 대기열에 없는 경우 적절히 핸들링
- Double에서 Long으로의 안전한 타입 변환
기존 메서드들과 일관된 구조로 잘 작성되었습니다.
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/controller/ReservationController.java (3)
73-73: API 문서 설명 개선이 좋습니다.기존 엔드포인트의 설명을 더 명확하게 수정한 것이 좋습니다.
105-111: 서비스 계층 getAllMyWaitings 예외 처리 확인 요청서비스 메서드 getAllMyWaitings 내에서 다음 지점을 검토해주세요:
- 파일: nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java
throw new IllegalArgumentException("Waiting not found");사용 위치- 빈 대기목록 조회 시 예외 대신 빈 리스트 반환 또는 404 응답 처리 로직이 필요한지 결정
위 사항을 반영하여 적절한 예외 타입(예: 커스텀 NotFoundException) 혹은 컨트롤러/글로벌 예외 핸들러 매핑이 정상 동작하는지 검증해 주세요.
16-16: MyWaitingQueueDto 및 서비스 메서드 정의 확인 완료
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/dto/MyWaitingQueueDto.java클래스 존재 및 필드/어노테이션 모두 적절nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java에
public List<MyWaitingQueueDto> getAllMyWaitings(CustomOAuth2User)메서드가 올바르게 정의되어 있고 반환 타입이List<MyWaitingQueueDto>로 일치추가 수정이나 검증은 필요 없습니다.
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/dto/MyWaitingQueueDto.java (1)
11-16: DTO 클래스 구조가 적절하게 설계되었습니다.Lombok 어노테이션과 Swagger 문서화가 잘 적용되어 있고, 대기열 정보를 포괄적으로 담을 수 있는 구조입니다.
nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java (2)
3-6: import 문이 적절히 추가되었습니다.새로운 기능에 필요한 time 관련 클래스들이 올바르게 import되었습니다.
12-19: 새로운 의존성이 적절히 추가되었습니다.
MyWaitingQueueDto와DepartmentRepository의존성이 올바르게 추가되었습니다.Also applies to: 42-42
...ser-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java
Show resolved
Hide resolved
...ser-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java
Outdated
Show resolved
Hide resolved
- 성능 개선 로직 추가
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
#128
문제점 및 어려움
해결 방안
Reference
Summary by CodeRabbit
신규 기능
버그 수정
기타