-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Labels
🧼 refactor코드의 효율/가독성을 위해 수정한 경우코드의 효율/가독성을 위해 수정한 경우
Description
About Issue 🚀
- 과도한 책임 분산
- UserRepositoryImpl은 UserRetriever, UserRegister에 의존
- PhoneVerificationRepositoryImpl은 PhoneVerificationRegister, PhoneVerificationRetriever, PhoneVerificationRemover에 의존
- 단순한 CRUD 작업을 위해 너무 많은 클래스가 필요
- 일관성 없는 패턴
- UserRetriever.findById()는 UserEntity를 반환하지만, 다른 메서드들은 User 도메인 객체를 반환
- UserRepositoryImpl.findById()에서 불필요한 변환 작업 수행
- 중복된 변환 로직Entity ↔ Domain 변환이 여러 곳에서 반복됨
- UserEntity.fromDomain(), UserEntity.toDomain() 호출이 산재
Progress ✔️
- repository layer 리팩토링
Metadata
Metadata
Assignees
Labels
🧼 refactor코드의 효율/가독성을 위해 수정한 경우코드의 효율/가독성을 위해 수정한 경우