[MVC 구현하기 - 3단계] 도기(김동호) 미션 제출합니다. (#603)

* feat: JspView 구현

* feat: JsonView 구현

* refactor: DispatcherServlet에서 View에게 render를 처리하도록 수행

* remove: 테스트를 위한 클래스 제거

* fix: support에서 잘못된 연산으로 생긴 버그 수정

* refactor: 주입하는 basePackages 수정

* refactor: 로그인 및 회원가입 페이지 요청 URL 수정

* refactor: Legacy MVC 제거

* refactor: 주석 제거

* feat: HandlerExecution에 ViewAdapter를 두어 컨트롤러의 반환값에 상관없이 적절한 View를 반환하도록 구현

* feat: asis 패키지에 있는 ForwardController를 디스패처 서블릿의 DefaultController로 승격, DispatcherServlet의 패키지를 mvc 패키지 아래로 이동

* refactor: AnnotationHandlerMapping의 경로를 좀 더 광범위하게 적용

* refactor: 레거시 MVC 전체 제거

* rename: 기본 위치로 요청이 들어오는 경우 Index 페이지로 응답하도록 하는 성격에 맞도록 네이밍 수정

* rename: 두둠이 공유해주신 문서에 맞도록 패키지 네이밍 변경

* refactor: 예외 종류 변경

Author: kdkdhoho

app/src/main/java/com/techcourse/DispatcherServletInitializer.java

@@ -4,6 +4,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import web.org.springframework.web.WebApplicationInitializer;
+import webmvc.org.springframework.web.servlet.mvc.dispatcher.DispatcherServlet;
 
 /**
  * Base class for {@link WebApplicationInitializer}

app/src/main/java/com/techcourse/ManualHandlerMapping.java

@@ -1,27 +1,42 @@
 package com.techcourse.controller;
 
+import static web.org.springframework.web.bind.annotation.RequestMethod.GET;
+import static web.org.springframework.web.bind.annotation.RequestMethod.POST;
+
 import com.techcourse.domain.User;
 import com.techcourse.repository.InMemoryUserRepository;
+import context.org.springframework.stereotype.Controller;
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
-import webmvc.org.springframework.web.servlet.mvc.asis.Controller;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import web.org.springframework.web.bind.annotation.RequestMapping;
 
-public class LoginController implements Controller {
+@Controller
+public class LoginController {
 
     private static final Logger log = LoggerFactory.getLogger(LoginController.class);
 
-    @Override
-    public String execute(final HttpServletRequest req, final HttpServletResponse res) throws Exception {
-        if (UserSession.isLoggedIn(req.getSession())) {
+    @RequestMapping(value = "/login", method = GET)
+    public String getLoginPage(HttpServletRequest request, HttpServletResponse response) {
+        return UserSession.getUserFrom(request.getSession())
+                .map(user -> {
+                    log.info("logged in {}", user.getAccount());
+                    return "redirect:/index.jsp";
+                })
+                .orElse("/login.jsp");
+    }
+
+    @RequestMapping(value = "/login", method = POST)
+    public String login(HttpServletRequest request, HttpServletResponse response) {
+        if (UserSession.isLoggedIn(request.getSession())) {
             return "redirect:/index.jsp";
         }
 
-        return InMemoryUserRepository.findByAccount(req.getParameter("account"))
+        return InMemoryUserRepository.findByAccount(request.getParameter("account"))
                 .map(user -> {
                     log.info("User : {}", user);
-                    return login(req, user);
+                    return login(request, user);
                 })
                 .orElse("redirect:/401.jsp");
     }

app/src/main/java/com/techcourse/controller/AnnotatedControllerForTest.java

@@ -1,14 +1,18 @@
 package com.techcourse.controller;
 
+import static web.org.springframework.web.bind.annotation.RequestMethod.GET;
+
+import context.org.springframework.stereotype.Controller;
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
-import webmvc.org.springframework.web.servlet.mvc.asis.Controller;
+import web.org.springframework.web.bind.annotation.RequestMapping;
 
-public class LogoutController implements Controller {
+@Controller
+public class LogoutController {
 
-    @Override
-    public String execute(final HttpServletRequest req, final HttpServletResponse res) throws Exception {
-        final var session = req.getSession();
+    @RequestMapping(value = "/logout", method = GET)
+    public String logout(HttpServletRequest request, HttpServletResponse response) {
+        final var session = request.getSession();
         session.removeAttribute(UserSession.SESSION_KEY);
         return "redirect:/";
     }

app/src/main/java/com/techcourse/controller/LoginController.java

@@ -1,21 +1,30 @@
 package com.techcourse.controller;
 
+import static web.org.springframework.web.bind.annotation.RequestMethod.GET;
+import static web.org.springframework.web.bind.annotation.RequestMethod.POST;
+
 import com.techcourse.domain.User;
 import com.techcourse.repository.InMemoryUserRepository;
+import context.org.springframework.stereotype.Controller;
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
-import webmvc.org.springframework.web.servlet.mvc.asis.Controller;
+import web.org.springframework.web.bind.annotation.RequestMapping;
 
-public class RegisterController implements Controller {
+@Controller
+public class RegisterController {
 
-    @Override
-    public String execute(final HttpServletRequest req, final HttpServletResponse res) throws Exception {
-        final var user = new User(2,
-                req.getParameter("account"),
-                req.getParameter("password"),
-                req.getParameter("email"));
-        InMemoryUserRepository.save(user);
+    @RequestMapping(value = "/register", method = GET)
+    public String getRegisterPage(HttpServletRequest request, HttpServletResponse response) {
+        return "/register.jsp";
+    }
 
+    @RequestMapping(value = "/register", method = POST)
+    public String register(HttpServletRequest request, HttpServletResponse response) {
+        User user = new User(2,
+                request.getParameter("account"),
+                request.getParameter("password"),
+                request.getParameter("email"));
+        InMemoryUserRepository.save(user);
         return "redirect:/index.jsp";
     }
 }