[MVC 구현하기 - 3단계] 호이(이건호) 미션 제출합니다. #604
Conversation
| @@ -23,11 +23,9 @@ public class DispatcherServletInitializer implements WebApplicationInitializer { | |||
| @Override | |||
| public void onStartup(final ServletContext servletContext) { | |||
| final HandlerMappings handlerMappings = new HandlerMappings(); | |||
| handlerMappings.add(new ManualHandlerMapping()); | |||
| handlerMappings.add(new AnnotationHandlerMapping()); | |||
| handlerMappings.add(new AnnotationHandlerMapping("com.techcourse.controller")); | |||
There was a problem hiding this comment.
해당 부분이 문자열로 하드코딩 되어있다보니 문제가 생길 가능성이 높아 보이는데요, 어떻게 개선할 수 있을까요?
There was a problem hiding this comment.
상수화를 할 수도 있겠지만....
this.getClass().getPackageName();로 패키지를 가져올 수 있네요 ❗️
| @@ -0,0 +1,23 @@ | |||
| package webmvc.org.springframework.web.servlet.view; | |||
|
|
|||
| public class JspViews { | |||
There was a problem hiding this comment.
이 부분의 패키지를 보면 SpringFramework에서 App을 알고있는 관계가 형성되네요!
이런 구조가 문제가 없을지, 어떻게 개선할 수 있을지 고민해 보시면 좋을 것 같습니다.
There was a problem hiding this comment.
앗.. 중복만 해결해보자고 했는데.. 코드를 작성하면서 패키지 양방향 의존이 제일 보기 어려운거 같아요.. 😰
구조까지 섬세히 봐주시다니 역시 말랑..
JspViews를 app 모듈로만 옮겨도 app -> mvc 단방향이 가능할 거 같습니다..!!
| final ModelAndView mav = handlerAdapter.execute(request, response, handler); | ||
| mav.render(request, response); |
There was a problem hiding this comment.
(단순 질문입니다)
HandlerAdapter에서 view를 rendering하는 역할까지 부여하는 것에 대해서는 어떻게 생각하시나요?
이렇게 구현하신 이유가 궁금합니다!
There was a problem hiding this comment.
이 부분을 구현할 때 3가지 방법이 있을 거 같습니다.
1번
handlerAdapter.execute(request, response, handler);2번
final ModelAndView mav = handlerAdapter.execute(request, response, handler);
mav.render(request, response);3번
final ModelAndView mav = handlerAdapter.execute(request, response, handler);
final Map<String, Object> model = mav.getModel();
final View view = mav.getView();
view.render(model, request, response);1번을 사용하면 코드가 깔끔해지지만 HandlerAdapter가 너무 많은 책임을 갖게 되고...
3번을 사용하면 책임 분리가 잘 되었지만 응집성이 초큼 떨어지는 거 같습니다..
그래서 적당한.. 2번 방식을.. 사실 ModelAndView의 책임을 어디까지로 보는게 맞는지 잘 모르겠네요..
(Model도 View에 사용하는 데이터이니... 렌더링 해줘도 되지않을....)
그래도 2번이 아닌 1번과 3번을 선택해야 한다면 3번을 선택할 거 같습니다..!
말랑의 생각도 궁금합니다!
|
|
||
| public class JsonView implements View { | ||
|
|
||
| private final ObjectMapper mapper = new ObjectMapper(); |
|
고생하셨습니다 호이~~ |
안녕하세요. 말랑 🙇♂️
3단계는 legacy를 지우고 JsonView를 구현하였습니다..
그러고
DisptacherServlet을 mvc 패키지로 이동시켰습니다..이번 단계도 잘 부탁드립니다!