-
Notifications
You must be signed in to change notification settings - Fork 302
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.
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 구현하기 1, 2단계] 하디(전동혁) 미션 제출합니다. #340
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 도기 반갑습니다 ~ 하디입니다.
1단계와 2단계를 같이 진행하였는데요.
사실 모르는게 많은 것 같아요.
DispatcherServlet
은 왜app
에 있는지도 이해가 잘 안됩니다...패키지에 구조를 잘 봐주셨으면 좋겠어요 !
공부할게 너무 많네요..ㅠㅠ
ReflectionUtils
는 사용하려 했으나 제가 봤을 때 뭔가 .. 더 가독성이 떨어져 보여서 사용하지 않았습니다.LegacyHandlerMapping
이라는abstract class
가 있을텐데요 !ManualHandlerMapping
과AnnotaionHandlerMapping
의getHandler
부분의 인자가 달라서@Override
하기 힘들더라구요 ..!그래서 중간에 추상 클래스를 하나두어
ManualHadnlerMapping
의getHandler
부분을 바꾸지 않고 해결해보려 시도해보았습니다..!전체적인 구조는
LMS
에 나와있는 2단계 최종구조와 비슷하게 가져가려고 노력했습니다 !! 물론 다른 부분도 있습니다 ㅎㅎ..리뷰 잘 부탁드려요 !