[EAT-77] controller와 service의 결합도 줄이기

[wjdwnsdnjs13]

상위 이슈

• [None]

  이슈 링크

• EAT-77

  이슈 내용

• 이미지
• 설명
  • controller에서 service가 사용하는 데이터를 표시하는 것은 결합도가 높다고 생각됩니다.
  • 이전에 논의했던 대로 service에게 데이터를 넘길 때는 DTO를 사용해서 하는 것으로 통일했으면 합니다.

[jaewoong-gwon]

위처럼 작성되있는 것은 Controller 가 Service 가 필요한 데이터를 미리 알고 호출하는것 같네요

[wjdwnsdnjs13]

그런 문제를 방지하기 위해 하나의 데이터를 받는 경우에도 DTO에서 받아서 서비스단에 넘기는 것으로 수정하려합니다.

[jaewoong-gwon]

그럼 @PathVariable 을 쓰지 않는쪽으로 바꾸신다는 거죠?

[wjdwnsdnjs13]

답장이 조금 늦었습니다 ㅎㅎ 아무래도 Swagger UI를 활용한 API 명세서 관리와 DTO 관리의 편의, 통일성을 위해서 DTO를 사용하는 걸 통일하는 쪽으로 했으면 합니다.

[jaewoong-gwon]

• RequstDto : 요청을 받기 위한 객체
  • Service Layer 에서는 요청을 받기 위한 객체를 사용하는 것이 아닌 실제 로직에 필요한 객체를 사용해야함
  • 요청을 받기 위한 Dto 를 생성한 뒤, Service Layer 에서 로직을 실행 시 필요한 객체로 변환
  • Service Layer 에서 사용하는 객체를 다시 Entity 로 변환하여 DB 와 통신
• ResponseDto : 응답을 보내기 위한 객체
  • 응답을 보내기 위해 가공을한 객체로 Service Layer 에서 생성 or Controller Layer 에서 생성후 응답에 사용

[jaewoong-gwon]

• Controller Layer 와 Service Layer 간은 Dto 를 주고 받음
  • Controller Layer 에서는 RequestDto 를 다룸
  • RequestDto 를 Service Layer 에서 필요한 type 으로 변환하여 전송
  • Service Layer 에서 사용하는 새로운 Dto 가 필요할 것 같음

    참고

• 우아한 기술블로그 Service 와 Controller 의 강한 결합 : 2번 단락 참고할 것