Лучший таймер для использования в службе Windows

Мне нужно создать службу Windows, которая будет выполняться каждые N периодов времени.
Возникает вопрос:
какой таймер мне следует использовать: System.Timers.Timerили System.Threading.Timerодин? На что-то влияет?

Я спрашиваю, потому что слышал много свидетельств некорректной работы System.Timers.Timerслужб Windows.
Спасибо.

Оба так System.Timers.Timerи System.Threading.Timerбудут работать на сервисы.

Таймеры, которых следует избегать, - это System.Web.UI.Timerи System.Windows.Forms.Timer, которые предназначены соответственно для приложений ASP и WinForms. Их использование приведет к загрузке службой дополнительной сборки, которая на самом деле не нужна для того типа приложения, которое вы создаете.

Используйте System.Timers.Timerследующий пример (также убедитесь, что вы используете переменную уровня класса, чтобы предотвратить сборку мусора, как указано в ответе Тима Робинсона):

using System;
using System.Timers;

public class Timer1
{
    private static System.Timers.Timer aTimer;

    public static void Main()
    {
        // Normally, the timer is declared at the class level,
        // so that it stays in scope as long as it is needed.
        // If the timer is declared in a long-running method,  
        // KeepAlive must be used to prevent the JIT compiler 
        // from allowing aggressive garbage collection to occur 
        // before the method ends. (See end of method.)
        //System.Timers.Timer aTimer;

        // Create a timer with a ten second interval.
        aTimer = new System.Timers.Timer(10000);

        // Hook up the Elapsed event for the timer.
        aTimer.Elapsed += new ElapsedEventHandler(OnTimedEvent);

        // Set the Interval to 2 seconds (2000 milliseconds).
        aTimer.Interval = 2000;
        aTimer.Enabled = true;

        Console.WriteLine("Press the Enter key to exit the program.");
        Console.ReadLine();

        // If the timer is declared in a long-running method, use
        // KeepAlive to prevent garbage collection from occurring
        // before the method ends.
        //GC.KeepAlive(aTimer);
    }

    // Specify what you want to happen when the Elapsed event is 
    // raised.
    private static void OnTimedEvent(object source, ElapsedEventArgs e)
    {
        Console.WriteLine("The Elapsed event was raised at {0}", e.SignalTime);
    }
}

/* This code example produces output similar to the following:

Press the Enter key to exit the program.
The Elapsed event was raised at 5/20/2007 8:42:27 PM
The Elapsed event was raised at 5/20/2007 8:42:29 PM
The Elapsed event was raised at 5/20/2007 8:42:31 PM
...
 */

Если вы выберете System.Threading.Timer, вы можете использовать следующее:

using System;
using System.Threading;

class TimerExample
{
    static void Main()
    {
        AutoResetEvent autoEvent     = new AutoResetEvent(false);
        StatusChecker  statusChecker = new StatusChecker(10);

        // Create the delegate that invokes methods for the timer.
        TimerCallback timerDelegate = 
            new TimerCallback(statusChecker.CheckStatus);

        // Create a timer that signals the delegate to invoke 
        // CheckStatus after one second, and every 1/4 second 
        // thereafter.
        Console.WriteLine("{0} Creating timer.\n", 
            DateTime.Now.ToString("h:mm:ss.fff"));
        Timer stateTimer = 
                new Timer(timerDelegate, autoEvent, 1000, 250);

        // When autoEvent signals, change the period to every 
        // 1/2 second.
        autoEvent.WaitOne(5000, false);
        stateTimer.Change(0, 500);
        Console.WriteLine("\nChanging period.\n");

        // When autoEvent signals the second time, dispose of 
        // the timer.
        autoEvent.WaitOne(5000, false);
        stateTimer.Dispose();
        Console.WriteLine("\nDestroying timer.");
    }
}

class StatusChecker
{
    int invokeCount, maxCount;

    public StatusChecker(int count)
    {
        invokeCount  = 0;
        maxCount = count;
    }

    // This method is called by the timer delegate.
    public void CheckStatus(Object stateInfo)
    {
        AutoResetEvent autoEvent = (AutoResetEvent)stateInfo;
        Console.WriteLine("{0} Checking status {1,2}.", 
            DateTime.Now.ToString("h:mm:ss.fff"), 
            (++invokeCount).ToString());

        if(invokeCount == maxCount)
        {
            // Reset the counter and signal Main.
            invokeCount  = 0;
            autoEvent.Set();
        }
    }
}

Оба примера взяты со страниц MSDN.

Не используйте для этого сервис. Создайте обычное приложение и создайте запланированную задачу для его запуска.

Это общепринятая передовая практика. Джон Гэллоуэй согласен со мной. Или, может быть, все наоборот. В любом случае, дело в том, что создание службы Windows для выполнения периодической задачи, запускаемой по таймеру, не рекомендуется.

«Если вы пишете службу Windows, которая запускает таймер, вам следует пересмотреть свое решение».

–Джон Галлоуэй, менеджер программы сообщества ASP.NET MVC, автор, супергерой по совместительству

Либо одно должно работать нормально. Фактически, System.Threading.Timer внутренне использует System.Timers.Timer.

Сказав это, легко злоупотребить System.Timers.Timer. Если вы не храните объект Timer где-нибудь в переменной, он может быть собран сборщиком мусора. Если это произойдет, ваш таймер больше не сработает. Вызовите метод Dispose, чтобы остановить таймер, или используйте класс System.Threading.Timer, который представляет собой более удобную оболочку.

Какие проблемы вы уже видели?

Я согласен с предыдущим комментарием, что, возможно, лучше всего рассмотреть другой подход. Я предлагаю написать консольное приложение и использовать планировщик Windows:

Это будет:

• Уменьшите количество кода, воспроизводящего поведение планировщика
• Обеспечение большей гибкости с точки зрения поведения планирования (например, запуск только по выходным), при этом вся логика планирования абстрагируется от кода приложения.
• Используйте аргументы командной строки для параметров без необходимости устанавливать значения конфигурации в файлах конфигурации и т. Д.
• Намного проще отлаживать / тестировать во время разработки
• Разрешить пользователю службы поддержки выполнять действия путем прямого вызова консольного приложения (например, полезно в ситуациях поддержки)

Как уже было сказано, оба так System.Threading.Timerи System.Timers.Timerбудут работать. Большая разница между ними в том, что System.Threading.Timerвокруг другого находится оболочка.

System.Threading.Timerбудет больше обработки исключений, в то время как System.Timers.Timerпроглотит все исключения.

В прошлом это доставляло мне большие проблемы, поэтому я всегда использовал System.Threading.Timer и все еще очень хорошо обрабатывал ваши исключения.

Я знаю, что эта ветка немного устарела, но она пригодилась для конкретного сценария, который у меня был, и я подумал, что стоит отметить, что есть еще одна причина, по которой System.Threading.Timerможет быть хороший подход. Когда вам нужно периодически выполнять задание, которое может занять много времени, и вы хотите, чтобы между заданиями использовался весь период ожидания, или если вы не хотите, чтобы задание запускалось снова до завершения предыдущего задания в случае, когда задание занимает больше времени, чем период таймера. Вы можете использовать следующее:

using System;
using System.ServiceProcess;
using System.Threading;

    public partial class TimerExampleService : ServiceBase
    {
        private AutoResetEvent AutoEventInstance { get; set; }
        private StatusChecker StatusCheckerInstance { get; set; }
        private Timer StateTimer { get; set; }
        public int TimerInterval { get; set; }

        public CaseIndexingService()
        {
            InitializeComponent();
            TimerInterval = 300000;
        }

        protected override void OnStart(string[] args)
        {
            AutoEventInstance = new AutoResetEvent(false);
            StatusCheckerInstance = new StatusChecker();

            // Create the delegate that invokes methods for the timer.
            TimerCallback timerDelegate =
                new TimerCallback(StatusCheckerInstance.CheckStatus);

            // Create a timer that signals the delegate to invoke 
            // 1.CheckStatus immediately, 
            // 2.Wait until the job is finished,
            // 3.then wait 5 minutes before executing again. 
            // 4.Repeat from point 2.
            Console.WriteLine("{0} Creating timer.\n",
                DateTime.Now.ToString("h:mm:ss.fff"));
            //Start Immediately but don't run again.
            StateTimer = new Timer(timerDelegate, AutoEventInstance, 0, Timeout.Infinite);
            while (StateTimer != null)
            {
                //Wait until the job is done
                AutoEventInstance.WaitOne();
                //Wait for 5 minutes before starting the job again.
                StateTimer.Change(TimerInterval, Timeout.Infinite);
            }
            //If the Job somehow takes longer than 5 minutes to complete then it wont matter because we will always wait another 5 minutes before running again.
        }

        protected override void OnStop()
        {
            StateTimer.Dispose();
        }
    }

    class StatusChecker
        {

            public StatusChecker()
            {
            }

            // This method is called by the timer delegate.
            public void CheckStatus(Object stateInfo)
            {
                AutoResetEvent autoEvent = (AutoResetEvent)stateInfo;
                Console.WriteLine("{0} Start Checking status.",
                    DateTime.Now.ToString("h:mm:ss.fff"));
                //This job takes time to run. For example purposes, I put a delay in here.
                int milliseconds = 5000;
                Thread.Sleep(milliseconds);
                //Job is now done running and the timer can now be reset to wait for the next interval
                Console.WriteLine("{0} Done Checking status.",
                    DateTime.Now.ToString("h:mm:ss.fff"));
                autoEvent.Set();
            }
        }