인프런 커뮤니티 질문&답변

안녕하세요 루키스님
part4 강의로 만들어진 서버와
part5에서 만들어진 클라이언트를
조금 수정해가며 추가 기능을 만들던 중
문제가 생겼는데 해결이 잘 안되어서 질문드립니다.

우선 추가하려는 기능은 강의에서 배운 Q를 눌러서 종료하는 기능 외에
플레이어(3명)들이 접속해있는 상태에서 언리얼 편집창을 닫아버렸을 때
3명 모두 접속이 끊어지며 플레이어들을 서버에서 제거해주는 기능을 추가하려고 합니다..

이 때, 발생하는 문제는
player가 3명이고 objectId가 각각 1번부터 3번까지 있다고 했을 때,
어쩔 땐 player들 모두 제거되고,
어쩔 땐 player(2번), player(1번), player(1번)처럼
objectId가 1번인 플레이어를 제거하는 처리가 중복 호출되며
2번과 1번에 대한 삭제만 이루어지고,
3번에 대한 삭제가 이루어지지 않는 문제가 발생합니다.

아래는 작성한 코드입니다.
먼저 GameSession::OnDisconnected()의 맨 마지막 줄에 GRoom->DoAsync()로
Room::HandleLeavePlayer를 호출해주는 코드를 추가했고,

이 때 들어오는 player들의 objectId들을 확인해봤을 때
1, 2, 3 모두 중복 없이 들어오는 것을 확인했습니다.

void GameSession::OnDisconnected()
{
	if (player.load() == nullptr)
	{
		return;
	}
	cout << "objectId: " << player.load().get()->objectInfo->object_id() << endl;

	RoomRef room = player.load()->room.load().lock();
	if (room == nullptr)
	{
		return;
	}

	GRoom->DoAsync(&Room::HandleLeavePlayer, player.load());
}

다음은 Room::LeaveRoom()에서 위에서 말한 상황일 때
파라미터로 들어오는 object가 중복으로 들어오는 걸 확인했습니다.
if(!success) { return success; }는
이미 삭제된 object를 중복으로 RemoveObject(objectId)를 할 경우 false를 돌려주는데
밑에서 패킷 보내는 작업을 넘겨버리기 위해 추가했습니다.
그리고 코드 거의 마지막 부분에
GSessionManager.Remove(session)로

세션을 제거하는 코드를 추가했습니다.

bool Room::LeaveRoom(ObjectRef object)
{
	if (object == nullptr)
	{
		return false;
	}

	const uint64 objectId = object->objectInfo->object_id();
	bool success = RemoveObject(objectId);
	if (!success)
	{
		return success;
	}

	// 퇴장할 플레이어에게 퇴장 패킷 전송
	if (object->IsPlayer())
	{
		PlayerRef player = static_pointer_cast<Player>(object);

		Protocol::S_LEAVE_GAME leavePkt;

		SendBufferRef sendBuffer = ClientPacketHandler::MakeSendBuffer(leavePkt);
		if (GameSessionRef session = player->session.lock())
		{
			session->Send(sendBuffer);
		}
	}

	// 남아있는 플레이어들에게 퇴장 플레이어의 디스폰 패킷 전송
	{
		Protocol::S_DESPAWN despawnPkt;
		despawnPkt.add_object_ids(objectId);

		SendBufferRef sendBuffer = ClientPacketHandler::MakeSendBuffer(despawnPkt);
		Broadcast(sendBuffer, objectId);

		// 퇴장 플레이어는 이미 room에서 빠져나간 상태라 Broadcast 불가
		if (object->IsPlayer())
		{
			PlayerRef player = static_pointer_cast<Player>(object);
			if (GameSessionRef session = player->session.lock())
			{
				session->Send(sendBuffer);
				GSessionManager.Remove(session);
			}
		}
	}

	return success;
}

GameSession 소멸자

virtual ~GameSession() override
{
	cout << "~GameSession" << endl;
	player.store(shared_ptr<Player>());
}

Player 소멸자

Player::~Player()
{
	cout << "~Player" << endl;
	session.reset();
}

다음은 첫번째 실행 화면입니다.
(실행 화면에 출력되는 줄이 뒤죽박죽이라 서로 다른 색으로 구별 해봤습니다..ㅠ)
연두색으로 묶은 범위는 언리얼 에디터를 1번 실행했다가 닫았을 때의 단위로 묶었습니다.
빨간색으로 밑줄 그어진 objectId: 숫자로 출력되는 부분은
GameSession::OnDisconnected()에서 출력하고 있고,
노란색으로 밑줄 그어진 Remove Objedct숫자로 출력되는 부분은
Room::RemoveObject()에서 objectId를 출력하고 있습니다.

첫번째 실행 화면에서는 처음 실행하고 바로 해당 문제가 발생했습니다.
objectId: 3, objectId: 2, objectId: 1 순으로 중복 없이 잘 들어왔는데
Remove Object3, Remove Object1, Remove Object1 처럼 출력되었고,
2번 플레이어에 대한 제거가 되지 않았습니다.
때문에 두번째, 세번째 실행에서는 접속한 3명의 플레이어가 모두 잘 제거됨에도
처음에 제거되지 않은 플레이어 하나가 남아서
언리얼 에디터 실행 화면에 보이는 플레이어가 총 4명이었습니다.

다음은 두번째 출력 화면입니다.

두번째 실행 화면에서는 5번째가 되어서야 해당 문제가 발생했습니다.
그 전까지는 문제 없이 3명의 플레이어 모두 잘 제거되었습니다.
마찬가지로 objectId: 13, objectId: 14, objectId: 15로 들어왔고
Remove Object13, Remove Object15, Remove Object15로 실행되면서
13, 15번의 플레이어만 제거되고 14번 플레이어는 제거되지 않았습니다.

실제로 문제가 발생할 경우의 호출 스택을 따라가 봤을 때 OnDisconnected()에서
GRoom->DoAsync(&Room::HandleLeavePlayer, player.load())로
삭제 작업 예약 거는 것 까지는 플레이어가 중복 없이 잘 들어가는데

예약했던 Room::HandleLeavePlayer()가 호출되고
파라미터로 들어오는 플레이어의 objectId를 확인해보면
이미 삭제된 플레이어가 한번 더 들어오는 경우가 생기네요.

혹시 어떤 부분 때문일까요??