컨트롤러, 서비스 관심사 분리 질문이 있습니다.

Question

스프링부트를 공부하는 학생인데 궁금한게 생겨서 질문 드립니다,.반환 값에 따른 http 상태코드 반환하는 로직을 보통 컨트롤러에서 작성하는지, 서비스 계층에서 작성하는지 궁금합니다. 예를 들어, 이메일 중복 검사 api를 설계한다고 한다면 1. 이메일이 중복된다면 ResponseEntity.status(HttpStatus.CONFLICT) 2. 중복 되지 않는다면 ResponseEntity.ok 이런식으로 http 상태코드를 나눠서 전달하려하는데, 컨트롤러에 작성해주는 것이 맞을까요? 컨트롤러에 작성하긴 했는데, 뭔가 if문으로 상태코드를 나누다 보니 컨트롤러가 너무 지저분해지는게 아닌가 싶어서 여쭤봅니다.

캐티 · Answer

일반적으로 db에서 내려받은 데이터를 사용하겠쩌. 이렇다 하더라도 코드가 과도하게 결합되는 경우에 가독성이 떨어질 뿐만이 아니라 비즈니스 로직에서 실수를 하는 상황이 생길 수 있을 것이어요. 사실 머 애초에 이메일 유효성 같은 모델은 하찮아서 실수할 리가 없겠지만 말이어요. 이렇다 하더라도 정교하고 보기 좋은 가독성을 유지하면 일반적으로 좋스빈다. 이케까지 하는 기업이 잘 없다고 전망하빈다.if 문 사용을 자제하고 switch 문을 사용하면 좋스빈다. 이메일 검사 프로시저에서 성공하면 200(OK)를 반환하고, 이미 있는 이메일이면 409(CONFLICT)를 출력해요. switch case는 200 또는 409 기본값은 500 이면 되겠쩌.또 머 서비스 루틴에서 반환된 상태 코드를 그대로 사용할 것이지만 애초에 데이터베이스에 유니크 키를 조사하여 반환 받은 값을 사용하여서 데이터베이스에 프로시저를 만들어서 반환 값을 약속하여 공통적으로 사용하면 적절하빈다.

3CH-O · Answer

※ 개인적인 의견입니다.단순히 이메일이 존재하는지 존재하지 않는지 조회는 응답 안에 존재/비존재 모두가 포함되는 것이니, 이미 존재하더라도 둘 다 ok가 적합하다고 생각합니다.그리고 실제 가입 제출 시도(insert) 시에 중복 이메일이 있어서 등록할 수 없는 경우는 리포지토리 레이어에서 ConstraintViolationException을 터뜨릴 것이니 이 예외는 전역 예외처리기를 통해 처리하면 if문을 쓰지 않아도 응답상태를 제어할 수 있습니다.

마마마22636 · Answer

기슬관련된거는 컨트롤러에서 할일이져 서비스는 비지니스 로직만

LichKing · Answer

HTTP 관련 로직은 컨트롤러에 들어가는게 맞습니다. 코드를 보지않아 정확힌 모르겠지만 컨트롤러가 지저분해지는 문제는 다른 방식으로 해결을 고민해봐야죠

asdkkzk · Answer

답변 감사합니다..!

컨트롤러, 서비스 관심사 분리 질문이 있습니다.

5개의 답변

Q&A 목록