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

[FE] feat#27#221#222 서바이벌 모드 구현 및 게임 종료 후 호스트 변경 구현 #223

Merged
merged 13 commits into from
Nov 20, 2024

Conversation

ijun17
Copy link
Collaborator

@ijun17 ijun17 commented Nov 19, 2024

➕ 이슈 번호


🔎 작업 내용

  • 서바이벌 기능 구현
    • 현재 탈락한 플레이어를 렌더링하지 않고 있습니다
    • 단, 본인이 탈락한 경우 반투명 상태로 렌더링 됩니다
  • 호스트 변경
  • 소켓 모킹에 유틸 함수들 추가(준우님 보시면 좋을것 같습니다)
  • useServerDate 훅을 getServerTimestamp 유틸 함수로 변경
    • 주기?적으로 서버 시간과의 offset을 평균으로 계산

🖼 참고 이미지

image


🎯 리뷰 요구사항 (선택)

  • 특별히 봐줬으면 하는 부분이 있다면 적어주세요

✅ Check List

  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

- 주기적으로 요청하여 offset의 평균값을 계산
Copy link
Collaborator

@NewCodes7 NewCodes7 left a comment

Choose a reason for hiding this comment

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

준기님도 어제 수고하셨습니다!!
오늘 서바이벌 모드 시연가능한가요? ㅎㅎ

Comment on lines +17 to +19
offsetTotal += clientTime - res.serverTime;
offsetCount++;
offset = Math.floor(offsetTotal / offsetCount);
Copy link
Collaborator

Choose a reason for hiding this comment

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

offset을 평균으로 계산하고 있군요!!
네트워크 시간이 매번 달라질 수 있기 때문에 그런거죠? 좋습니다!

Copy link
Collaborator

@songbuild00 songbuild00 left a comment

Choose a reason for hiding this comment

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

좋습니다! 소켓 모킹 코드 리팩토링도 신경 쓰신 부분 좋았습니다 👍🏻

@@ -70,3 +72,7 @@ socketService.on('endQuizTime', (data) => {
useQuizeStore.getState().setQuizState(QuizState.END);
useQuizeStore.getState().setCurrentAnswer(Number(data.answer));
});

socketService.on('endGame', () => {
useQuizeStore.getState().resetQuiz();
Copy link
Collaborator

Choose a reason for hiding this comment

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

지금 봤는데, useQuizeStore 중간에 e는 오타인가요?!

Copy link
Collaborator

@always97 always97 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 ! LGTM

@ijun17 ijun17 merged commit dd1cfa9 into boostcampwm-2024:dev-fe Nov 20, 2024
@ijun17 ijun17 deleted the feature-fe-#27#221#222 branch December 5, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants