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

안녕하세요 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을 걸어주는게 좋을것 같다고 생각했는데

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

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

감사합니다!!