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