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

♻️ 쿼리 메서드 이름 개선 및 연관 로직 주석 추가 #271

Closed
uwoobeat opened this issue Mar 6, 2024 · 0 comments · Fixed by #274
Closed

♻️ 쿼리 메서드 이름 개선 및 연관 로직 주석 추가 #271

uwoobeat opened this issue Mar 6, 2024 · 0 comments · Fixed by #274
Assignees

Comments

@uwoobeat
Copy link
Member

uwoobeat commented Mar 6, 2024

📌 Description

주석 추가

  • 현재 MemberCustomRepositoryImpl 에서 requirmentVerified 를 통해 가입조건을 충족시켰는지를 검사하고 있습니다.
  • 하지만 Member 도메인에는 isGrantAvailable() 메서드가 이미 존재하므로, 논리적으로 같은 의미의 로직이 레포지터리 단과 도메인 단에 중복하여 존재합니다.
  • 이렇게 하는 경우 DRY 원칙을 잘 지키지 못하게 되고 동일한 논리 정보가 소스코드에 중복되어 존재하게 되므로 유지보수성이 낮아집니다.
  • 그러므로 도메인 로직을 우선시하도록 개선...하려고 했지만 페이징이 걸려있어서 수정이 어렵습니다.
  • 따라서 도메인 로직을 수정할 때 레포지터리 쿼리 조건을 확인하도록 하기 위해 javadoc 참조 주석을 추가합니다.

이름 변경

  • 그리고 쿼리 메서드 이름이 논리적으로 더 의미를 담도록 개선합니다.
@uwoobeat uwoobeat added the ♻️ refactor 기능 개선 label Mar 6, 2024
@uwoobeat uwoobeat self-assigned this Mar 6, 2024
@uwoobeat uwoobeat added 📝 documentation 문서화 작업 and removed ♻️ refactor 기능 개선 labels Mar 6, 2024
@uwoobeat uwoobeat changed the title ♻️ 가입조건 충족 여부를 도메인 로직으로만 검증하도록 개선 📝 서로 연관되어 있는 쿼리 조건과 도메인 로직에 대한 주석 추가 Mar 6, 2024
@uwoobeat uwoobeat added ♻️ refactor 기능 개선 and removed 📝 documentation 문서화 작업 ♻️ refactor 기능 개선 labels Mar 6, 2024
@uwoobeat uwoobeat changed the title 📝 서로 연관되어 있는 쿼리 조건과 도메인 로직에 대한 주석 추가 ♻️ 쿼리 메서드 이름 개선 및 연관 로직 주석 추가 Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant