Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 PR 제목
오늘의 맛집 캐싱 도입
✅ 작업 내용
오늘의 맛집 API 호출시 오늘자 기준 등록된 모든 맛집의 상세 페이지 조회수가 0일 경우 빈 값을 반환하는 이슈 존재.
하여 캐싱 활용하여 3개 이상 맛집에 조회수 카운트시 오늘자 기준으로 맛집 반환하게 변경.
00:05에 어제 조회수 기반 랭킹 캐싱🎯 관련 이슈
💬 기타 공유하고 싶은 내용
getDailyRankingByViews를public으로 변경했는데 이렇게 하면 캡슐화가 깨지는 거 같고, 서비스의 내부 구현이 외부에 노출되는 것 같아서 고민인 부분이 있습니다. 그래서 랭킹 계산 로직을RankingProvider같은 별도의 클래스로 분리하는 방법을 생각해봤는데 이건 괜찮은 방법일까요??