[MVC 구현하기 - 2,3단계] 토리(천은정) 미션 제출합니다. #632

ezzanzzan · 2023-09-25T08:59:42Z

안녕하세요 라온 !!
2,3단계 미션 한 번에 보내게 되었는데 양해해주셔서 감사합니다 🙇‍♀️
미션이 저한텐 조금 어렵게 다가와서 요구사항을 중점으로 힌트를 많이 참고해서 구현했어요..

2단계 미션 커밋
3단계 미션 커밋

오늘도 좋은 하루 되세요 ~~ 감사합니다 ❤️🌈

- initialize 내부 메소드 가독성을 위해 for문 대신 stream 사용 - RequestMapping 어노테이션의 method을 배열로 가져오도록 변경

mcodnjs

안녕하세요 토리 👧🏻
시간이 많이 부족했음에도 불구하고 잘 구현해주신거 같아요!!
2단계, 3단계 커밋도 나눠주셔서 보기 편했어요 ㅎㅎ

요구사항 위주로 리뷰 조금 남겼는데 고민해보시고 반영해주시면 좋을거 같아요 ㅎㅎ
다음 미션이 기다리고 있으니 .. 다시 요청 주시면 바로 머지하도록 하겠습니당 🚀

궁금하신 점 있으시면 언제든 DM 주세요 ~~
수고하셨습니당 마지막까지 파이팅 🙃 !!

mcodnjs · 2023-09-25T16:07:24Z

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

+@Controller
+public class LogoutController {
+ @RequestMapping(value = "/logout", method = RequestMethod.GET)
+ public ModelAndView show(final HttpServletRequest req, final HttpServletResponse res) {


로그아웃 역할을 담은 메서드명으로 수정해보는건 어떨까요?!

GET 으로 들어오는 요청에 대해서는 각 Controller에 대한 화면을 출력해준다는 의미로 통일성을 주고 싶었는데,
생각해보니 로그아웃은 화면을 띄워주는게 아니라 로그아웃 역할을 하는 부분이니 수정하는게 더 역할에 맞는 네이밍이겠네요 !!
logout으로 수정했습니다 ~ ˙ᵕ˙ 🙇‍♀️

mcodnjs · 2023-09-25T16:22:34Z

mvc/src/main/java/webmvc/org/springframework/web/servlet/mvc/tobe/ControllerScanner.java

+ .collect(Collectors.toMap(Function.identity(), this::instanticateClazz));
+ }
+
+ private Object instanticateClazz(final Class<?> clazz) {


오타가 👀👀

오마이갓... 바로 수정했습니다 !

mcodnjs · 2023-09-25T16:32:04Z

mvc/src/main/java/webmvc/org/springframework/web/servlet/view/JsonView.java

+ private String bodyToJson(final Object body) throws JsonProcessingException {
+ final ObjectMapper objectMapper = new ObjectMapper();
+ return objectMapper
+ .writerWithDefaultPrettyPrinter()


헉 .. writerWithDefaultPrettyPrinter() !!! 이런 메서드도 존재하는군요! 좋은데요 ?!

json을 예쁘게 출력하기 ! 저도 찾아보면서 배웠습니다 👍

mcodnjs · 2023-09-25T17:03:13Z

mvc/src/main/java/webmvc/org/springframework/web/servlet/view/JsonView.java

 @Override
 public void render(final Map<String, ?> model, final HttpServletRequest request, HttpServletResponse response) throws Exception {
+ final String body = bodyToJson(model);


요구사항의 힌트에 아래와 같은 설명이 써있어요!

model에 데이터가 1개면 값을 그대로 반환하고 2개 이상이면 Map 형태 그대로 JSON으로 변환해서 반환한다.

저는 이 힌트를 이렇게 해석했는데요, 단순히 string으로 반환하는 크루들도 보이더라구요!
토리가 생각하신 방향대로 모델에 데이터가 1개일 경우를 고려해서 수정해보시면 좋을거 같아요 ~~!

// 모델에 데이터가 1개일 경우 { "account" : "gugu", "password" : "1234" } // 모델에 데이터가 여러개일 경우 { "user1" : { "account" : "gugu", "password" : "1234" }, "user2" : { "account" : "raon", "password" : "1234" }, "user3" : { "account" : "tori", "password" : "1234" } }

처음에 어떤식으로 변경해야할지 헷갈려 기존의 방식대로 구현하였습니다 🥲
라온의 리뷰와 오늘 강의를 듣고 아래와 같이 출력되도록 리팩토링하였습니다 ˙ᵕ˙

// 모델에 데이터가 1개일 경우 { "id" : 1, "account" : "gugu", "password" : "password", "email" : "[email protected]" } // 모델에 데이터가 여러개일 경우 { "user1" : { "id" : 1, "account" : "gugu", "password" : "password", "email" : "[email protected]" }, "user2" : { "id" : 1, "account" : "raon", "password" : "raonlove", "email" : "[email protected]" } }

mcodnjs · 2023-09-25T17:12:20Z

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

요구사항대로 DispatcherServlet을 mvc 패키지로 옮겨볼까요?

mcodnjs · 2023-09-25T17:12:45Z

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

ManualHandlerMapping의 패키지도 생각해볼까요 ?!

asis 패키지로 이동했는데 어떨까요 .. ! 패키지가 제일 어려워요... 🥲

맞아요 .. 패키지 너무 어려워요 ㅠㅠ
저도 asis 패키지에 위치하는게 괜찮은거 같아요 ㅎㅎ

mcodnjs · 2023-09-25T17:16:40Z

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

 private static final Logger log = LoggerFactory.getLogger(ManualHandlerMapping.class);
-
 private static final Map<String, Controller> controllers = new HashMap<>();

 public void initialize() {
 controllers.put("/", new ForwardController("/index.jsp"));


ForwardController는 legacy 코드가 아닌지, mvc 패키지에 그대로 위치하는 것이 좋을지 고민해보면 좋을거 같아요!

HomeController 클래스를 생성하여 레거시 코드를 제거해주었습니다 ˙ᵕ˙ !!

mcodnjs · 2023-09-25T17:39:53Z

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

 private static final String DEFAULT_SERVLET_NAME = "dispatcher";

 @Override
 public void onStartup(final ServletContext servletContext) {
 final var dispatcherServlet = new DispatcherServlet();
+ dispatcherServlet.addHandlerMapping(new ManualHandlerMapping());
+ dispatcherServlet.addHandlerMapping(new AnnotationHandlerMapping());


AnnotationHandlerMapping은 생성자로 reflection의 대상이 되는 basePackage를 주입받는데 ..
여기서 주입을 해주지 않아도 app 모듈의 컨트롤러를 정상적으로 불러오네용 ..
이러한 경우에는 어떻게 주입이 되는지 궁금해서 디버깅을 해봤는데, 비어있는 경우 아래 사진에 있는 if문에 걸려/jwp-dashboard-mvc/app/src/main/webapp/WEB-INF/classes/ 경로에 있는 클래스를 모두 로딩해오지만, basePackage를 지정한 경우에는 inputsFilter로 한번 걸러지는거 같아요 ㅎㅎ

신기해서 공유드립니다 ㅎㅎ 관련해서 더 알고계시다면 공유해주세요 ~~

오 사실 저도 이 부분에 대해선 자세히 몰랐는데 라온 덕분에 하나 더 배워갑니다... 🙇‍♀️

- model에 데이터가 1개면 값을 그대로 반환, 2개 이상이면 Map 형태 그대로 JSON으로 변환해서 반환하도록 수정

mcodnjs

안녕하세요 토리 👧🏻
리뷰 반영 빠르게 해주셨네요 👍👍

간단한 커멘트 조금 남겼는데, 확인만 해주셔도 좋을거 같아요 ㅎㅎ
요구사항도 모두 만족한거 같아서 이번 미션은 머지하도록 하겠습니다!

mvc 미션 매우매우 수고 많으셨습니당 😊
jdbc도 파이팅 !!

mcodnjs · 2023-09-26T09:55:11Z

mvc/src/main/java/webmvc/org/springframework/web/servlet/mvc/DispatcherServlet.java

+ protected void service(final HttpServletRequest request, final HttpServletResponse response) throws ServletException {
+ log.debug("Method : {}, Request URI : {}", request.getMethod(), request.getRequestURI());
+ try {
+ final Object handler = handlerMappingRegistry.getHandler(request);


스프링의 경우 리소스가 존재하지 않는 경우 아래처럼 반환해주는 것처럼
Handler를 찾지 못한 경우, getHandler()에서 예외를 던지는 대신 404.jsp를 반환해주는 방법도 있을거 같아요 ㅎㅎ

{ "timestamp": "2023-09-26T08:59:35.244+00:00", "status": 404, "error": "Not Found", "path": "/api" }

오 좋은 방법이네요 !! 감사합니다 🙇‍♀️👍

mcodnjs · 2023-09-26T09:59:00Z

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

mcodnjs · 2023-09-26T10:01:59Z

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

+public class HomeController {
+ @RequestMapping(value = "/", method = RequestMethod.GET)
+ public ModelAndView show(HttpServletRequest req, HttpServletResponse res) {
+ return new ModelAndView(new JspView("redirect:/index.jsp"));


"redirect:/index.jsp"를 사용한 이유가 있을까용 ??

레거시 코드에서 "/"로 가는 경우 index.jsp 로 되어있어서 이렇게 구현하였습니다 .. !

ezzanzzan added 11 commits September 16, 2023 22:59

refactor: AnnotationHandlerMapping 리팩토링

473e87c

- initialize 내부 메소드 가독성을 위해 for문 대신 stream 사용 - RequestMapping 어노테이션의 method을 배열로 가져오도록 변경

refactor: ControllerScanner 클래스 추가 및 역할 분리

6437e72

feat: Legacy MVC와 @MVC 통합 기능 구현

1f93760

chore: 빌드 파일 수정

614c5c2

refactor: DispatcherServlet 내부 뷰에 대한 처리 로직 JspView로 이동

41d69a8

feat: JsonView 클래스 구현

fe88839

fix: getHandler 오류 해결

18be389

refactor: LoginController 어노테이션 기반 MVC로 변경

13eed19

refactor: view 요청 url 변경

334cfe2

refactor: Logoutcontroller 어노테이션 기반 MVC로 변경

073989e

feat: UserController 추가

27dd1c9

ezzanzzan changed the title ~~Step2~~ [MVC구현 / 2,3단계] 토리(천은정) 미션 제출합니다. Sep 25, 2023

ezzanzzan changed the title ~~[MVC구현 / 2,3단계] 토리(천은정) 미션 제출합니다.~~ [MVC 구현하기 - 2,3단계] 토리(천은정) 미션 제출합니다. Sep 25, 2023

mcodnjs requested changes Sep 25, 2023

View reviewed changes

ezzanzzan added 3 commits September 26, 2023 11:45

refactor: 메소드 네이밍 수정

6eb5507

refactor: JsonView class 리팩터링

cf2e62f

- model에 데이터가 1개면 값을 그대로 반환, 2개 이상이면 Map 형태 그대로 JSON으로 변환해서 반환하도록 수정

refactor: 레거시 MVC 제거 및 패키지 구조 변경

a565be8

mcodnjs approved these changes Sep 26, 2023

View reviewed changes

mcodnjs merged commit c004d65 into woowacourse:ezzanzzan Sep 26, 2023
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[MVC 구현하기 - 2,3단계] 토리(천은정) 미션 제출합니다. #632

[MVC 구현하기 - 2,3단계] 토리(천은정) 미션 제출합니다. #632

ezzanzzan commented Sep 25, 2023 •

edited

Loading

mcodnjs left a comment

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs Sep 25, 2023

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs Sep 26, 2023

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs Sep 25, 2023

ezzanzzan Sep 26, 2023

mcodnjs left a comment

mcodnjs Sep 26, 2023

ezzanzzan Sep 30, 2023

mcodnjs Sep 26, 2023

mcodnjs Sep 26, 2023

ezzanzzan Sep 30, 2023

[MVC 구현하기 - 2,3단계] 토리(천은정) 미션 제출합니다. #632

[MVC 구현하기 - 2,3단계] 토리(천은정) 미션 제출합니다. #632

Conversation

ezzanzzan commented Sep 25, 2023 • edited Loading

mcodnjs left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mcodnjs left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ezzanzzan commented Sep 25, 2023 •

edited

Loading