Как избежать одновременного вызова метода без блокировки?

Question

Я хочу запустить задачу очистки, которая может работать в течение нескольких секунд. Несколько потоков могут вызвать эту задачу, но я хочу запустить эту задачу только один раз. Все остальные вызовы должны просто пропустить.

Ниже приведена моя текущая реализация, но я не могу себе представить, что нет лучшего решения в структуре .net, что приводит к меньшему количеству строк кода.

object taskLock; 
    bool isRunning; 

    void Task() 
    { 
     if (isRunning) return; 

     try 
     { 
      lock (taskLock) 
      { 
       if (isRunning) return; 
       isRunning = true; 
      } 
      // Perform the magic 
     } 
     finally 
     { 
      isRunning = false; 
     } 
    } 

Answer 1

Да, есть лучшее решение. Вы можете использовать Interlocked.CompareExchange, код становится более простым и безблокировочным:

class Worker 
{ 
    private volatile int isRunning = 0; 

    public void DoWork() 
    { 
     if (isRunning == 0 && Interlocked.CompareExchange(ref isRunning, 1, 0) == 0) 
     { 
      try 
      { 
       DoTheMagic(); 
      } 
      finally 
      { 
       isRunning = 0; 
      } 
     } 
    } 

    private void DoTheMagic() 
    { 
     // do something interesting 
    } 
} 

В этом случае Interlocked.CompareExchange делает следующее в качестве атомарной операции (псевдо-код):

wasRunning = isRunning; 
if isRunning = 0 then 
    isRunning = 1 
end if 
return wasRunning 

Из документации MSDN:

public static int CompareExchange(
    ref int location1, 
    int value, 
    int comparand 
) 

Если comparand и значение в LOCATION1 равны, то значение хранится в местоположении1. В противном случае операция не выполняется. Сравнение и операции обмена выполняются как атомная операция. Значение возвращения CompareExchange это исходное значение в LOCATION1, принимает ли не обмен места