Skip to content

[5주차] 리펙토링#100

Closed
rmsxo200 wants to merge 37 commits intohanghae-skillup:mainfrom
rmsxo200:week5_KimKeunTae
Closed

[5주차] 리펙토링#100
rmsxo200 wants to merge 37 commits intohanghae-skillup:mainfrom
rmsxo200:week5_KimKeunTae

Conversation

@rmsxo200
Copy link

@rmsxo200 rmsxo200 commented Feb 9, 2025

작업 내용

커스텀 오류 코드 추가
커스텀 오류 추가

이번 주차에서 고민되었던 지점이나, 어려웠던 점을 알려 주세요.

  • 예외처리 시 계층간 오류를 전달하는 방법이 많이 고민되었습니다.
  • Custom Error Code를 사용하기 위해 CustomException을 추가하였습니다.
  • 하지만 도메인 계층에는 의존성 문제로 CustomException을 사용하기 어려웠습니다.
  • 그래서 모든 apllication 계층에서 한번 더 예외 처리를 하게 끔 하였습니다.
  • 문제는 해결된 듯 했지만 코드가 너무 복잡해진 것 같습니다.

리뷰 포인트

계층간 오류처리 방법에 문제가 있는지 궁금합니다!

기타 질문

계층간 오류를 전달을 효율적으로 할 수 있는 방법이 있는지 궁금합니다!

그동안 고생 많으셨습니다!
이번 과정을 통해 많이 성정하고 배워간 것 같습니다!
감사합니다!!

kim keun tae added 30 commits January 10, 2025 06:09
@rmsxo200 rmsxo200 closed this Feb 9, 2025
@rmsxo200 rmsxo200 deleted the week5_KimKeunTae branch February 9, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant