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-엔티티 변환 형식 통일 #60

Open
ja7811 opened this issue Aug 17, 2024 · 0 comments
Open

DTO-엔티티 변환 형식 통일 #60

ja7811 opened this issue Aug 17, 2024 · 0 comments
Labels
🛠 REFACTOR code refactoring

Comments

@ja7811
Copy link
Contributor

ja7811 commented Aug 17, 2024

제목이 곧 내용입니다
DTO랑 엔티티 간 변환하는 방식이 기능마다 제각각이여서 통일할 필요가 있습니다

코드 살펴보면 컨버터 없이 Builder로만 변환하는 방식이나
DTO 생성자에 Entity를 받아서 변환하는 방식도 있고 (캐스트)
Builder로 변환하는 걸 service 내에 따로 메소드로 뺀 방식도 있고 (캐스트 2)
Mapper를 따로 만들어서 변환하는 방식도 있고... (회원쪽)

한 가지로 통일하자면 Mapper를 따로 만드는 게 가장 좋을 것 같긴 해요
(User Entity를 DTO로 변환하고 싶다 -> UserMapper.toUserDTO(User user)를 스태틱 메소드로 짜는 방식으로)

변환 방식이 너무 번잡해서 언젠가 통일해야겠다 싶어서 적어둡니다

@ja7811 ja7811 added the 🛠 REFACTOR code refactoring label Aug 17, 2024
@ja7811 ja7811 changed the title DTO-엔티티 컨버터 형식 통일 DTO-엔티티 변환 형식 통일 Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 REFACTOR code refactoring
Projects
None yet
Development

No branches or pull requests

1 participant