Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

엔티티, dto 간 mapping 리팩토링, api 문서상 id명 수정 및 코드 리팩토링 #49

Closed
donghyun0304 opened this issue Mar 13, 2024 · 1 comment · Fixed by #51
Assignees
Labels
Priority: Medium 우선순위 중 Status: To Do 처리해야 되는 이슈 Type: Feature/Document 문서 추가 및 보완 작업 Type: Feature/Function 새로운 기능 및 개선 사항

Comments

@donghyun0304
Copy link
Contributor

donghyun0304 commented Mar 13, 2024

추가/개선 필요 요소

  1. 코드리뷰에서 나왔던 엔티티, dto 간의 mapping 을 mapper 클래스를 이용해서 리팩토링 예정,
  2. api 문서에 id값이 명확하게 적혀있지 않았던 것을 (ex) memberId 처럼 명확하게 수정하고, 코드에도 적용 예정

고민했던내용

MapStruct 라이브러리를 이용해서 mapping 작업을 고려했으나,
dto와 엔티티의 변환 작업이 service 레이어에서 이루어지고 있음.
MapStruct를 이용할 경우, 서비스 레이어가 라이브러리에 의해 순수하게 유지되지 못함.
순수하게 유지하기 위해 '어댑터 패턴'을 적용해볼까 생각을 했으나, 배보다 배꼽이 더 커지는 상황이 발생할것으로 예상.

결론

어댑터 패턴을 쓰면 배보다 배꼽이 더 커지는건 어쩔 수 없지만, 서비스 레이어가 순수하게 유지될 수 있다는 장점이 있으므로, 경험삼아 한번 써보는걸로

@donghyun0304 donghyun0304 added Priority: Medium 우선순위 중 Status: To Do 처리해야 되는 이슈 Type: Feature/Function 새로운 기능 및 개선 사항 Type: Feature/UI UI 수정 및 개선 사항 labels Mar 13, 2024
Copy link

Branch feature-49 created!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium 우선순위 중 Status: To Do 처리해야 되는 이슈 Type: Feature/Document 문서 추가 및 보완 작업 Type: Feature/Function 새로운 기능 및 개선 사항
Projects
None yet
1 participant