[MVC 구현하기 - 3단계] 준팍(박준현) 미션 제출합니다. #595
Conversation
| @Override | ||
| public void render(final Map<String, ?> model, final HttpServletRequest request, HttpServletResponse response) throws Exception { | ||
| public void render( |
There was a problem hiding this comment.
현재 준팍의 JsonView를 통해 랜더링 된 User 도메인을 보면 아래와 같이 나옵니다.
User{id=1, account='gugu', email='[email protected]', password='password'}
그러나, 위는 Json 형식이 아닌 것 같아요.
아래와 같이 나올 수 있도록 수정하시는건 어떨까요?
{
"user": {
"id": 1,
"account": "gugu",
"password": "password",
"email": "[email protected]"
}
}There was a problem hiding this comment.
이래서 눈치 빠른 제리는 좋단 말이지... 👍👍👍
| } | ||
|
|
||
| @RequestMapping(value = "/logout", method = RequestMethod.GET) | ||
| public ModelAndView logout(HttpServletRequest req, HttpServletResponse res) { |
There was a problem hiding this comment.
logout까지 LoginController로 이동하셨군요!!
이렇게 하면 프로젝트 규모가 켜졌을 때 로그아웃이 어디에 구현되어 있는지 헷갈릴 것 같아요. 저는 컨트롤러를 로그인과 로그아웃을 모두 포함하는 이름으로 추상화 하거나 로그아웃 컨트롤러로 다시 나누는게 좋아 보이는데 준팍은 혹시 왜 하나로 합쳤는지 설명해 주실 수 있나요?
There was a problem hiding this comment.
제리의 의견에 공감합니다.
AuthController로 네이밍 변경하였습니다.
두 기능을 하나의 컨트롤러로 합친 이유는, 두 기능은 서로 다른 하나만 구현하였을 때
큰 의미가 없으니 한번에 관리를 해야한다고 생각했고,
그렇기 때문에 하나의 컨트롤러에서 관리해야겠다고 생각해서 입니다.
| @@ -30,4 +35,16 @@ public void onStartup(final ServletContext servletContext) { | |||
|
|
|||
| log.info("Start AppWebApplication Initializer"); | |||
| } | |||
|
|
|||
| private static DispatcherServlet createDispatcherServlet() { | |||
| AnnotationHandlerMapping annotationHandlerMapping = new AnnotationHandlerMapping("com.techcourse.controller"); | |||
There was a problem hiding this comment.
컨트롤러를 검색할 패키지를 주입받아 완벽히 의존성을 끊어주었군요👍
안녕하세요 제리~!!
우선 DI학습테스트 제외하고 제출합니다.
빠른 시일내에 학습테스트 추가하겠습니다~^^