Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MVC 구현하기 3단계] 디투(박정훈) 미션 제출합니다. #625

Merged
merged 9 commits into from
Sep 26, 2023
Merged

[MVC 구현하기 3단계] 디투(박정훈) 미션 제출합니다. #625

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
5fe6ab3
refactor: 패키징 변경
shb03323 Sep 25, 2023
43e0a2e
refactor: view 역할 분리
shb03323 Sep 25, 2023
b5f2211
feat: JsonView 구현
shb03323 Sep 25, 2023
31b393b
feat: UserController 추가
shb03323 Sep 25, 2023
1450f4c
feat: 레거시 controller 제거
shb03323 Sep 25, 2023
e829dcd
refactor: dispatcherServlet 패키지 이동
shb03323 Sep 25, 2023
f384e25
refactor: User에 getter 추가
shb03323 Sep 25, 2023
92a75c5
refactor: 로그아웃 오류 수정
shb03323 Sep 25, 2023
2cc9a45
refactor: ObjectMapper를 JsonView 필드로 두기
shb03323 Sep 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions app/src/main/java/com/techcourse/DispatcherServletInitializer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import web.org.springframework.web.WebApplicationInitializer;
import webmvc.org.springframework.web.servlet.DispatcherServlet;

/**
* Base class for {@link WebApplicationInitializer}
Expand Down
3 changes: 1 addition & 2 deletions ...ava/com/techcourse/DispatcherServlet.java → ...mework/web/servlet/DispatcherServlet.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
package com.techcourse;
package webmvc.org.springframework.web.servlet;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DispatcherServelt의 먼 여정 커밋 순서대로 잘 확인했습니다! 흐름을 순서대로 정리하면 아래와 같이 될 것 같아요 :)

  1. 프레임워크 패키지가 애플리케이션 패키지 의존성을 가지도록 하고 DispatcherServlet, Initialize mvc 패키지로 수정
  2. 애플리케이션 패키지 의존성을 제거하고 DispatcherServlet, Initialize app 패키지로 수정
  3. DispatcherServlet만 mvc 패키지로 수정

각 과정을 진행하면서 어떤 생각의 흐름으로 진행하셨는지 궁금해요. 디투의 여정을 설명해주시면 감사합니다 ㅎㅎ

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅋㅋㅋ 맨 처음 기능 요구 사항을 보기 전에 저 DispatcherServlet을 옮기려고 mvc 패키지로 옮겼습니다.
아니 그런데 다른 모듈이라서 build.gradle 파일에 의존성을 추가시켜 줘야 하더라구요! 그렇게 해야 app 모듈에 있는 ManualXXX를 import 할 수 있었어요. 그런데 그렇게 되면 실행할 때 mvc 패키지와 app 패키지에서 순환참조가 발생하더라구요...

그래서 일단 DispatcherServlet을 다시 app 패키지로 옮기고 작업을 했어요!
마지막에 ManualXXX를 싹 다 지워서 build.gradle에 의존성 추가하지 않아도 되게끔 수정을 했고, mvc로 옮겼습니다.

Initialize도 옮기고 싶었지만, 그렇게 되면 uri mapping을 읽지 못하더라구요. ServletContext 때문이라고 생각하는데, 급한 나머지 자세히 뜯어보진 않았습니다ㅠㅠ

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

슬랙에 코치님께서 Initialize를 굳이 옮기지 않아도 된다고 하셔서 뭔가 설정이 있겠구나 하고 넘어갔습니다!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

과정을 하나씩 진행하면서 직접 겪었던 문제를 바탕으로 수정하신 점이 인상깊네요 👍


import jakarta.servlet.ServletException;
import jakarta.servlet.http.HttpServlet;
import jakarta.servlet.http.HttpServletRequest;
import jakarta.servlet.http.HttpServletResponse;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import webmvc.org.springframework.web.servlet.ModelAndView;
import webmvc.org.springframework.web.servlet.adapter.HandlerAdapter;
import webmvc.org.springframework.web.servlet.adapter.HandlerAdapters;
import webmvc.org.springframework.web.servlet.handlermapping.HandlerMappings;
Expand Down