-
카테고리
-
세부 분야
게임 프로그래밍
-
해결 여부
해결됨
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을 걸어주는게 좋을것 같다고 생각했는데
걸지 않아도 괜찮은건가요? 걸어주는게 맞는건가요?
혹시 제가 놓쳤거나 틀린 부분있으면 알려주시면 감사하겠습니다!!
감사합니다!!
답변을 작성해보세요.
1
Rookiss
지식공유자2021.03.23
Flush랑 Broadcast는 외부에서 멋대로 호출한다면
말씀주신 문제가 발생하겠지만,
일감으로 만들어서 JobQueue를 통해서 호출하기 때문에 안전합니다.
JobQueue가 lock을 대신한다고 보시면 됩니다.
0
답변 2