인프런 커뮤니티 질문&답변
작성한 질문수
JobQueue관련
작성
·
255
1
public void Push(Action job)
{
bool _flush=flase;
lock(_lock)
{
jobQueue.Enqueue(job);
if(flush==false)
_flush=flush=true;
}
if(_flush)
Flush();
}
위 함수의 일부인 flush 여부 확인 부분에서
if(flush==false)
_flush=flush=true;
저의 짧은 생각으로는 Push() 와 Pop() 메소드에서 jobQueue에 대한 배타적접근을 하므로
아래와 같이 flush의 false여부를 확인하고 바로 Flush()함수를 수행할지 결정해도 되지않을까 생각하는데, 제 생각의 어떤부분에 오류가 있는지 궁금합니다.
if(flush==false)
{
flush=true;
Flush();
}
답변 3
5
네 맞습니다. 그러니 lock(_lock) 안에 Flush가 있으니,
혹시라도 Flush 하는 부분이 오래 걸리면
다른 쓰레드들의 불필요한 대기가 길어진다는 의미였습니다.
2
미묘한 차이가 있는데 말씀하신대로 하면 락을 잡고 Flush를 하는 것이기 때문에,
혹시라도 Flush 과정이 오래걸리면,
다른 쓰레드들이 대기를 해야 합니다.
반면 첫번째 버전에서는 먼저 실행할 쓰레드만 가려내기 위해 잠시만 락을 잡은 후,
승자만 가려내고 락을 재빨리 풀어버리기 때문에
대기 시간이 상대적으로 짧아집니다.
참고로 if(flush==false) 가 아니고 if (_flush==false) 입니다! ㅎㅎ
0
public void Push(Action job)
{
bool _flush=flase;
lock(_lock)
{
jobQueue.Enqueue(job);
if(flush==false)
{
flush=true;
Flush();
}
}
}
이렇게 바꾸는걸 여쭤본거 였어요 ㅎㅎ