[1 - 3단계 - 방탈출 예약 관리] 카피(김상혁) 미션 제출합니다.

[tkdgur0906]

안녕하세요 미르!
저는 카피입니다!

피드백 받으면서 궁금한 것이 생기면 열심히 질문드리겠습니다!

이전의 웹 개발 경험

• 스프링을 통한 프로젝트 개발 경험이 있습니다.
• AWS를 통해 배포해본 적이 있습니다.

[ddu0422]

안녕하세요 카피! 미르입니다 :)
방탈출 예약 관리 미션 구현하느라 고생 많으셨습니다!
몇 가지 코멘트 남겼으니 확인 부탁드립니다
궁금한 점이 있으면 언제든지 코멘트 남겨주시거나 DM주세요 😄

먼저 요청하셨는데 제가 알림을 놓쳤네요 ㅜㅡㅜ
늦게 리뷰드려 죄송합니다 🙏🏻
다음부턴 알림을 꼼꼼히 확인하도록 하겠습니다.

[tkdgur0906]

Controller에서 사용자로 부터 받는 request와 내보내는 response를 dto로 변경하였습니다.
dto를 통해 원하는 정보만 보여줄 수 있고, 요청을 받을 때 id가 null로 저장되지 않게 하기 위해 dto를 사용했습니다.

궁금한점

1. dto와 entity간의 변환 로직의 위치

하지만 여기서 고민한 부분은 dto와 entity간의 변환 로직의 위치 입니다.
생각해본 방안은 다음과 같습니다.

1. dto(request) -> entity : request dto내부 변환 메서드
   entity -> dto(request) : entity 내부 변환 메서드

2. dto(request) -> entity : controller에서 dto의 getter를 사용해 entity를 new로 생성
   entity -> dto(request) : controller에서 entity의 getter를 사용해 dto를 new로 생성

3. dto(request) -> entity : request dto내부 변환 메서드
   entity -> dto(request) : request dto 내부 변환 메서드

여기서 3번 방안을 선택하였습니다.
이유는 컨트롤러와 entity가 변환 로직을 알게하고 싶지 않았기 때문입니다.
해당 방식에 대해 어떻게 생각하시는지 미르의 의견이 궁금합니다!

2. List 형식으로 Json 반환하는 것

예전에 봤던 글 중에서 List 자체를 Json으로 반환하게 되면 확장성이 좋지 않아 껍데기 역할을 하는 클래스로 감싸 반환하는 것이 좋다라는 내용을 본 적이 있습니다.
미르도 실제로 리스트를 반환하는 경우 감싸 반환하시는지 궁금합니다!

[ddu0422]

안녕하세요 카피! 미르입니다 :)
구현 잘 해주셔서 이만 머지해도 될 것 같아요.
추가 코멘트는 다음 미션 진행하시면서 확인해주시면 감사하겠습니다 😄

다음 미션도 화이팅입니다!