[MVC 구현하기 - 3단계] 엔초(권예진) 미션 제출합니다. #575
Conversation
| public void init() { | ||
| handlerMappingRegistry.addHandlerMapping(new ManualHandlerMapping()); | ||
| handlerMappingRegistry.addHandlerMapping(new AnnotationHandlerMapping(Application.class.getPackageName() + ".*")); | ||
| handlerAdapterRegistry.addHandlerAdapter(new ControllerHandlerAdapter()); | ||
| handlerAdapterRegistry.addHandlerAdapter(new HandlerExecutionHandlerAdapter()); | ||
| handlerExecutor = new HandlerExecutor(handlerAdapterRegistry); | ||
| } | ||
|
|
||
| public void addHandlerMapping(final HandlerMapping handlerMapping) { | ||
| handlerMappingRegistry.addHandlerMapping(handlerMapping); | ||
| } | ||
|
|
||
| public void addHandlerAdapter(final HandlerAdapter handlerAdapter) { | ||
| handlerAdapterRegistry.addHandlerAdapter(handlerAdapter); | ||
| } |
There was a problem hiding this comment.
현재 이 부분을 보면 HandlerExecutor의 경우 DispatcherServlet의 init()메소드에서 초기화가 되는데
HandlerMapping과 HandlerAdapter의 경우에는 DispatcherServletInitializer에서 값을 추가하는 방식으로 구현을 해주셨어요. 이 부분을 분리해서 값을 추가해주신 이유가 있을까요?
혹시 AnnotationHandler의 basePackage 설정을 위함일까요?
엔초의 생각흐름이 궁금합니다!
There was a problem hiding this comment.
addHandlerMapping과 addHandlerAdapter는 DispatcherServletInitializer의 onStartUp()에서 직접 호출하고 있고, DispatcherServlet의init()은 onStartUp()이 끝난 후에 호출이 되고 있습니다.
HandlerExecutor는 HandlerAdapterRegistry를 사용하기 때문에 HandlerAdapterRegistry에 대한 설정이 끝난 이후에 사용되어야 한다고 생각했습니다. init() 메서드가 HandlerAdapterRegistry를 설정해주는 onStartUp()이 끝난 후에 호출되기 때문에 이렇게 분리하면 더 명확하다고 생각했습니다.
| final Optional<Object> nullableHandler = handlerMappingRegistry.getHandler(request); | ||
|
|
||
| if (nullableHandler.isEmpty()) { | ||
| response.setStatus(HttpServletResponse.SC_NOT_FOUND); | ||
| render(new ModelAndView(new JspView("404.jsp")), request, response); | ||
| return; | ||
| } | ||
|
|
||
| final Object handler = nullableHandler.get(); | ||
| try { | ||
| final HandlerAdapter handlerAdapter = handlerAdapterRegistry.getHandlerAdapter(handler); | ||
| final ModelAndView modelAndView = handlerAdapter.handle(request, response, handler); | ||
| final ModelAndView modelAndView = handlerExecutor.handle(request, response, handler); | ||
| render(modelAndView, request, response); | ||
| } catch (Exception e) { | ||
| throw new RuntimeException(e); |
There was a problem hiding this comment.
이번에 HandlerExecutor를 이번에 도입해주셨는데 HandlerExecutor 내부에 HanderMappingResitry를 가지고 있고 handler를 찾아와서 실행시켜주는 것 같습니다. 하지만 현재 코드를 보면 404.jsp를 나타낼 때에는 handlerMappingResitry에서 직접꺼내와서 작업을 진행하는 것도 작업이 있는데 이 부분도 HandlerExecutor 내부에서 처리하는 방식으로 수정하면 좋을 것 같습니다!
There was a problem hiding this comment.
handleNotFound 메서드를 추가해서 404 ModelAndView를 반환하도록 수정했습니다!
| final String body = objectMapper.writeValueAsString(model); | ||
| response.setContentType(MediaType.APPLICATION_JSON_UTF8_VALUE); | ||
| response.getWriter().write(body); |
There was a problem hiding this comment.
model에 데이터가 1개면 값을 그대로 반환하고 2개 이상이면 Map 형태 그대로 JSON으로 변환해서 반환한다.
힌트에 위와 같이 요구조건이 있는데 그 부분이 지켜지지 않은 것 같습니다. 물론 결과는 지금과 같게 나오기는 합니다!
그래도 수정 한번 부탁드립니다!
| final String requestURI = request.getRequestURI(); | ||
| log.debug("Method : {}, Request URI : {}", request.getMethod(), requestURI); | ||
|
|
||
| final Optional<Object> nullableHandler = handlerMappingRegistry.getHandler(request); | ||
|
|
||
| if (nullableHandler.isEmpty()) { | ||
| response.setStatus(HttpServletResponse.SC_NOT_FOUND); | ||
| render(new ModelAndView(new JspView("404.jsp")), request, response); |
안녕하세요 아코! 2단계 때 정성스러운 리뷰 정말 감사합니다.
질문 남겨주신 부분에 대해 답변 남겨놨습니다~ 2단계 pr
총 8개의 커밋 중 앞의 4개(범위)는 2단계에서 리뷰해 주신 내용 적용한 것과 간단한 리팩토링(HandlerExecutor 추가 및 DispatcherServletInitializer에서 핸들러 매핑과 핸들러 어댑터를 추가하도록 변경)에 해당하는 부분이고,
뒤에 있는 4개의 커밋(범위)이 3단계 구현에 해당합니다.
이번 리뷰도 잘 부탁드립니다!