[MVC 구현하기 - 3단계] 라온(문채원) 미션 제출합니다.

[mcodnjs]

안녕하세요, 파워!
빠르게 리뷰해주신 덕분에 3단계도 빠르게 구현할 수 있었습니다 ㅎㅎ
커밋이 많지 않아서, 커밋 보시면서 따라가셔도 좋을거 같아요 ~!
리뷰하시면서 참고하실만한 사항 아래 커멘트에 적어놨습니다 😊
마지막 리뷰도 잘 부탁드립니다 🙇‍♂️👍

[thdwoqor]

안녕하세요 라온 고생하셨습니다.
잘 구현해주셔서 크게 수정할 부분은 없네요 👍
코멘트 확인하시고 다시 리뷰 요청주시면 머지 하겠습니다 🙂

[thdwoqor]

if (viewName.startsWith(REDIRECT_PREFIX))
가 true일경우

model.keySet().forEach(key -> {
    log.debug("attribute name : {}, value : {}", key, model.get(key));
    request.setAttribute(key, model.get(key));
});

해당 연산이 필요없는데 밑에 둔이유가있나요? 기존처럼 위에서 redirect을 판단하고 redirect일경우 바로 return해주면 좋을것같아요

[mcodnjs]

해당 연산은 어플리케이션에서 처리한 데이터를 request 객체를 통해 view에서 활용하기 위해 하는 연산이라고 생각하였는데 (<- 연산의 의도가 제가 생각한게 맞을까요?!), redirect 하는 경우에도 해당 객체가 쓰일 수 있다고 생각해서 이와 같이 배치했습니다 ..
그치만 생각해보니 redirect하는 경우에는 새로운 request 객체가 쓰이겠군요 😅 수정하였습니다 ㅎㅎ

[thdwoqor]

해당 연산은 어플리케이션에서 처리한 데이터를 request 객체를 통해 view에서 활용하기 위해 하는 연산이라고 생각하였는데 (<- 연산의 의도가 제가 생각한게 맞을까요?!)

넵 forward방식은 request를 재사용하고 redirect 방식은 재사용하지않기때문에 라온이 생각한게 맞아요 👍

https://stir.tistory.com/225

[mcodnjs]

안녕하세요, 파워!
남겨주신 커멘트 모두 확인하고 반영해서 다시 리뷰 요청 드립니다!
패키지 변경을 해서 바뀐 부분이 많을거 같은데, 커멘트 주신 부분은 커밋으로 확인하시면 더 편하실거예요!

항상 꼼꼼한 리뷰 해주셔서 너무 좋았습니다 👍
생각해볼 주제까지 던져주시고 감사합니다 ㅎㅎ
마지막까지 잘 부탁드립니다 🙇‍♂️

[mcodnjs]

해당 연산은 어플리케이션에서 처리한 데이터를 request 객체를 통해 view에서 활용하기 위해 하는 연산이라고 생각하였는데 (<- 연산의 의도가 제가 생각한게 맞을까요?!), redirect 하는 경우에도 해당 객체가 쓰일 수 있다고 생각해서 이와 같이 배치했습니다 ..
그치만 생각해보니 redirect하는 경우에는 새로운 request 객체가 쓰이겠군요 😅 수정하였습니다 ㅎㅎ

[thdwoqor]

새벽 4시까지 고생많으셨습니다,,

MVC 미션은 마무리 할게요 다음 미션도 화이팅 👍