-
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단계] 주노(최준호) 미션 제출합니다. #346
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안녕하세요 주노! 깔끔하게 잘 구현해주셔서 리뷰할 게 많이 없네요..
보고 배웠습니다
@@ -44,6 +44,7 @@ public void await() { | |||
public void stop() { | |||
try { | |||
tomcat.stop(); | |||
tomcat.destroy(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
꼼꼼하시네요! 덕분에 stop과 destroy의 차이를 알아갑니다요.
그런데 사용자에 입장에서는 stop()메서드에서 destroy()까지 일어날 거라 예상하지 못할 수 있으니 destroy메서드를 분리할 수도 있겠네요
public void initialize() { | ||
log.info("Initialized AnnotationHandlerMapping!"); | ||
Reflections reflections = new Reflections(basePackage); | ||
Set<Class<?>> controllers = reflections.getTypesAnnotatedWith(Controller.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reflections를 사용하지 않고 이름을 통해 가져오는 방법도 해보셨었네요!
여담이지만 저는 Object.class를 상속한 class들을 가져오는 방식으로 해보려했으나 안되더군요..
ClassPathScanningCandidateComponentProvider.scanCandidateComponents()를 보니 기존 주노와 비슷하게 fileName으로 찾고있네요
|
||
RequestMapping annotation = method.getAnnotation(RequestMapping.class); | ||
for (RequestMethod requestMethod : annotation.method()) { | ||
HandlerKey key = new HandlerKey(annotation.value(), requestMethod); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
requestMapping의 값으로 url을 잘 가져와서 등록해주고 계시네요!
혹시 controller의 value로 까지 url prefix를 해주고 싶다면 추가해줄 수도 있을 것 같아요
GET, HEAD, POST, PUT, PATCH, DELETE, OPTIONS, TRACE; | ||
|
||
public static RequestMethod from(String method) { | ||
return Arrays.stream(values()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
정적 팩토리 메서드까지 잘 사용해주셨습니다. 저는 valueOf()란 메서드를 사용했는데 이렇게 직접 정의하는 게 더 맞는 방식이라 느껴져요
안녕하세요 도치! 🦔
MVC 구현하기 미션을 함께하게되었네요!! 😆
1단계 진행 간 다음과 같은 내용을 구현했습니다.
AnnotationHandlerMapping 구현
AnnotationHandlerMapping은 initialize 수행 시 다음과 같은 흐름으로 동작합니다.
@Controller
어노테이션이 붙은 클래스를 찾아온다.@RequestMapping
어노테이션이 붙은 메소드를 찾는다.HandlerKey
생성HandlerExecution
생성(클래스 -> 인스턴스 획득)AnnotationHandlerMapping
의handlerExecutions
Map에 저장AnnotationHandlerMapping
을 사용하는 측에서는 getHandler() 메소드를 통해HandlerExecution
을 반환 받고 실행구체적인 구현사항은 코드 내용으로 확인 부탁드립니다 :)
HandlerExecution 구현
HandlerExecution은 인스턴스와 Method를 필드로 가지며 handle 메소드 수행 시 리플렉션의 invoke 메소드를 통해 Method를 실행하는 역할을 수행합니다.