view 분리에 대해 질문이 있습니다!

Question

public class MyView { private String viewPath; public MyView(String viewPath) { this.viewPath = viewPath; } public void render(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { RequestDispatcher dispatcher = request.getRequestDispatcher(viewPath); dispatcher.forward(request, response); } } 꼭 viewPath를 인스턴스로 필드로 설정해서 객체로 만든 이유가 있나요? FrontController 내부에 요청,응답 객체와 view의 주소인 String 타입의 파라미터를 받는 메서드로 만들면 안 되나요?

David · Answer

안녕하세요. iimjjo님, 공식 서포터즈 David입니다. 프론트 컨트롤러에서 모든 걸 구현해도 되는데 컨트롤러, 뷰에 책임을 부여하고 분리한 이유를 생각해 보시면 좋을 것 같습니다. 컨트롤러와 뷰에 책임을 부여하고, 프론트 컨트롤러에서 뷰 렌더링(각 컨트롤러에서 진행해도 되지만 중복 제거를 위해)을 하게 되면, 뷰는 본인이 렌더링 해야하는 파일 경로를 가지는 게 자연스럽습니다:) 감사합니다.

뎅뚠 · Answer

MemberFormControllerV1 을 예로 들어보겠습니다. process(..) 는 다음과 같이 두가지 부분으로 나뉘어 질 수 있습니다. 왜 그럴까요? 위의 빨간 상자는 계속 변할 수 있기 때문입니다. 반면 아래 상자는 변함이 없는 로직입니다. 따라서, 이 두 박스를 분리해줍니다. 다른 Controller들을 비교해보면 이 process의 2줄 로직이 모두 동일하다는 걸 알 수 있습니다. 그렇다면 이걸 그냥 효율적으로 MyView 라는 객체를 만들어서 한군데에서 처리해주겠다는 걸 적용하려고 하니까 고쳐야할 점이 보입니다. 이 getRequestDispatcher(..) 가 viewPath를 사용한다는 점입니다. 따라서, 이 viewPath를 MyView에 넘겨줘야합니다. 어떤 값을 넘겨줄려고 할 때 스프링 기본편에서 배웠던 Dependency Injcection 을 사용하면 MyView생성자가 viewPath를 외부에서 주입받는 방향으로 코딩을 하게 됩니다. 결국, MyView는 viewPath의 구체적인 값을 몰라도 되게 됩니다. 그렇다면 질문주신 " String 타입의 파라미터를 받는 메서드로 만들면 안 되나요?" 에 대한 답변은 만들 수는 있다입니다. 하지만 파라미터의 갯수를 늘리는 방식으로 리팩터링을 하면 단점이 있습니다. 해당 메서드를 호출하는 쪽에서도 알맞는 값을 넣어줘야 한다는 단점이 있습니다. 반복되는 행위를 하나의 클래스를 만들어서 그 클래스가 반복되는 행위를 하게끔 하는 것이 객체지향적으로도 효율면에서도 더 좋기 때문에 객체로 따로 뺀것입니다.