[MVC 구현하기 - 1단계] 히이로(문제웅) 미션 제출합니다. #390
Merged
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
parkmuhyeun
approved these changes
Sep 13, 2023
Comment on lines
+4
to
+16
| - [] 어노테이션 기반의 MVC 프레임워크를 구현한다. | ||
| - [] mvc 모듈과 app 모듈의 영역이 잘 구분되어야 한다. | ||
| - [] 새로운 컨트롤러가 생겨도 MVC 프레임워크 영역까지 영향이 미치면 안된다. | ||
| - [x] AnnotationHandlerMapping 구현 사항 | ||
| - [x] initialize 기능 구현 | ||
| - [x] basePackage 하위에 존재하는 @Controller 어노테이션 선언 클래스 탐색 | ||
| - [x] Controller 클래스 내 선언된 @requestMapping 어노테이션 선언 메서드 탐색 | ||
| - [x] 각 Method 객체를 조합으로 가지는 HandlerExecution 객체를 생성해서 저장 | ||
| - [x] getHandler 기능 구현 | ||
| - [x] HttpServletRequest 객체를 인자로 받아서 HandlerKey 객체 생성 | ||
| - [x] 생성된 HandlerKey 객체를 key로 가지는 value 값에 해당하는 handler를 반환 | ||
| - [x] HandlerExecution 구현 사항 | ||
| - [x] handler 객체에서 request, response 객체를 인자로 받아 적합한 비즈니스 로직을 수행하는 기능을 구현한다. |
| log.info("Initialized AnnotationHandlerMapping!"); | ||
| } | ||
|
|
||
| private void addAnnotatedHandlerExecution(Class<?> controller) { | ||
| for (Method method : controller.getMethods()) { | ||
| if (!method.isAnnotationPresent(RequestMapping.class)) { |
There was a problem hiding this comment.
stream으로 좀 더 깔끔하게 뎁스를 줄여볼 수 있을거 같아요!
| } | ||
| } | ||
|
|
||
| private Set<Class<?>> getAnnotatedControllerClasses() { |
There was a problem hiding this comment.
메서드 위치를 순서대로 두면 좀 더 읽기 수월할 것 같습니다.
| public Object getHandler(final HttpServletRequest request) { | ||
| return null; | ||
| String requestURI = request.getRequestURI(); | ||
| RequestMethod requestMethod = RequestMethod.valueOf(request.getMethod().toUpperCase()); |
There was a problem hiding this comment.
method는 대문자로만 들어와서 toUpperCase 해줄 필요 없지 않을까요
There was a problem hiding this comment.
생각해보니 소문자로 들어오면 대문자로 변환해줄 게 아니라 예외처리를 해줘야겠네용 ㅎㅎ... 감사합니다! 👍
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.
안녕하세요 무민, 히이로입니다! 🙇
이번 MVC 미션 리뷰이로 함께하게 되어 기쁩니다 ㅎㅎ
듣기로는 전설의 톰캣 컨트리뷰터라고 하시던데 많이 배워가겠습니다 ^^
첫번째 단계는 AnnotationHandlerMappingTest를 통과시키기 위한 구현을 완료했습니다. 이 외 별다른 구현사항은 없었어서 자세한 내용은 README의 기능 명세서를 봐주시면 좋을 것 같아요.
리뷰 잘 부탁드립니다~ 😄