Skip to content
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

🐛 [STMT-146] 표준 예외가 아닌 커스텀 예외를 사용하도록 변경 #99

Merged
merged 1 commit into from
Mar 16, 2024

Conversation

zxcv9203
Copy link
Collaborator

💁 해결 하려는 문제를 적어주세요

  • 표준 예외를 사용하여 핸들링 되지 않았던 예외들을 커스텀 예외를 사용하도록 변경 필요

🤔 어떤 방식으로 해결했는지 적어주세요

  • 커스텀 예외를 이용하여 예외를 던지도록 변경했습니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

  • 특정 데이터가 존재하지 않는 예외에 대해 계층 구조를 만들었습니다. (BusinessException -> NotExistsException -> MemberNotExistsException)
  • 도메인 관련 예외들은 domain 패키지에 exception이라는 하위 패키지를 만들었는데 어떻게 생각하시는지 궁금합니다.

🧑‍🏫 이해를 위해 필요한 자료가 있다면 첨부해주세요

  • 다음은 계층화된 커스텀 Exception의 예시입니다.
image

@zxcv9203 zxcv9203 added the D-1 1일 전 까지 리뷰해주세요 label Mar 16, 2024
@zxcv9203 zxcv9203 requested a review from 05AM as a code owner March 16, 2024 14:53
Copy link
Collaborator

@05AM 05AM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

계층화된 custom exception 도입 좋습니다! 고생하셨습니다

@zxcv9203 zxcv9203 merged commit 2813dc1 into dev Mar 16, 2024
1 check passed
@zxcv9203 zxcv9203 deleted the STMT-146-error-handling branch March 16, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-1 1일 전 까지 리뷰해주세요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants