Thread beenden

[Underfaker]

Hallo zusammen,

ich versuche einen thread zu beenden, leide rklappt da snicht.
ich habe es mit stop() probiert (die ja veraltet ist), damit klappte es nicht - der Thread lief weiter.

wenn ich interruot benutze und mittels try/catch auf InterruptedException prüfe sagt Eclipse lediglich, dass diese Exception dort niemals fliegt.

Der Aufbau ist in etwa so:
new Thread()
{
      public void run()
     {
            openWaitForGenerationWindow(); // öffnet ein fenster names frame_WaitForGeneration
     }
}.start();

thread = new Thread()

{ 
public void run()
{
     while (keine passende Aufgabe gefunden)
     {
     generiereAufgabe();
     }
     frame_WaitForGeneration.dispose();
}
}
thread.start();

Falls der User in dem geöffneten Fenster nun schließen drückt, soll der Thread unterbochen werden.

So sieht der Listenerteilausschnitt aus:

public void windowClosing(WindowEvent e) 
			
			{
				if (frame_WaitForGeneration != null && frame_WaitForGeneration.isVisible())
				{
					frame_WaitForGeneration.dispose(); // schließe das Fenster
					thread.stop(); // Beende den Thread
					clearView(); // Blende die aktuelle Ansicht aus
					panel_Start.setVisible(true); // Blende die Ansicht ein, die bei Abbruch wieder zu sehen sein soll
				}
			}

Das funktioniert nun nicht.
1. sagt er mir das

thread.isAlive()

true liefert (bin nicht sicher was da szu sagen hat)
2. Wird beim zweiten Starten der Thread nicht mehr beendet, beim ersten Mal schon.
Das heißt, das Fenster wird geschlossen die Ansicht korrekt gewechselt, aber im Hintergrund wird munter weiter generiert.

Kann mir jemand sagen wie ich das korrekt implementieren kann?

Vielen Dank und beste Grüße

 

[Underfaker]

Also mein fehler es funktioniert schon so, das heißt auch bei mehrmaligen Aufruf aber die methode stop() sollte ja eigentlich nicht benutzt werden, kann mir also trotzdem jemand erklären wie man es alternativ macht?

 

[Joose]

In dem du den Thread auslaufen lässt
Sprich wenn du eine "while-Endlosschleife" in der run-Methode hast, dann musst du die Abbruchbedingung so erweitern das du von außen nur ein bestimmtes Flag ändern musst.

public class MyWorker implements Runnable {
   private boolean run = false;

   public void run() {
      run = true;
      while (run) { // die bedingung kannst du natürlich noch erweitern um was du willst
         // do something
      }
   }

   public void stop()/break()/end() {
      run = false; 
      // außerdem solltest du natürlich den thread hier auch wieder anstupsen falls dieser auf etwas wartet oder ähnliches
   }
}

 

[Underfaker]

Ja das war sehr leicht.
Ich danke dir, das hat es denke ich erschlagen.