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

[Refact & Feat] MiniGame 종료 시 판정 구현 및 버그 수정 #36

Merged
merged 11 commits into from
Jul 8, 2024

Conversation

kamothi
Copy link
Member

@kamothi kamothi commented Jul 7, 2024

✏️ 작업 개요

미니게임 종료 시 판정과 결과 저장에 대한 기능 구현

⛳ 작업 분류

  • Status가 Over일 경우 핸들러 구현
  • WebSocket Test 코드 작성(아직 Over시는 못함)
  • 불필요한 패킷, 함수 삭제 및 버그 수정

🔨 작업 상세 내용

  1. 미니게임과 관련된 버그 및 Over 시의 기능 구현

💡 생각해볼 문제

  • 현재 GameRoom이나 GameRoomManager, Matching 등 다양한 함수들이 존재하고 얽혀있습니다. 테스트 코드를 작성할 때 보니 생각보다 꽤 얽혀 있어서 조금 더 나은 방향으로 클래스들을 구분하고 다시 만들어야하나 고민 중입니다.

Copy link

github-actions bot commented Jul 7, 2024

Test Results

16 tests   16 ✅  7s ⏱️
 6 suites   0 💤
 6 files     0 ❌

Results for commit 83a6f6a.

Copy link
Member

@david-parkk david-parkk left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 현재 GameRoom 관련하여 Caused by: reactor.netty.channel.AbortedException: Connection has been closed BEFORE send operation error 로그 배포환경에서 발생하는데 이것도 ec2 성능 문제인지 모르겠네요

Copy link
Member

Choose a reason for hiding this comment

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

현재 로직은 아직 db 처리가 안되 있는 것으로 알고 있는데 나중에 db 로직이 추가되었을 경우, ci 환경에서 오류가 발생할 수 있습니다.(ci 환경문제)

Copy link
Member Author

Choose a reason for hiding this comment

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

저도 현재 그 문제 때문에 db 부분은 주석처리를 해두었습니다. 한 번 해결해보겠습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

고생하셨습니다. 현재 GameRoom 관련하여 Caused by: reactor.netty.channel.AbortedException: Connection has been closed BEFORE send operation error 로그 배포환경에서 발생하는데 이것도 ec2 성능 문제인지 모르겠네요

해당 부분의 경우 어디서 발생하는 건가요? 혹시 이미지 첨부 가능하신가요.

@david-parkk david-parkk merged commit 3f6228e into main Jul 8, 2024
3 checks passed
@kamothi kamothi deleted the minigame-update branch July 11, 2024 12:20
@david-parkk
Copy link
Member

/쿠타버스 배포

Copy link

🌎 배포하였습니다.

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.

2 participants