-
Notifications
You must be signed in to change notification settings - Fork 0
feat: store, menu custom exception 추가 #39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Caution Review failedThe pull request is closed. Walkthrough이 변경사항은 메뉴와 매장 서비스 전반에 걸쳐 입력 파라미터와 엔티티 존재 여부에 대한 명확한 검증 및 도메인별 커스텀 예외를 도입합니다. 기존의 일반 예외를 도메인 특화 예외로 교체하고, 입력값이 null 또는 비어있을 때 명확한 예외를 발생시키도록 로직을 강화했습니다. 또한, 컴포넌트 스캔 범위가 확장되고, 관련 에러 메시지 상수가 추가되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Service
participant Repository
Client->>Service: 메뉴/매장 관련 메서드 호출(파라미터 포함)
alt 파라미터 null/empty
Service-->>Client: 도메인별 ParamEmptyException throw
else 정상 파라미터
Service->>Repository: 엔티티 조회
alt 엔티티 없음
Service-->>Client: 도메인별 NotFoundException throw
else 엔티티 존재
Service-->>Client: 정상 응답 반환
end
end
Possibly related PRs
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (20)
✨ 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 (
|
작업 요약
store, menu custom exception 추가
Issue Link
#36
문제점 및 어려움
해결 방안
Reference
Summary by CodeRabbit
신규 기능
버그 수정
리팩터링
문서화