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

무분별한 커스텀 예외 생성보다는 표준 예외를 적극적으로 활용한다. #435

Open
3 tasks
woo-chang opened this issue Sep 28, 2022 · 0 comments
Labels
🤝 FE needs 해당 이슈를 해결하기 위해 프론트엔드 팀과 협업해야 함 👀 good first issue Good for newcomers

Comments

@woo-chang
Copy link
Collaborator

사전 수행 issue

없음.

이슈 내용

Effective Java에서 표준 예외 사용을 적극적으로 권장하고 있습니다. 대표적인 표준 예외라 함은 아래와 같이 존재합니다.

  • IllegalArgumentException
  • IllegalStateException
  • NullPointerException
  • IndexOutOfBoundsException

표준 예외를 사용하게 됨으로 얻게 되는 장점으로는 API를 익히고 사용하기 쉬워지며 예외 클래스 수가 줄어들기에 메모리 사용량과 클래스 적재 비용이 줄어들게 됩니다.

현재 저희 프로젝트에서는 예외 상황에 대해 표준 예외를 활용하기보다는 네이밍을 통한 커스텀 예외 클래스를 무수히 많이 생성한 것으로 알고 있는데 표준 예외로 변경할 수 있다면 변경하여 예외 클래스 수를 줄이는 것이 좋아보입니다.

또한 앞으로 개발하는 기능에 대해서도 예외 상황 처리 시 표준 예외 문서를 먼저 확인 후 사용할 수 있는 예외가 존재한다면 해당 예외를 사용해서 예외를 처리하는 것이 좋다고 생각합니다.

추가로 좋은 에러 메시지에 관한 내용이 존재하여 링크 추가하였습니다.
https://toss.tech/article/how-to-write-error-message

이슈 수행 사항

  • 기존에 생성한 커스텀 예외 클래스를 표준 예외로 변경할 수 있는지 확인 후 변경
  • 변경된 예외로 인해 테스트가 깨지지 않는지 확인
  • 프론트 팀에게 변경된 예외 전달하기
@woo-chang woo-chang added 👀 good first issue Good for newcomers 🤝 FE needs 해당 이슈를 해결하기 위해 프론트엔드 팀과 협업해야 함 labels Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤝 FE needs 해당 이슈를 해결하기 위해 프론트엔드 팀과 협업해야 함 👀 good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant