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

feat : 숨캐찾 정답 처리 로직 캐싱 적용 및 정답 관련 정보를 coord -> position으로 수정 (CC-159) #34

Merged
merged 7 commits into from
Aug 15, 2024

Conversation

j2noo
Copy link
Collaborator

@j2noo j2noo commented Aug 14, 2024

📝 작업내용

  • 숨캐찾 정답 정보를 int coord -> double position으로 수정
  • 현재 시간 기준 숨캐찾 게임 조회, 정답 조회하는 레포지토리 레벨 메서드 캐싱
  • 정답 처리 로직 약간 변경

💬 침고사항

  • FindingGamePlayService의 87번째줄에, 정답을 1개 맞췄으면 correctList.remove() 하는 코드가 있음.
    캐싱을 서비스레벨의 메서드가 아닌 레포지토리레벨의 메서드에 걸어 놓았는데, 위의 코드 때문에 버그가 발생함
    • 정답을 1개만 맞춘 경우, correctList.remove()의 코드 때문에 변화가 감지?되어, 같은 요청을 한번 더 보냈느데 correctList가 2개가 아니라 1개가 반환되어버림. 따라서 일단 remove하는 코드를 제거함.
    • 두 개의 정답이 서로 오차범위(0.1) 내에 겹치는 부분이 없다고 가정하면 로직상 문제는 없어보임.

📖 레퍼런스

@j2noo j2noo added 진우 feat📌 기능 개발 labels Aug 14, 2024
@j2noo j2noo self-assigned this Aug 14, 2024
@j2noo
Copy link
Collaborator Author

j2noo commented Aug 15, 2024

deep copy도 확인 후 수정하겠습니다

@putdata putdata merged commit 265eace into develop Aug 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat📌 기능 개발 진우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants