[BE] fix-room-clean

[DongHoonYu96]

🔎 작업 내용

이 Pull Request는 Redis 기반 게임 서비스에서 비활성 방 처리와 정리 프로세스를 개선하기 위한 여러 업데이트를 포함합니다. 가장 중요한 변경사항은 비활성 방 확인을 위한 새로운 메소드 추가, 방 정리 프로세스 수정, 그리고 중복 코드 제거입니다.

• 비활성 방 처리 개선:
  BE/src/game/redis/game-redis-memory.service.ts: cron 작업을 사용하여 주기적으로 비활성 방을 확인하고 정리하는 새로운 메소드 checkInactiveRooms를 추가했습니다. 이 메소드는 Redis SCAN을 사용하여 대규모 방 세트를 안전하게 처리하고 배치 단위로 처리합니다.

• 방 정리 프로세스 강화:
  BE/src/game/redis/subscribers/room.cleanup.subscriber.ts: 방 정리 프로세스를 업데이트하여 방 데이터 자체뿐만 아니라 해당 방의 플레이어 목록을 가져와서 그들의 데이터도 삭제하도록 수정했습니다.

• 코드 정리 및 리팩토링:
  BE/src/game/service/game.room.service.ts: GameRedisMemoryService의 새로운 구현으로 대체되었기 때문에 기존의 checkInactiveRooms 메소드와 관련 cron 작업을 제거했습니다.

• 사소한 변경사항:
  BE/src/game/redis/game-redis-memory.service.ts: 일관된 구조를 따르도록 imports 순서를 재정렬했습니다.
  BE/src/game/service/game.room.service.ts: 사용하지 않는 import 구문을 제거했습니다.

🖼 참고 이미지

• 퇴장전
  
• 마지막 플레이어 퇴장후
  

🎯 리뷰 요구사항 (선택)

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

✅ Check List

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 suggestion.

Comments skipped due to low confidence (1)

BE/src/game/redis/game-redis-memory.service.ts:25

• There is a redundant comment for the checkInactiveRooms method. The comment on line 25 can be removed.

/**

The comment has a minor spelling issue. It should be '200명 미만 -> smembers 사용!' instead of '200명미만 -> smembers 사용!'.

Suggested change

	// 1. 방에 속한 플레이어 목록 가져오기, 200명미만 -> smembers 사용!
	// 1. 방에 속한 플레이어 목록 가져오기, 200명 미만 -> smembers 사용!

[NewCodes7]

👍👍

[NewCodes7]

P3: try/catch 바깥에 하나만 둬도 될 것 같습니다!
(취소)

[NewCodes7]

아 그런데 바깥에 하나만 두면 rooms.map 돌다가 에러 났을 때 다음 커서로 못 가겠네요.

그러면 그 대신에 아래 부분을 함수로 분리하는 건 어떨까요? depth가 너무 깊어보여서요!

async (roomId) => {
  try {
    const lastActivity = await this.redis.hget(REDIS_KEY.ROOM(roomId), 'lastActivityAt');

    if (lastActivity && now - parseInt(lastActivity) > this.INACTIVE_THRESHOLD) {
      await this.redis.publish('room:cleanup', roomId);
      this.logger.verbose(`비활성으로 인해 방 ${roomId} 정리 시작`);
      processedCount++;
    }
  } catch (error) {
    this.logger.error(`방 ${roomId} 처리 중 오류 발생: ${error.message}`);
  }
})

[songbuild00]

👍

[ijun17]

LGTM!