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

[refactor] UserService 구현 및 Dto / Mapper 정리 #170

Closed
2 tasks
kimminkyeu opened this issue Oct 8, 2024 · 0 comments · Fixed by #180
Closed
2 tasks

[refactor] UserService 구현 및 Dto / Mapper 정리 #170

kimminkyeu opened this issue Oct 8, 2024 · 0 comments · Fixed by #180
Assignees
Labels
backend 백엔드 관련 refactor 🚧 리팩토링 관련

Comments

@kimminkyeu
Copy link
Collaborator

kimminkyeu commented Oct 8, 2024

기능

  • Structure에서 NameProvider 로직 제거 + 캐싱 으로 변경
  • 이름의 경우 CustomTitle이 null 인 경우만 에러이고, 그 나머지는 모두 정상. 빈 제목도 사용자가 설정 가능하며, 기본적으로는 Pick 당시의 제목을 프론트에서 보내준다. --> ⚠️ Link의 제목을 사용할 필요가 없다. 이건 검색 때만 필요하다.

⚠️ 리팩토링 검증 기준

  1. DTO는 불변 객체여야 한다.
    • Setter가 없어야 한다.
  2. 서로 다른 도메인의 의존성을 주입받을 땐, 최상단 레이어 서비스만 주입뱓을 수 있다.
    • Ex. Pick, Folder 쪽의 모든 기능은 사용자 정보를 확인할 때 UserSerive만 이용해야함,
  3. Mapper는 의존성이 없어야 한다.
    • Mapper는 변환 코드들을 하나의 클래스로 묶어둔 더미 코드일 뿐, 외부 서비스에 의존해선 안된다.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 관련 refactor 🚧 리팩토링 관련
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant