Anmutig Umgang Herunterfahren eines Windows-Dienstes

Nach mehr Forschung und einigem nachdenken kam ich zu erkennen, dass die Probleme, die ich bisher erlebt wurden verursacht durch eine sehr häufige design-Fehler in Bezug auf threads in Windows-Dienste.

Die Konstruktionsfehler

Stellen Sie sich vor Sie haben ein Gewinde, die nicht alle Ihre Arbeit. Ihre Arbeit besteht aus Aufgaben, die ausgeführt werden soll wieder und wieder auf unbestimmte Zeit. Dies ist oft wie folgt implementiert:

volatile bool keepRunning = true;
Thread workerThread;

protected override void OnStart(string[] args)
{
    workerThread = new Thread(() =>
    {
        while(keepRunning)
        {
            DoWork();
            Thread.Sleep(10 * 60 * 1000); //Sleep for ten minutes
        }
    });
    workerThread.Start();
}

protected override void OnStop()
{
    keepRunning = false;
    workerThread.Join();
    //Ended gracefully
}

Dies ist die sehr häufige design Fehler, die ich erwähnt. Das problem ist, dass während dieser wird kompiliert und ausgeführt, wie erwarteten, werden Sie schließlich erleben, dass Ihr Windows-Dienst reagiert nicht auf die Befehle der service-Konsole in Windows. Dieses ist, weil Ihre Aufruf-Thread.Sleep() blockiert den thread, dass Ihr Dienst nicht mehr reagiert. Sie wird nur auftreten dieses Fehlers, wenn der thread-Blöcke länger als der timeout konfiguriert, die von Windows HKLM\SYSTEM\CurrentControlSet\Control\WaitToKillServiceTimeout, weil dieser registry-Wert, den diese Umsetzung kann für Sie arbeiten, wenn dein thread ist konfiguriert, um zu schlafen für eine sehr kurze Zeit, und es ist das arbeiten in einer akzeptablen Zeit.

Die alternative

Anstelle von Thread.Sleep() habe ich beschlossen zu gehen für ManualResetEvent und System.Threading.Timer statt. Die Umsetzung sieht wie folgt aus:

OnStart:

this._workerTimer = new Timer(new TimerCallback(this._worker.DoWork));
this._workerTimer.Change(0, Timeout.Infinite); //This tells the timer to perform the callback right now

Callback:

if (MyServiceBase.ShutdownEvent.WaitOne(0)) //My static ManualResetEvent
    return; //Exit callback

//Perform lots of work here
ThisMethodDoesAnEnormousAmountOfWork();

(stateInfo as Timer).Change(_waitForSeconds * 1000, Timeout.Infinite); //This tells the timer to execute the callback after a specified period of time. This is the amount of time that was previously passed to Thread.Sleep()

OnStop:

MyServiceBase.ShutdownEvent.Set(); //This signals the callback to never ever perform any work again
this._workerTimer.Dispose(); //Dispose of the timer so that the callback is never ever called again

Abschluss

Durch die Implementierung von System.Threading.Timer und ManualResetEvent vermeiden Sie, dass Ihr Dienst immer nicht mehr auf die service console-Befehle als Ergebnis des Threads.Sleep() blockiert.

PS! Sie dürfen nicht aus dem Wald nur noch!

Ich glaube jedoch, es gibt Fälle, in denen ein callback zugeordnet ist, so viel Arbeit, die durch den Programmierer, dass der Dienst nicht mehr reagiert werden, um service-console-Befehle während der workload-Ausführung. Wenn das passiert, können Sie wünschen zu betrachten, alternative Lösungen, wie Sie die überprüfung Ihrer ManualResetEvent tiefer in Ihrem code, oder vielleicht der Umsetzung CancellationTokenSource.