[MVC 구현하기 - 3단계] 에코(조은찬) 미션 제출합니다.

[echo724]

안녕하세요 우가!
이번에 데모데이 잘 하셨나요? ㅎㅎ
데모데이때 너무 달려서 주말에는 좀 쉬어가느라 제출이 늦었습니다 ㅜ
이번에는 리펙토링 80%, 미션 구현 20% 정도로 한 것 같은데요, 힌트의 구조를 좀 보고 추상화를 시도했습니다! 아마 힌트 구조랑 많이 비슷해서 이해하기 쉬우실 것 같네요!
우가 리뷰 잘 부탁 드려요~~~!

[wugawuga]

안녕하세요 에코!!
다행히도 데모데이를 마무리 잘 했답니다! 에코는 잘 하셨나요?

이번에는 의견을 조금만 더 나누고 싶어서 Request Change 드립니다.

[wugawuga]

이 부분 분리를 한다면 getModelMessage() 라던가?

더 가독성이 높아질 것 같은데 에코 의견이 궁금합니다.

render 안에서 response 세팅과 model json 을 만드는 과정이 어색하지는 않지만, 에코의 의견이 궁금해요!

[echo724]

음 아무래도 render라는 메서드에 json body 생성, Response header 설정, Response에 body 작성 등 여러 작업을 하더라구요! 그래서 우가 얘기해주신대로 분리가 가능할 것 같았고, 해당 부분은 아래와 같이 createJsonBody 메서드로 분리했습니다! 확실히 가독성이 좋아졌네요 ㅎㅎ

private String createJsonBody(final Map<String, ?> model) throws JsonProcessingException {
        final ObjectMapper objectMapper = new ObjectMapper();
        String json = "";
        if (model.size() == 1) {
            json = objectMapper.writeValueAsString(model.get(model.keySet().iterator().next()));
        } else {
            json = objectMapper.writeValueAsString(model);
        }
        return json;
    }

[wugawuga]

추석에도 열심히 공부하는 모습은 너무나도 존경스럽네용 ㅎㅎㅎ
본받겠습니다 😎

MVC 미션에 대해서 에코랑 여러 얘기를 나눌 수 있어서 좋았습니다.
고생하셨어요 즐거운 추석되셔요!