Unerwarteter Blockzustand mit Control.Invoke

Question

Der folgende Code ist ein vereinfachter Proof of Concept. In einer Windows Forms-Anwendung gibt es einen FormClosing-Ereignishandler, um eine Bereinigung durchzuführen. Die Anwendung startet einige Threads, die mit Control.Invoke auf das Formular schreiben. Ich weiß, dass ich Control.BeginInvoke verwenden könnte, aber ich möchte besser verstehen, was passiert und eine andere Lösung finden.

List<Thread> activeThreadlist; 
volatile bool goOnPolling = true; 
private void Form1_FormClosing(object sender, FormClosingEventArgs e) 
    { 
     goOnPolling = false; 
     Thread.Sleep(1000); 
     foreach (Thread element in activeThreadlist) 
      if (element.IsAlive) 
       element.Abort(); 

    } 

Ein Flag auf falsch gesetzt, um die Schleife auf den Fäden zu stoppen, sie haben Zeit zu beenden, und wenn man noch am Leben ist, wird er mit Abort beendet.

ist der von den anderen Threads ausgeführt Code:

while (goOnPolling) 
{ 
    //some long elaboration here 
    if (!goOnPolling) continue; 
    aControl.Invoke(new Action(() => 
    { 
     //display some result about the elaboration 
    })); 
    if (!goOnPolling) continue; 
    //some long elaboration here  

} 

In etwa 50% der Fälle, wenn die Form die Fäden Blöcke auf Control.Invoke geschlossen ist, so dass sie während der Ruhezeit nicht tun beenden und Abort wird aufgerufen. Ich dachte, die Flagge goOnPolling vor dem Aufrufen von Invoke wäre in 99,999% der Fälle ausreichend gewesen, aber ich lag falsch. Wie im Code erwähnt, tun die Threads lange Ausarbeitungen (mindestens 100ms), also erwartete ich goOnPolling war wahrscheinlich, während sie zu ändern. Warum liege ich falsch? Gibt es eine andere einfache Lösung, ohne zu BeginInvoke zurückzukehren, die einen zusätzlichen Thread erstellt?

aktualisieren

Nach dem Lesen der Kommentare ich den Titel klar war falsch. Ich schrieb ursprünglich Deadlock, aber es ist nur ein unerwarteter (für mich) Blockzustand. Am Anfang konnte ich nicht verstehen, warum meine Fäden noch am Leben waren, nachdem ich ihre Abschlüsse für 1000ms gewartet hatte, also setzte ich einen Abbruch, um es herauszufinden. Ich habe den Unterschied zwischen völlig falsch verstanden Invoke und BeginInvoke, danke für die Klärung.

@ Jon B

Ich stimme Pause ist besser geeignet als fortsetzen, aber dieser Code ist innerhalb while (goOnPolling) Block, so konnte ich nur einige CPU-Zyklen, nichts mehr retten.

Irgendeine Idee warum so oft goOnPolling Änderungen in den frühen Stadien von Invoke? Wenn ich eine lange Ausarbeitung mache, sollte es die meiste Zeit dort passieren.

Answer 1

Wir wissen, dass diese anderen Threads durch Invoke auf den UI-Thread zugreifen möchten, und wir wissen, dass Sie den Thread in einem Sleep binden.

Dies ist eine der wenigen Zeiten sein kann, wo Application.DoEvents die am wenigsten schlechte Option ist hier:

private void Form1_FormClosing(object sender, FormClosingEventArgs e) 
{ 
    goOnPolling = false; 
    for(int i=0;i<10;i++) 
    { 
     Application.DoEvents(); 
     Thread.Sleep(50); 
    } 
    //foreach (Thread element in activeThreadlist) 
    // if (element.IsAlive) 
    //  element.Abort(); 
} 

Und auch die Fäden mit wobei Ausnahmen geworfen zu bewältigen ändern, wenn sie zu Invoke versuchen, nachdem das Formular bereits geschlossen, anstatt es mit einem Abort niederträchtig zu versuchen.

---

Was warum so viele Threads in dem Zustand sind, wo sie Invoke wollen, sind, dann ist es aus dem Code unklar bisher geschrieben - aber bedenken, dass Invoke Zugriff auf den UI-Thread serialisiert. Es ist möglich, dass alle Threads aufgefordert, Invoke lange bevor der UI-Thread Verarbeitung die Windows-Nachricht, die das "Formular schließen" Ereignis übersetzt, wenn der Zugriff auf den Thread stark überlastet ist.