Как правильно реализовать повторяющиеся действия через определенный промежуток времени?

Question

[WayMax]

Есть "функция" которая может выполняться 30 секунд, а может 30 минут. Поэтому промежуток времени должен быть не между двумя вызовами функции, а одним завершением функции и ее следующим вызовом. Пока это выглядит как-то так (это не реальный код, а пример, так что не нужно бомбить):

public class MyBackgroundWorker : BackgroundWorker
{
    public Boolean Terminated = false;

    protected override void OnDoWork(DoWorkEventArgs e)
    {
        while (true)
        {
            MyFunc(); // время выполнения данной функции неизвестно

            Thread.Sleep(60000);

            if (Terminated)
                return; 
        }
    }
}

В данном примере приходится ждать 10 минут для корректного завершения работы потока.
Этот вариант просто первое что пришло в голову.

Как вообще правильно реализовать повторяющиеся действия через определенный промежуток времени?

Answer 1

[Larry Underwood]

var timer = new System.Timers.Timer();
timer.Interval = 10000;
timer.Start();
timer.Elapsed += (sender, args) =>
{
  timer.Enabled = false;
  // action
  timer.Enabled = true;
};

Как то так

Comments

[WayMax]

Спасибо, посмотрю.

[WayMax]

Как я понял вместо "timer.Enabled = false" лучше один раз использовать "AutoReset = false".
А как сделать чтобы первый запуск функции происходил сразу после "timer.Start()" а не через заданный интервал? Если изменять значение интервала после вызова "timer.Start()" счетчик будет сбрасываться.

[Larry Underwood]

WayMax:
Action action = () => { };
var timer = new System.Timers.Timer { Interval = 10000 };
action(); // immediate invoke
timer.Elapsed += (sender, args) =>
{
timer.Enabled = false;
action(); // future invoke
timer.Enabled = true;
};
timer.Start();

Thread.Sleep() использовать не следует (оверхед на простое потока) хотя работать будет, тот вариант, который предлагает Петр тоже будет работать, но я бы тоже его не стал применять для этой цели (оверхед на поллинг сигнального состояния).

[Петр]

Максим Ванюшкин: А как System.Timers.Timer реализован? ))

[Larry Underwood]

Петр: ManualResetEvent уходит на ожидание в ядро, а System.Timers.Timer вызывается CLR и только тогда, когда его время подходит.

Answer 2

[Петр]

Ваш код тоже подойдет.
Только вместо Thread.Sleep(60000); лучше

waitEvent.WaitOne(60000);
// где
waitEvent = new ManualResetEvent(false);

Установите event в сигнальное состояние и ожидание прервется.

Comments

[WayMax]

Спасибо. Не подскажете, можно ли реализовать выход из цикла с помощью события?

[Петр]

WayMax: Terminated = true; waitEvent.Set();

[WayMax]

Петр: Я имел в виду выход из цикла без "if (Terminated) return;", чтобы вообще избавиться от переменной "Terminated"

[Петр]

WayMax: У вас несколько разных действий. Проверка и ожидание, одним не заменишь.