Search
Duplicate

1단계 피드백

목차
1단계 종료

1. Controller RequestBody로는 DTO를 사용하자.

API요청하는 측에서 보내는 JSON및 Model에 대응하는 RequestBody 혹은 ModelAttribute는 DTO를 통하도록 하자. 그밖에 외부로 노출하거나 외부에서 받는 모든 Object에 대해서 엔티티로 바로 받기보단 DTO와 같은 데이터 전송 객체를 사용하는 것이 좋다.

2. 유연성을 키우는것도 좋지만 지나치게 코드를 늘리지

기존 작성한 코드에서는 Map 타입을 바로 파라미터로 넘겨주는 경우와, 각각의 파라미터(ex: name, color)를 순서대로 넣어주는 메서드를 둘 다 만들어서 사용했는데, 필수불가결한 경우가 아닌경우 억지로 늘리지 않는다. 보내는 측에서 바로 꺼내서 줘도 된다.
해당 미션진행중에는 Map을 파라미터로 받는 메소드에서 해당 map의 파라미터를 꺼내어 순서대로 파라미터를 세팅해 메소드를 호출했는데 여기선 또 api요청을 하기위해 param을 HashMap에다가 담아서 보내는지라 Map에서 파라미터를 풀었다가 다시 감싸는 경우가 생겨서 하나로 통합하도록 했다.