좋은 Controller와 Service의 코드는 어떻게 되어야하나요?

Question

안녕하세요. 현재 만 2년차 개발자로 재직하고 있습니다.항상 생각해왔던건데 Controller에서는 요청에 대한 값 검증, 리턴값 검증 등을 하고,실제 로직은 Service에서 도는 걸로 알고있습니다.근데 이것도 Controller에서 Service 메소드 한개만 호출하고 Service에서 자신의 메소드를 호출해서 결과값만 넘기는게 맞는건지아니면 Controller에서 여러 Service 메소드를 호출해서 해당 메소드의 리턴값으로 다른 서비스를 호출해야 하는건지 모르겠습니다.예시를 들면 아래와 같겠죠.1번 예시&#64;RequiredConstructor &#64;Controller public class TestController { private final TestService testService; &#64;RequestMapping(value&#61;&#34;/Test.do&#34;, method&#61;RequestMethod.GET) public String test() { return testService.test(); } } &#64;Service public class TestService() { public String test() { return &#34;SUCCESS&#34;; } }2번 예시&#64;RequiredConstructor &#64;Controller public class TestController { private final TestService testService; &#64;RequestMapping(value&#61;&#34;/Test.do&#34;, method&#61;RequestMethod.GET) public String test() { String result1 &#61; testService.test1(); String result2 &#61; testService.test2(); return testService.test3(result1, result2) } } &#64;Service public class TestService() { public String test1() { return &#34;TESTING1&#34;; } public String test2() { return &#34;TESTING2&#34;; } public String test3(String result1, String result2) { return &#34;SUCCESS&#34;; } }1번처럼 하자니 서비스 메소드끼리 호출하다보니 서로가 서로에게 의존하게 되어 메소드 하나만 바꿔도 다른 메소드를 다 바꿔줘야 할거같고,2번처럼 하자니 결과를 받아서 그 결과로 메소드를 호출하는것도 결국 비즈니스 로직 아닌가? 싶기도 하구요.뭐가 더 나을까요?

-하니아빠- · Accepted Answer

처리해야하는 task가 어떤지에 대해서 달라질 수 있을꺼 같네요.트랜잭션 관리가 필요한 경우는 1번이어야 하고, 분산된 조회라고 하면 1, 2번을 해도 상관없겠죠--서로가 서로에게 의존하게 되어 메소드 하나만 바꿔도 다른 메소드를 다 바꿔줘야 할거같고,--> 의존성이 문제라면 줄일 수 있는 방법으로 로직을 구성해야겠죠 (파라미터 타입 선택에서 부터 적절한 오버로딩으로 분배 등)

아이티소년 · Answer

1번

장독깨기 · Answer

서비스에 트랜잭션이 적용되어 있다면 어케 해야 할지 생각을 해보는게 좋겠습니다.

시인들 · Answer

음..네;; 뭐가 좋다고는 말씀 드리기 어렵네연..참;; 1번 방법도 데이터 검증을 위행 쓸 수도 있는 방법이공2번 또한 쓰기도 하니..답변하기 어렵네연;; 물론 검증에 대한 다른 방법동 있기동 하궁;; 답이 있을려나연 ㅎㅎ;;나중에라동 정답알면 저에겡 알려줘영 ㅎㅎ;;

살다보니개발자 · Answer

1번? 2번이 더 무거울것 같아요 뭔가.. 동작하는데 있어서 ㅋㅋㅋ

극한초보 · Answer

서비스를 두개이상 호출할려면별도의 레이어(facade)를 둬서 관리하는게 좋을거 같네요.그편이 소스보기 변할꺼 같습니다.

노로스 · Answer

보통 Controller - Service 를 1:1로 매핑하고 Service - Mapper를 1:N으로 사용합니다. Service에서 트랜잭션을 관리 하구요.

좋은 Controller와 Service의 코드는 어떻게 되어야하나요?

7개의 답변

#cleancode 목록