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: 러닝 레코드 Repository 리팩터링 #297

Merged
merged 8 commits into from
Nov 6, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Nov 5, 2024

🔗 이슈 연결

💡 반영할 내용 및 변경 사항 요약

  • 기간 안의 데이터를 조회하는 함수명을 하나로 통일 했습니다.
    • 기존의 ByDateRange, AndDateRange 등 여러개로 나타나있던 접미사를 WithDateRage로 변경했습니다.
  • 리파지토리의 함수 순서를 다음과 같이 변경했습니다.
    • find -> update -> save -> delete 순, jpa -> jooq 순
  • 날짜 관련 없이 사용자에 따라 전체 러닝 거리 합을 조회하는 함수가 여러 곳에서 사용해서 사용자 id에 따라 전체 러닝 거리 합을 조회하는 리파지토리 함수 findTotalDistanceMeterByMemberId를 추가했습니다.
  • 기존에 사용자의 전체 러닝 거리를 조회하는 로직에서 findTotalDistanceMeterByMemberIdWithRangeDatefindTotalDistanceMeterByMemberId로 변경했습니다.

🔍 리뷰 요청/참고 사항

  • 사용자에 따라 전체 러닝 거리 합을 조회하는 함수가 여러 곳에서 필요할 것 같아서 따로 만들었는데 혹시 괜찮은가요?
  • 리파지토리의 함수들이 늘어나면서 한번에 보기 힘들어 지는 것 같아요! 함수의 순서를 다음 백엔드 스크럼때 같이 정하면 좋을 것 같아요!

@hee9841 hee9841 added the refactor 코드 리팩토링 label Nov 5, 2024
@hee9841 hee9841 self-assigned this Nov 5, 2024
@hee9841 hee9841 requested a review from WonSteps November 5, 2024 13:35
Copy link
Member

@WonSteps WonSteps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

사용자에 따라 전체 러닝 거리 합을 조회하는 함수가 여러 곳에서 필요할 것 같아서 따로 만들었는데 혹시 괜찮은가요?

좋습니다 👍

리파지토리의 함수들이 늘어나면서 한번에 보기 힘들어 지는 것 같아요! 함수의 순서를 다음 백엔드 스크럼때 같이 정하면 좋을 것 같아요!

리포지터리 함수들이 많아지긴 했는데, 말씀하신 순서대로 추가/수정하면 좋을 것 같아요!

@hee9841 hee9841 merged commit 51075f1 into main Nov 6, 2024
1 check passed
@hee9841 hee9841 deleted the refactor/running-record-refactor branch November 6, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants