Как заблокировать таймер при обработке прошедшего события?

Question

У меня есть таймер, который не должен обрабатывать обработчик прошедших событий одновременно.Но обработка одного Прошедшего события мочь мешать другим.Я реализовал приведенное ниже решение, но что-то кажется неправильным;похоже, что либо я должен использовать таймер по-другому, либо использовать другой объект в пространстве потоков.Таймер, казалось, подходил лучше всего, потому что мне действительно нужно периодически проверять состояние, но иногда проверка занимает больше времени, чем мой интервал.Это лучший способ подойти к этому?

// member variable
private static readonly object timerLock = new object();
private bool found = false;


// elsewhere
timer.Interval = TimeSpan.FromSeconds(5).TotalMilliseconds;
timer.Elapsed = Timer_OnElapsed;
timer.Start();


public void Timer_OnElapsed(object sender, ElapsedEventArgs e)
{
  lock(timerLock)
  {
    if (!found)
    {
      found = LookForItWhichMightTakeALongTime();
    }
  }
}

Answer 1

Вы можете установить для AutoReset значение false, а затем явно сбросить таймер после того, как закончите его обработку. Конечно, то, как вы справитесь с этим, зависит от того, как вы ожидаете, что таймер будет работать. Выполнение этого способа позволит вашему таймеру отклониться от фактического заданного интервала (как при остановке и перезапуске). Ваш механизм позволил бы запускать и обрабатывать каждый интервал, но это может привести к отставанию необработанных событий, которые обрабатываются сейчас, когда истекает таймер, вызывающий вызов обработчика.

timer.Interval = TimeSpan.FromSeconds(5).TotalMilliseconds;
timer.Elapsed += Timer_OnElapsed;
timer.AutoReset = false;
timer.Start();


public void Timer_OnElapsed(object sender, ElapsedEventArgs e)
{
    if (!found)
    {
      found = LookForItWhichMightTakeALongTime();
    }
    timer.Start();
}

Answer 2

Обычно я останавливаю таймер во время его обработки, вхожу в блок try / finally и возобновляю таймер после завершения.

Answer 3

Если LookForItWhichMightTakeALongTime () займет много времени, я бы рекомендовал не использовать System.Windows.Forms.Timer , так как это заблокирует ваш пользовательский интерфейс нить и пользователь может убить ваше приложение, думая, что оно заморожено.

Вы могли бы использовать BackgroundWorker (вместе с Timer , если это необходимо).

public class MyForm : Form
{
  private BackgroundWorker backgroundWorker = new BackgroundWorker();

  public MyForm()
  {
    InitializeComponents();
    backgroundWorker.DoWork += backgroundWorker_DoWork;
    backgroundWorker.RunWorkerCompleted +=
                                backgroundWorker_RunWorkerCompleted;
    backgroundWorker.RunWorkerAsync();
  }

  private void backgroundWorker_DoWork(object sender, DoWorkEventArgs e)
  {
    e.Result = LookForItWhichMightTakeALongTime();
  }

  private void backgroundWorker_RunWorkerCompleted(object sender,
                                             RunWorkerCompletedEventArgs e)
  {
    found = e.Result as MyClass;
  }
}

И вы можете вызывать RunWorkerAsync () из любого места, где хотите, даже из Timer , если хотите. И просто убедитесь, что BackgroundWorker уже запущен, поскольку вызов RunWorkerAsync () во время его работы вызовет исключение.

private void timer_Tick(object sender, EventArgs e)
{
  if (!backgroundWorker.IsBusy)
    backgroundWorker.RunWorkerAsync();
}

Answer 4

timer.enabled = false

или

timer.stop();

и

timer.enabled = true

или

timer.start();

Answer 5

Я использую System.Threading.Timer примерно так

 class Class1
    {
        static Timer timer = new Timer(DoSomething,null,TimeSpan.FromMinutes(1),TimeSpan.FromMinutes(1));

        private static void DoSomething(object state)
        {
            timer = null; // stop timer

            // do some long stuff here

            timer = new Timer(DoSomething, null, TimeSpan.FromMinutes(1), TimeSpan.FromMinutes(1));
        }



    }