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

[REFACTOR] Exception 처리 구조 변경 #95

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

yeseul106
Copy link
Member

📝 Summary

  • Exception을 처리하는 구조를 변경하였습니다.

👩‍💻 Contents

  • 각 Status Code에 맞는 Exception 클래스를 만들어서 해당 Exception이 발생했을 때 객체로 커스텀 해서 던질 수 있도록 구조를 변경하였습니다.
  • 위에처럼 Status Code마다 사용하는 클래스 자체가 다르기 때문에 ErrorStatus에는 Status Code 값이 필요 없을 것 같아서 제외하고 message만 가지도록 변경하였습니다.

📝 Review Note

  • 애플리케이션 단에서 발생하는 Exception을 모두 500 નાસ 로 던지는 것보다는 에러 처리를 각각 케이스에 맞게 처리해서 클라분들에게 정확하고 자세하게 전달해드리는 것이 중요하다고 생각해서 Advice에서 냅다 Exception 에러를 잡지 않도록 했습니다. 이에 따른 꼼꼼한 에러 처리가 필요할 것 같습니다 !

📣 Related Issue

📬 Reference

Copy link
Contributor

@unanchoi unanchoi left a comment

Choose a reason for hiding this comment

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

수고하셨어요 ~!

Copy link
Member

@Seokyeong237 Seokyeong237 left a comment

Choose a reason for hiding this comment

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

고생하셨어용!!

@yeseul106 yeseul106 merged commit a57039f into develop Aug 8, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] Exception 구조 변경
3 participants