Skip to content

Conversation

@sehee123
Copy link
Collaborator

@sehee123 sehee123 commented Aug 3, 2025

🛰️ Issue Number

🪐 작업 내용

  • AOP로 적용할 수 없는 부분들이 있어서 수동 실행하는 LockExecutor 를 생성
  • userId와 roomId 모두 락이 필요한 경우들에 있어서는 userId -> roomId 로 순서를 일관되게 유지
  • 내부 호출이 얽혀있는 상황에서는 처음 호출할 때 락을 걸어서 진행
image
  • userId로 roomId를 가져오면 타이밍 이슈로 새로 입장하는 방 roomId를 가져와서 disconencted 처리
  • sessionId ,roomId 매핑되어있는 맵 생성하여 게임방 중복입장을 방지 (handleDisconnectedListener 이전방 roomId가져오기)

📚 Reference

✅ Check List

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

sehee123 and others added 4 commits July 30, 2025 22:41
# Conflicts:
#	backend/src/main/java/io/f1/backend/domain/game/app/RoomService.java
#	backend/src/main/java/io/f1/backend/domain/game/websocket/controller/GameSocketController.java
#	backend/src/main/java/io/f1/backend/domain/game/websocket/eventlistener/WebsocketEventListener.java
@sehee123 sehee123 self-assigned this Aug 3, 2025
@sehee123 sehee123 added the refactoring 코드 리팩토링 label Aug 3, 2025
@sehee123 sehee123 linked an issue Aug 3, 2025 that may be closed by this pull request
1 task
@LimKangHyun LimKangHyun self-requested a review August 3, 2025 16:28
Copy link
Collaborator

@LimKangHyun LimKangHyun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Collaborator

@silver-eunjoo silver-eunjoo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Collaborator

@jiwon1217 jiwon1217 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@sehee123 sehee123 merged commit d9c79cb into dev Aug 4, 2025
@sehee123 sehee123 deleted the refactor/163 branch August 4, 2025 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[refactor] enter,exit 동시성 적용

5 participants