예외처리를 어떻게 하는게 가장 베스트일까요?

Question

일단 실무 경험이 없는 취준생임을 밝힙니다.북마크 기능을 구현하려고 합니다. RequestDTO 을 사용해서 RequestBody 를 통해 JSON 으로 데이터를 받을 생각인데, 예외 처리를 어떻게 해주어야 가장 깔끔할까에 대해 고민이 생겨서 이렇게 질문을 남깁니다.잘못 이해하고 있는 내용이 있을 수 있습니다. 거침없는 지적과 훈수 부탁드립니다.첫번째 : 도메인 계층에서 처리한다. ( Controller -> Service -> Domain )VoteEvaluation 을 생성할 때 매개변수로 투표 타입을 단순하게 String 타입으로 받아서 VoteEvaluationType 정적 팩토리 메서드를 통해 생성할 때 검증하는 방식private VoteEvaluation(final Vote vote, final Member member, final String voteEvaluationType) { this.vote &#61; vote; this.evaluator &#61; member; this.voteEvaluationType &#61; VoteEvaluationType.of(voteEvaluationType); } public static VoteEvaluation of(final Vote vote, final Member member, final String voteEvaluationType) { return new VoteEvaluation(vote, member, voteEvaluationType); }장점- 깔끔하다.- 안정적이다.단점Application Layer 를 거쳐서 들어오기 때문에 DB Connection 을 획득한다.정리투표 타입이 다르면 애초에 DB Connection 을 획득하기 전에 예외처리를 해주는 것이 한정된 DB Connection 자원을 효율적으로 사용하여 불필요한 병목을 줄이는데 좋을 것이다.두번째 : 표현계층에서 처리한다. (by Bean Validation &#43; &#64;JsonCreator )public enum VoteEvaluationType { LIKE, DISLIKE; &#64;JsonCreator public static VoteEvaluationType from(final String voteEvaluationType){ return Arrays.stream(VoteEvaluationType.values()) .filter(it -> it.name().equalsIgnoreCase(voteEvaluationType)) .findFirst() .orElse(null); } } &#64;Getter &#64;NoArgsConstructor(access &#61; AccessLevel.PRIVATE) public class VoteEvaluateRequest { &#64;NotNull(message &#61; &#34;투표 타입을 입력해주세요(like, disLike)&#34;) private VoteEvaluationType voteEvaluationType; public VoteEvaluateRequest(final String voteEvaluationType){ this.voteEvaluationType &#61; VoteEvaluationType.from(voteEvaluationType); } }String 타입으로 요청을 받고 Enum 타입으로 pasing을 할 때, 미리 정의해놓은 타입이 아닌 경우 null로 반환하게 하여 Bean Validation 의 &#64;NotNull 을 통해 BindException 을 발생시키고 ControllerAdvice의 ExceptionHandler 로 잡아 예외처리를 해주는 방식장점- 불필요한 DB Connection 을 획득하지 않는다.단점- 불안전하다.정리이 구조는 도메인 계층이 표현계층을 의존하는 꼴이다. 만약 ReqeustDTO 를 누군가 고치게 되고 Bean Validation 의 &#64;NotNull 을 제거하게 된다면...? 그대로 null 값이 DB 에 영속화될 것이다. 불안전하다. 물론 예외 처리를 한번더하면 되지만 이러면 나중에 유지보수할 때 한 지점이 아닌 두 지점을 모두 수정해야 원활하게 동작하기에 피곤해진다.세번째 : 표현계층에서 처리한다. (by Controller &#43; VO의 정적 팩토리 메서드)Request DTO는 단순하게 String 타입으로 받고 파싱해서, Controller 에서 VO의 정적 팩토리 메서드를 통해 생성하게 하는 방식&#64;Getter &#64;NoArgsConstructor(access &#61; AccessLevel.PRIVATE) public class VoteEvaluateRequest { &#64;NotNull(message &#61; &#34;투표 타입을 입력해주세요&#34;) private String voteEvaluationType; public VoteEvaluateRequest(final String voteEvaluationType){ this.voteEvaluationType &#61; voteEvaluationType; } } &#64;PostMapping(&#34;/{vote-id}/evaluations&#34;) &#64;ResponseStatus(HttpStatus.CREATED) &#64;Login public void evaluateVote(&#64;LoginMember MemberPayLoad memberPayLoad, &#64;PathVariable(name &#61; &#34;vote-id&#34;) final Long voteId, &#64;RequestBody &#64;Valid final VoteEvaluateRequest voteEvaluateRequest){ final VoteEvaluationType voteEvaluationType &#61; VoteEvaluationType.from(voteEvaluateRequest.getVoteEvaluationType()); voteService.evaluate(memberPayLoad, voteId, voteEvaluationType); } public enum VoteEvaluationType { LIKE, DISLIKE; public static VoteEvaluationType from(final String voteEvaluationType){ return Arrays.stream(VoteEvaluationType.values()) .filter(it -> it.name().equalsIgnoreCase(voteEvaluationType)) .findAny() .orElseThrow(() -> new VoteException(VoteExceptionType.INVALID_VOTE_EVALUATION_TYPE)); } }&#96;&#96;&#96;장점- 불필요한 DB Connection 을 획득하지 않는다.- 깔끔하다.단점- 아직까지는 모르겠음

집킨 · Answer

애초에 DTO의 필드로 Enum을 지정해주면 클라이언트에서 보낸 JSON을 DTO로 매핑할때 예외가 발생합니다

드리데이 · Answer

mybatis 이용하는경우 TypeHandler를 통해 DTO 자동매핑 지원하게 커스텀 가능한점 참고하시면 될거구요.DB커넥션에 너무 편향되는듯 한 느낌인데사용하시는부분은 @Transactional (트랜젝션 적용)했다는 가정하에 성능상 index만 적절하면 잦은 호출도 문제 없어보입니다.(메일기준 색상별표 ) 그리고 예외상황은 보통 좁을 수록 좋다했습니다.ex service dao등 단에서 Exception을 발생 시키거나 발생하는(수동 및 타의) 경우엔 Controller에서 최종 받아서 Exception에 맞게 처리하는게 보편적인 spring exception핸들링 방법입니다.간혹 무분별하게 try catch도배로인해 메시지도 안찍히는 웃픈 상황이 되는데되도록이면 종단메시지처리는 spring exceptionhandlerresolver를 이용하여 기본적인 인지하고있는 exception 대응,그 외에는 Controller단에서 취합하여 최종 메시지 처리하시는 걸 추천드립니다.

yeori · Answer

장점, 단점에서 db connection을 자주 언급하시는 점을 감안했을때,코드의 표현력을 개선하는 노력과 성능을 개선하는 노력을 분리해서 시도하시면 좋습니다.코드의 표현력, 디자인, 설계 등을 고민할때는 보통 성능까지 고려하지 않습니다(일반적으로).잘 설계된 구현물은 성능 문제도 크게 발생하지 않음성능이 문제라면 그때가서 다른 해결책이 있음미리 성능, 효율성을 신경쓰지 않음

초무쿤 · Answer

계속 언급되는”불필요한 커넥션을 획득한다“가 카넥션풀을 사용하는 상황에서의미가 있을까용..(커넥션은 이미 맺아져사 풀에서 괸리가 되고있는데..)게다가 조회의 경우 &#64;Transactional 로 트랜잭션을 명시적으로 시작하지 않는다면 실제 SQL이 실행될때 커넥션풀에서 borrow 할가 같읃데요

예외처리를 어떻게 하는게 가장 베스트일까요?

4개의 답변

기술 목록