System.Threading.Timer у C #, схоже, не працює. Він працює дуже швидко кожні 3 секунди

У мене об’єкт таймера. Я хочу, щоб це працювало щохвилини. Зокрема, він повинен запускати OnCallBackметод і стає неактивним під час роботи OnCallBackметоду. Після OnCallBackзакінчення методу він (a OnCallBack) перезапускає таймер.

Ось що я зараз маю:

private static Timer timer;

private static void Main()
{
    timer = new Timer(_ => OnCallBack(), null, 0, 1000 * 10); //every 10 seconds
    Console.ReadLine();
}

private static void OnCallBack()
{
    timer.Change(Timeout.Infinite, Timeout.Infinite); //stops the timer
    Thread.Sleep(3000); //doing some long operation
    timer.Change(0, 1000 * 10);  //restarts the timer
}

Однак, схоже, це не працює. Він працює дуже швидко кожні 3 секунди. Навіть коли підняти період (1000 * 10). Схоже, це закриває очі1000 * 10

Що я зробив не так?

Це не правильне використання System.Threading.Timer. Коли ви створюєте таймер, ви майже завжди повинні робити наступне:

_timer = new Timer( Callback, null, TIME_INTERVAL_IN_MILLISECONDS, Timeout.Infinite );

Це доручить таймеру поставити галочку лише один раз, коли пройде інтервал. Потім у своїй функції зворотного виклику ви змінюєте таймер після завершення роботи, а не раніше. Приклад:

private void Callback( Object state )
{
    // Long running operation
   _timer.Change( TIME_INTERVAL_IN_MILLISECONDS, Timeout.Infinite );
}

Таким чином, немає необхідності в механізмах блокування, оскільки немає одночасності. Таймер запустить наступний зворотний виклик після закінчення наступного інтервалу + час тривалої операції.

Якщо вам потрібно запустити таймер рівно за N мілісекунд, то я пропоную вам виміряти час тривалої операції за допомогою секундоміра, а потім зателефонувати відповідним методом Change:

private void Callback( Object state )
{
   Stopwatch watch = new Stopwatch();

   watch.Start();
   // Long running operation

   _timer.Change( Math.Max( 0, TIME_INTERVAL_IN_MILLISECONDS - watch.ElapsedMilliseconds ), Timeout.Infinite );
}

Я настійно заохочую всіх, хто робить .NET, і використовує CLR, який не читав книгу Джефрі Ріхтера - CLR через C # , щоб прочитати її якнайшвидше. Таймери та басейни з нитками пояснюються там дуже детально.

Не потрібно зупиняти таймер, дивіться приємне рішення з цієї публікації :

"Ви можете дозволити таймеру продовжувати запускати метод зворотного дзвінка, але загортати код, який не реєструється, у Monitor.TryEnter / Exit. У цьому випадку не потрібно зупиняти / перезапускати таймер; виклики, що перекриваються, не отримають блокування та повернуться негайно."

private void CreatorLoop(object state) 
 {
   if (Monitor.TryEnter(lockObject))
   {
     try
     {
       // Work here
     }
     finally
     {
       Monitor.Exit(lockObject);
     }
   }
 }

Використовується System.Threading.Timerобов'язково?

Якщо ні, то System.Timers.Timerє зручні Start()та Stop()методи (і AutoResetвластивість, яку ви можете встановити на false, так що цей Stop()файл не потрібен, і ви просто зателефонуєте Start()після виконання).

Я просто зробив би:

private static Timer timer;
 private static void Main()
 {
   timer = new Timer(_ => OnCallBack(), null, 1000 * 10,Timeout.Infinite); //in 10 seconds
   Console.ReadLine();
 }

  private static void OnCallBack()
  {
    timer.Dispose();
    Thread.Sleep(3000); //doing some long operation
    timer = new Timer(_ => OnCallBack(), null, 1000 * 10,Timeout.Infinite); //in 10 seconds
  }

І ігноруйте параметр періоду, оскільки ви намагаєтеся самостійно контролювати періодику.

Ваш вихідний код працює так само швидко , як це можливо, так як ви зберігаєте вказівки 0для dueTimeпараметра. Від Timer.Change:

Якщо dueTime дорівнює нулю (0), негайно викликається метод зворотного виклику.

 var span = TimeSpan.FromMinutes(2);
 var t = Task.Factory.StartNew(async delegate / () =>
   {
        this.SomeAsync();
        await Task.Delay(span, source.Token);
  }, source.Token, TaskCreationOptions.LongRunning, TaskScheduler.Default);

source.Cancel(true/or not);

// or use ThreadPool(whit defaul options thread) like this
Task.Start(()=>{...}), source.Token)

якщо вам подобається використовувати певну петлю всередині ...

public async void RunForestRun(CancellationToken token)
{
  var t = await Task.Factory.StartNew(async delegate
   {
       while (true)
       {
           await Task.Delay(TimeSpan.FromSeconds(1), token)
                 .ContinueWith(task => { Console.WriteLine("End delay"); });
           this.PrintConsole(1);
        }
    }, token) // drop thread options to default values;
}

// And somewhere there
source.Cancel();
//or
token.ThrowIfCancellationRequested(); // try/ catch block requred.