[MVC 구현하기 - 3단계] 가비 미션 제출합니다. #609
Conversation
| import web.org.springframework.web.bind.annotation.RequestMethod; | ||
| import webmvc.org.springframework.web.servlet.ModelAndView; | ||
| import webmvc.org.springframework.web.servlet.view.JspView; | ||
|
|
|
|
||
| private static final Logger log = LoggerFactory.getLogger(LoginController.class); | ||
|
|
||
| @RequestMapping(value = "/login", method = RequestMethod.GET) |
There was a problem hiding this comment.
loginController와 loginViewController가 합쳐지면서 /login/view URI에 대한 요청을 처리해 줄 컨트롤러가 사라진 것 같습니다. application 구동시켜서 동작시켜보시면 현재 안되는 부분들이 있는데 확인해서 수정 부탁드릴게요!
There was a problem hiding this comment.
의도하고 날린 부분이었는데요
*/view 와 같은 방식으로 경로에 뷰임을 지정해줘야 하는 것도 레거시라고 생각했습니다
다만 API 명세 자체를 바꿔버린 꼴이 됐네요
요구사항 잘못 이해하고 바꾼 것 같습니다 다시 바꿔둘게요
|
|
||
| private Object createResult(Map<String, ?> model) { | ||
| if (model.size() == SINGLE_JSON_SIZE) { | ||
| return model.values(); |
There was a problem hiding this comment.
model에 담긴 value가 하나일 때 values()를 통해 배열을 반환하고 있네요! 현재 프로젝트 요구사항에서는 model에 값이 하나일 때 값을 그대로 반환한다고 되어 있는데 그대로 반환한 것일까요? 그리고 User 객체를 json 응답으로 내려주는데 account 정보 하나만 반환되는 것이 괜찮을까요? 저도 여기 부분에서 어떻게 처리해야할지 고민을 조금 했었는데 model이 하나의 값을 가질 때 어떻게 응답을 해줄지 좀 더 고민해보셔도 좋을 것 같습니다. 😃
There was a problem hiding this comment.
아고 이해하는데 좀 걸렸네요
toString으로 user 모델의 모든 값들을 그대로 출력하도록 변경했습니다...!
|
SonarCloud Quality Gate failed.
|
안녕하세요 히이로!
JspView는 2단계에서 미리 해버렸더니..
이렇게 두가지 작업을 하게 됐습니다
여유로우실때 편하게 읽어주셨으면 좋겠습니다..!