[MVC 구현하기 - 1단계] 애쉬(정설희) 미션 제출합니다. #336
Conversation
|
|
||
| public void initialize() { | ||
| Reflections reflections = new Reflections(basePackage); | ||
| Set<Class<?>> controllers = reflections.getTypesAnnotatedWith(Controller.class, true); |
There was a problem hiding this comment.
refelections.getTypesAnnotatedWith(Controller.class)를 호출해도 테스트들을 잘 통과하던데 isHonorInherited 파라미터를 true로 전달한 이유는 무엇인가요?
@Class 어노테이션을 처리하기 위해서 어떤 점을 고려해서 사용한 건지 궁금합니다.
| private void mapController(final Class<?> clazz) { | ||
| Method[] methods = clazz.getDeclaredMethods(); | ||
| for (final Method method : methods) { | ||
| mapControllerMethod(createControllerInstance(clazz), method, getRequestPath(clazz)); |
There was a problem hiding this comment.
저는 메서드들이 등장한 순서대로 정렬하는 방법이 코드를 읽을 때 편해서 더 선호하는데 애쉬의 생각은 어떤가요?
| } | ||
| } | ||
|
|
||
| public String getRequestPath(final Class<?> clazz) { |
There was a problem hiding this comment.
ide 메서드 추출 기능으로 하다가 실수했나봐요 🥲
다시 private으로 닫아두겠습니다!
| private void mapController(final Class<?> clazz) { | ||
| Method[] methods = clazz.getDeclaredMethods(); | ||
| for (final Method method : methods) { | ||
| mapControllerMethod(createControllerInstance(clazz), method, getRequestPath(clazz)); |
There was a problem hiding this comment.
mapControllerMethod에서 특정 클래스에서 핸들러에 등록할 메서드를 찾는 과정에서 메서드마다 클래스의 인스턴스를 새로 만들어주고 있는데 for문 이전에 하나만 만들어서 재사용하는 건 안 되나요?
학습 테스트도 미션 시작도 안 한 상태라 기반 지식이 전혀 없는 상태라 궁금하네요...
| modelAndView.addObject("id", request.getAttribute("id")); | ||
| return modelAndView; | ||
| } | ||
| @RequestMapping(value = "/update-test", method = {RequestMethod.PUT, RequestMethod.PATCH}) |
There was a problem hiding this comment.
RequestMethod[] 형태로 가져오는데 method를 두 개 가지는 메서드를 만들어서 테스트해주신 점 너무 좋았습니다 👍👍
| import webmvc.org.springframework.web.servlet.view.JspView; | ||
|
|
||
| @Controller | ||
| @RequestMapping("/test") |
There was a problem hiding this comment.
Class 레벨에 달린 @RequestMapping 까지 잘 처리하는군요 😆
안녕하세요 썬샷!! ☀️
AnnotationHandlerMapping 클래스를 리플렉션을 통해 구현해보았습니다!
app쪽 코드랑 뷰 관련 코드들은 이후 단계에서 진행하는 것 같아 우선 간단하게 어노테이션 기반 맵핑 코드만 작성했습니다!
(AnnotationHandlerMapping과 HandlerExecution 클래스만 변경해서.. 제가 뭔가 요구사항을 놓친건가 싶네요)
이번 단계에서
@RequestParam@PathVariable어노테이션도 추가적으로 구현을 하는건지 잘 모르겠네요!구현이 필요하다면 코멘트 남겨주시면 추가 구현 하도록 하겠습니다아
잘 부탁 드립니다 🙇🏻♀️