[MVC 구현 - 3단계] 스플릿-박상현 미션 제출합니다. #562
Conversation
…Adapter 추가 현 커밋에서 존재하지 않는 renderWithRequestAndResponse 메서드 e862dfa8f7844d4db2c91c54d3ec46719c0f92d9 커밋 에서 추가
메서드 선언부 컨벤션 변경
| return new ModelAndView(REDIRECT_INDEX_PAGE_VIEW); | ||
| } | ||
| return "redirect:/401.jsp"; | ||
| return new ModelAndView(REDIRECT_UNAUTHORIZED_VIEW); |
|
|
||
| public void setNotFound(final HttpServletResponse response) { | ||
| response.setStatus(404); | ||
| } | ||
| } |
There was a problem hiding this comment.
이 부분에 대해서 지난번 리뷰에 이어서 답변하겠습니다..!
음..HandlerExceptionResolver를 인터페이스로 만들고 HandlerNotFoundExceptionResolver를 구현체로 만들어서 DispatcherServletInitializer 클래스에서 해당 Resolver를 추가해주는 방식은 어떨까요?
그리고 말씀해주신 상태코드 수정은 뷰에서 하는 역할인지는 사실 더 고민해봐야 할 것 같네요..ㅠㅠ디스패쳐 서블릿에서 상태코드 변경후 위에서 말한 방식으로 뷰에게 404페이지를 렌더링하는 역할을 넘기는 방법도 있을것같아요..!
There was a problem hiding this comment.
지금 있는 구조에서 해당 부분에 대해서 진행할 수 있게 코드 추가해서 커밋했습니다!!!😊
There was a problem hiding this comment.
리팩토링하기 어려운 부분이라 생각했는데 이렇게 빠르게 해내시다니..감동3...
|
|
||
| public HandlerAdapter getFirstHandleableAdapterForHandler(final Object handler) { | ||
| final List<HandlerAdapter> handleableAdaptersForHandler = getHandleableAdaptersForHandler(handler); | ||
| if (handleableAdaptersForHandler.isEmpty()) { | ||
| throw new HandlerAdapterNotExistException(); |
|
|
||
| Optional<Object> getHandler(final HttpServletRequest request); | ||
| boolean isHandleable(final HttpServletRequest request); |
There was a problem hiding this comment.
해당 메서드가 없으면 HanlderMappings 가 Handler 를 찾는 과정에서 stream 을 통해 실제로 받은 Handler 가 null 인지를 체크하는 방식을 택해야 하더라구요!!😭
개인적으로 null을 다루는 것보다 해당 객체에게 직접 handleable 한지 물어보는 것이 더 바람직하다고 생각하여 해당 메서드를 생성하였습니다!!
| final HandlerMapping handlerMapping = handlerMappings.stream() | ||
| .filter(mapping -> mapping.isHandleable(request)) | ||
| .findFirst() | ||
| .orElseThrow(HandlerNotExistException::new); | ||
|
|
||
| return handlerMapping.getHandler(request); |
|
|
||
| @DisplayName("/ 경로 요청 테스트") | ||
| class HomeControllerTest extends UsingTomcatTest { | ||
| @DisplayName("GET 요청시 200 상태코드 와 함께 메인 페이지로 응답한다.") | ||
| @Test | ||
| void save() { | ||
| try (CloseableHttpClient httpClient = HttpClients.createDefault()) { | ||
| //given | ||
| final HttpGet httpGet = new HttpGet(tomcatUrl + "/"); |
|
SonarCloud Quality Gate failed.
|
안녕하세요 채채😊!!
저번 step2 마지막 코멘트들에 답변 했으니 해당 부분 더 얘기해보면 좋을 것 같아요!!👍
이번 리뷰도 잘 부탁드립니다!! 🙇