인프런 커뮤니티 질문&답변

안녕하세요.

MyView 클래스에서 render() 메서드 재활용에 대한 질문이 있습니다.

public class MyView {

    ...

    // #1 render 
    public void render(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        RequestDispatcher dispatcher = request.getRequestDispatcher(viewPath);
        dispatcher.forward(request, response);
    }

    // #2 render 
    public void render(Map<String, Object> model, HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        modelToRequestAttribute(model, request);
        RequestDispatcher dispatcher = request.getRequestDispatcher(viewPath);
        dispatcher.forward(request, response);
    }

    private void modelToRequestAttribute(Map<String, Object> model, HttpServletRequest request) {
        model.forEach(request::setAttribute);
    }
}

영한님께서는 #2 render() 를 작성하실 때, #1의 코드를 복사해서 넣어주면 된다고 하셨는데요.

public class MyView {

    ...

    // #1 render 
    public void render(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        RequestDispatcher dispatcher = request.getRequestDispatcher(viewPath);
        dispatcher.forward(request, response);
    }

    // #2 render 
    public void render(Map<String, Object> model, HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        modelToRequestAttribute(model, request);
        render(request, response);
    }

    private void modelToRequestAttribute(Map<String, Object> model, HttpServletRequest request) {
        model.forEach(request::setAttribute);
    }
}

위와 같이 #1 render() 를 호출하여 코드 중복을 제거하는 방법을 사용해도 괜찮을까요?

영한님께서 학습용이기에 간단한 코드를 작성하신건지,

아니면 다른 이유로 사용하시지 않은 이유가 있는지 궁금합니다!