Feat: 러닝 데이터의 평균 값을 구하는 함수를 Repository에 추가

[hee9841]

🔗 이슈 연결

• 180

🚀 구현한 API

• X

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

• 기간 내 달린 평균 거리 값을 조회하는 함수를 Repository에 추가합니다.
• 기간 내 달린 평균 시간 값을 조회하는 함수를 Repository에 추가합니다.

🔍 리뷰 요청/참고 사항

[hee9841]

가독성 측면이나, 코드의 명확성 측면에서 지금 방식이 괜찮을 것 같은데

새로운 repository 함수 대신 service 계층에서 2주 동안의 데이터를 조회

혹시 이렇게 하는 방식을 추천하신 이유가 있으신가요?

[Jaewon-pro]

만약 2주치 데이터를 불러온다면 추가적인 함수나 테스트를 구현하지 않아도 될 것 같아서요!

그리고 함수 호출도 1번만 하니 네트워크 비용을 줄일 수 있고 평균을 구하는 로직은 디비가 아닌 was에서 실행하면 디비 부담도 줄일 수 있을 거라고 생각했어요

[hee9841]

네트워크 비용적인 측면에서 그렇게 생각해볼 수 있군요 👍👍

하지만 저희가 초기 사용자가 매우 적을 것으로 예상이 되는데요. 그래서 초기에는 성능적인 부담이 적어서 코드의 가독성과 유지보수성을 좀 더 우선시하는게 좋다고 개인적으로 생각합니다.

만약 운영하면서 사용자가 늘어나면, 그 때 가서 이런 성능에 대한 최적화나 비용에 대해서 생각하는게 좋을 것 같다고 생각하는데 혹시 어떻게 생각하시나요??