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

게임 결과창에서 Back to Lobby 클릭 시 & 게임중 탈주 시 발생하는 에러 해결 #134

Merged
merged 2 commits into from
Jul 14, 2024

Conversation

forrest1398
Copy link
Collaborator

@forrest1398 forrest1398 commented Jul 14, 2024

Closes #133

PR 타입 (하나 이상의 PR 타입 선택)

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 버그 수정
  • 설정 변경 (Config Class)
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 배포환경
  • 문서 수정 (ex. README)

반영 브랜치

#133

반영 사항

  1. 게임 결과창에서 Back to Lobby 클릭 시
    • Redis에서 나가는 유저를 찾지 못하는 예외 발생
    • 예외로 인해 이후 로직이 수행되지 않음
    • 실제 사용자는 로비로 이동했지만, 게임에도 존재하는 버그 발생
    • 원인
      • Redis에서 유저를 찾을 때, 닉네임, 프로필이미지URL, 전적을 같이 사용
      • 결과창에서 전적이 변경되는 api 호출 -> DB와 Redis 사이의 정합성이 깨짐
    • 해결
      • Redis에서 유저를 찾을 때, 닉네임만을 사용해서 찾도록 변경
  2. 게임중 탈주 시
    • 클라이언트에서 NEXT가 빈 배열( "[ [ ] ]" ) 처리를 못하고 에러박스 생성하는 버그 발생
    • 해결
      • 응답값을 회색으로 채운 배열로 변경

유의 사항

관련이슈

#133

참여자

Comment on lines +63 to +70
String[][] emptyBoard = {{"#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "orange", "#303030", "#303030", "#303030", "#303030", "#303030"},
{"#303030", "#303030", "#303030", "#303030", "orange", "#303030", "#303030", "#303030", "#303030", "#303030"},
Copy link
Member

Choose a reason for hiding this comment

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

무엇을 위한 상수값인가요?

@seonghoo1217 seonghoo1217 merged commit 65ebe12 into dev Jul 14, 2024
2 checks passed
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 this pull request may close these issues.

게임 결과창에서 Back to Lobby 버튼 동작 버그를 해결한다.
2 participants