Closed mobzzzzz closed 4 months ago
간단 요약
User 각 계층의 로직을 제외한 기초 코드를 작성했습니다. 로직을 제외한 대부분의 책임은 Dto에 위임하는 방식을 적용했습니다.
PR에서 작업한 사항들을 적어주세요(이미지, 스크린샷등을 활용해도 좋아요)
명시적인 Dto 이름을 제거했습니다. User status enum의 ALIVE 를 ACTIVE로 수정했습니다. (ERD도 수정함)
리뷰시 봐주었으면 하는 부분이 있다면 적어주세요
다른 변환의 책임은 다들 작성하신 거랑 비슷한 모습인데 UpdateUserRequest 에서 작성한 내부 함수 apply는 예전에 지나가다 본 레퍼런스에서 Entity의 어떤 내용을 업데이트 하는지도 숨기길래 이거 좀 신기하네 하고 써봤습니다.
UpdateUserRequest
apply
그런데 다른 Dto의 Companion object 랑 모양새가 다르기도 하고 이정도로 로직에서 숨겨야하나? 같은 생각도 같이 들어서 이 부분에 대한 리뷰어분들의 호불호가 궁금합니다.
closes: #15
요약
User 각 계층의 로직을 제외한 기초 코드를 작성했습니다. 로직을 제외한 대부분의 책임은 Dto에 위임하는 방식을 적용했습니다.
작업 사항
명시적인 Dto 이름을 제거했습니다. User status enum의 ALIVE 를 ACTIVE로 수정했습니다. (ERD도 수정함)
리뷰 요청 사항(선택)
다른 변환의 책임은 다들 작성하신 거랑 비슷한 모습인데
UpdateUserRequest
에서 작성한 내부 함수apply
는 예전에 지나가다 본 레퍼런스에서 Entity의 어떤 내용을 업데이트 하는지도 숨기길래 이거 좀 신기하네 하고 써봤습니다.그런데 다른 Dto의 Companion object 랑 모양새가 다르기도 하고 이정도로 로직에서 숨겨야하나? 같은 생각도 같이 들어서 이 부분에 대한 리뷰어분들의 호불호가 궁금합니다.
해결한 이슈
closes: #15