• 카테고리

    질문 & 답변
  • 세부 분야

    게임 프로그래밍

  • 해결 여부

    해결됨

GameRoom class에서는 lock을 사용하지 않아도 멀티스레드 환경에서 안전할까요?

21.03.23 00:11 작성 조회수 159

0

안녕하세요 Rookiss님!

양질의 강의를 제공해주셔서 정말 감사드립니다!

질문) GameRoom class에서는 lock을 사용하지 않아도 멀티스레드 환경에서 안전할까요?

public void Flush()
{
	foreach (ClientSession s in _sessions)
		s.Send(_pendingList);
	//Console.WriteLine($"Flushed {_pendingList.Count} items");
	_pendingList.Clear();
}

public void Broadcast(ArraySegment<byte> segment)
{
	_pendingList.Add(segment);			
}

제 생각에 lock을 사용하지 않으면  _pendingList가 Clear되기 직전에 추가된 segment는 Send되지 못하고 삭제당하는 등

올바르지 못하게 동작하는 상황이 발생할 수 있을것 같았습니다.

(Flush method는 main Thread, BroadCast method는 background Thread. 서로 다른 Thread에서 동작한다고 생각했습니다.)

public void Flush()
{
	lock(_lock)
    {
		foreach (ClientSession s in _sessions)
			s.Send(_pendingList);
		//Console.WriteLine($"Flushed {_pendingList.Count} items");
		_pendingList.Clear();
    }
}

public void Broadcast(ArraySegment<byte> segment)
{
	lock(_lock)
    {
		_pendingList.Add(segment);			
    }
}

그래서  위처럼 lock을 걸어주는게 좋을것 같다고 생각했는데

걸지 않아도 괜찮은건가요? 걸어주는게 맞는건가요?

혹시 제가 놓쳤거나 틀린 부분있으면 알려주시면 감사하겠습니다!!

감사합니다!!

답변 2

·

답변을 작성해보세요.

1

Flush랑 Broadcast는 외부에서 멋대로 호출한다면
말씀주신 문제가 발생하겠지만,
일감으로 만들어서 JobQueue를 통해서 호출하기 때문에 안전합니다.

JobQueue가 lock을 대신한다고 보시면 됩니다.

0

흥푸님의 프로필

흥푸

질문자

2021.03.23

앗 정말 감사합니다

생각해보니 전에 수업에서도 말씀해주셨던것 같기도 한데

오랜만에 복습하다보니 헷갈렸습니다.

감사합니다! 즐거운 하루 보내세요!!