Ausnahme beim schließen der Form (thread + aufrufen)

Ich habe gerade angefangen zu lernen, über threads und methodinvoking in c#, aber ich habe ein problem, welches ich nicht finden konnte, die Lösung der.

Machte ich einen basic-C# - form-Programm, das fortlaufend aktualisiert wird und die Anzeige eine Zahl, die durch die Gründung eines thread, und rufen Sie delegieren.

Starten neuen thread auf Form1_load:

private void Form1_Load(object sender, EventArgs e)
  {
        t = new System.Threading.Thread(DoThisAllTheTime);
        t.Start();
  }

Public void DoThisAllTheTime (was hält die Aktualisierung der Anzahl) :

public void DoThisAllTheTime()
  {
     while(true)
      {
        if (!this.IsDisposed)
         {
           number += 1;
           MethodInvoker yolo = delegate() { label1.Text = number.ToString(); };
           this.Invoke(yolo);
         }
      }
  }

Wenn ich nun auf das X-button des Formulars bekomme ich folgende exception:

"Eine nicht behandelte Ausnahme des Typs 'System.ObjectDisposedException " ist in System.Windows.Forms.dll

Kann nicht aktualisiert werden ein gelöschtes Objekt'

Während ich tatsächlich überprüfen, ob die form entsorgt wurde oder nicht.

EDIT: ich Hinzugefügt catch (ObjectDisposedException ex -) mit dem code, der das problem behoben.
Arbeiten code:

  public void DoThisAllTheTime()
  {
     while(true)
      {
         number += 1;

         try {  
              MethodInvoker yolo = delegate() { label1.Text = number.ToString(); };
              this.Invoke(yolo);
             }
         catch (ObjectDisposedException ex)
             {
              t.Abort();
             }
      }
 }
  • Gut, die form bekam entsorgt nach (!this.IsDisposed) berechnet wurde, aber bevor this.Invoke(yolo); genannt wurde. Willkommen in der Welt der Rennen.
  • Ich habe deine bearbeiteten Titel. Bitte sehen, "Sollten Fragen enthalten "tags" im Titel?", wo der Konsens ist ", Nein, Sie sollten nicht".
  • +1 für die variable yolo
  • Wie würde ich das beheben? Ich nahm an, die zu überprüfen wäre schneller als die methodinvoke. Es wird auch verwendet, auf der MSDN-Website, dass Art und Weise. ( msdn.microsoft.com/en-us/library/... )
  • Wie andere schon sagten, dies ist ein Rennen-problem siehe: stackoverflow.com/questions/1874728/...
  • zählen Sie nicht auf MSDN-Beispielcode, die richtige. In der Regel ist es das nicht.

InformationsquelleAutor John | 2013-10-27
  1. 3

    Ihren Anruf this.IsDisposed ist immer veraltet. Sie müssen zum abfangen von Ihrer form-closing-Ereignis aus und beenden Sie den thread explizit. Dann werden Sie nicht haben zu tun, dass IsDisposed test überhaupt.

    Gibt es viele Möglichkeiten, wie Sie dies tun können. Persönlich würde ich die System.Threading.Tasks namespace, aber wenn Sie möchten, um Ihre Nutzung der System.Threading sollten Sie definieren eine member-variable _updateThread, und starten Sie es in Ihrem load-Ereignis:

    _updateThread = new System.Threading.Thread(DoThisAllTheTime);
_updateThread.Start();

    Dann in Ihre Abschlussveranstaltung:

    private void Form1_Closing(object sender, CancelEventArgs e)
{
    _stopCounting = true;
    _updateThread.Join();
}

    Schließlich, ersetzen Sie den IsDisposed test mit überprüfung auf den Wert Ihres neuen _stopCounting member-variable:

    public void DoThisAllTheTime()
{
    MethodInvoker yolo = delegate() { label1.Text = number.ToString(); };
    while(!_stopCounting)
    {
        number += 1;
        this.Invoke(yolo);
    }
}
    • Ich habe versucht, das hinzufügen von 't.abort (); " auf Form1_closing, aber das hat auch nicht geholfen.
    • Versuchen Sie, eine Flagge und rufen dann t.Join(); zu warten, bis der thread hat erkannt, es.
    • Ich fügte catch (ObjectDisposedException ex) zu. Dies das problem behoben. Ich update meine Frage.
    • Als oefe gesagt, das ist abscheulich! 😀
    • Und in der Tat, brauchen Sie nicht zu definieren, eine neue Stellvertretung in jeder Phase des Loops-das wird Ihr garbage collector viel härter arbeiten, als es nötig ist. Fügen Sie stattdessen eine Methode, um Ihre Klasse einen Aufruf.
    • Da Invoke blockiert wartet der Haupt-thread, gibt es eine Möglichkeit für einen deadlock, wenn der Form1_Closing Methode ausgeführt wird auf der Linie number += 1;. Die Wahrscheinlichkeit von Deadlocks steigt, da mehr und mehr die Arbeit erfolgt auf der Linie. Ich fixe das in meiner app mit BeginInvoke statt Invoke seit BeginInvoke blockieren nicht den thread beenden und damit den Join wird nicht immer blockieren.

    InformationsquelleAutor Rob Lyndon
  2. 2

    Nur diese überschreibung in Ihrer form-Klasse:

    protected override void OnClosing(CancelEventArgs e) {
    t.Abort();
    base.OnClosing(e);
}
    • +1 Für eine schöne Lösung. Um mehr Infos zu bekommen: siehe stackoverflow.com/questions/1874728/...
    • Ich habe versucht: Es funktioniert auch. Danke.
    • Dank buddy. Die Antwort ist kompatibel mit starter die Qualifikationen, die ich habe -und gedacht -, die aus der Frage. Aber Dank wirklich. Cheers
    • Normalen Szenario Herunterfahren MÜSSEN KEINE Anrufe zu Thread.Abort(), Application.Exit() usw. Ich habe gesehen, dass ein bug verursacht durch das -- Transaktionen nicht zurückgesetzt, weil der eine spurios Environment.Exit() innerhalb eines OnClose() event-handler.
    InformationsquelleAutor javad amiry
  3. -3
    private void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
Thread.CurrentThread.Abort();
}
    InformationsquelleAutor gunderhold
Schreibe einen Kommentar