AWT Threads richtig beenden! Wie?

[Starbuck1865]

Hallo,

ich hab hier ein kleines Problem, wenn ich in meinem AWT Hauptprogrammfenster einen Button drücke, erzeuge ich einen neuen Thread mit einem kleinen AWT Fenster (Die Klasse die in dem Thread instanziert wird, erbt also von Frame und implementiert runnable). In der run() Methode des Threads ist eine Endlosschleife die nach bestimmter Zeit ein repaint auslöst.
Wenn das Fenster geschlossen wird, soll über die WindowsClosing() Methode auch der Thread richtig beendet werden.
Meine Frage ist, wie mache ich das richtig. Denn bisher habe ich es nur geschafft, den Thread in den suspend Modus zu bringen. Das meint jedenfalls der Debugger vom Eclipse.
Aber dann ist der ganze Speicher ja noch nicht freigegeben, oder?

Meine WindowClosing Methode und run Methode();

public void windowClosing ( WindowEvent e)
{ 
				    	  
				    	  			    	  
	e.getWindow().setVisible(false);
	e.getWindow().dispose();	
				    	  
}

public void run()
{
    do
   {
	 try
         {
		            	
		 Thread.sleep(100);
		 repaint();
         }
        catch(InterruptedException interruptedexception)
	{
		            	
	}
		            
   }
   while(true);
			
}

Ich hab auch schon mit thread.stop experimentiert, oder versucht die Endlosschleife mit einem interrupt zu unterbrechen. Beides hat auch blos zum suspend des Threads geführt.

Muss ich eventuell in der WindowClosed() Methode noch irgendwas freigeben? Aber die kann ja eigentlich gar nicht mehr erreicht werden, wenn der Thread, welcher das Frame beherbergt geschlossen wird.
Und müsste nicht der Carbage Collector mit dem Müll aufräumen der da suspended ist?


nach schließen des Threads, zeigt mir der Eclipse Debugger folgendes an:

Thread [Thread-4] (Suspended (exception NullPointerException))
SignalMonitor.paint(Graphics) line: 225
SignalMonitor.repaint() line: 237
SignalMonitor.run() line: 197
Thread.run() line: 595


Vielen Dank für Hilfe im voraus

 

[Mag1c]

Hi,

erst ein Flag setzen, damit der Thread weiß, daß er sich beenden soll (muß dann dort beim while(true) rein).
Dann den Thread unterbrechen thread.interrupt(), falls der sich gerade im sleep(...) befindet.
Damit sollte der Thread sicher beendet sein und du kannst das Fenster schließen.

Gruß
Mag1c

 

[Starbuck1865]

Hallo Mag1c,

du meinst also ich soll die run() Methode so schreiben:

public void run()
{
			
		       do
		       {
		    	   
		            try
		            {
		            	Thread.sleep(100);
		                repaint();
		            }
		            catch(InterruptedException interruptedexception)
		            {
		            	Thread.currentThread().interrupt();
		            }
		            
		       }
		       while(!Thread.currentThread().isInterrupted());
			
}

und am Anfang der WindowClosing() ein Thread.currentThread().interrupt(); einfügen.

Das habe ich schon ausprobiert, führt zu dem gleichen Ergebnis. Thread wird nur suspended.
Ich denke mal, dass das in meinem Program gar nichts bewirkt. Liegt vielleicht daran, dass ich nur runnable implementiere und nich direkt von Thread ableite.

 

[Beni]

erst ein Flag setzen, damit der Thread weiß, daß er sich beenden soll (muß dann dort beim while(true) rein).

:arrow:

public class IrgendeinThread extends Thread{
  private boolean flag = false;

  public void run(){
     while( !flag ){
        // irgendwas machen
     }
  }

  public void friendlyStop(){
    flag = true;
    this.interrupt();
  }
}

 

[Starbuck1865]

Okay danke, hab mein Fehler gefunden.

Ich hab den interrupt immer in der WindowClosing() direkt vorm dispose geschickt. Das ging anscheinend nicht. Jetzt hab iihn in die WindowClosed() geschrieben und es funzt. Juhuu!

 

[Guest]

Halt Stop,

war quatsch was ich eben geschrieben habe. Ich hab jetzt zwar ne Lösung gefunden und das Prolem erkannt aber meine Lösung is noch nicht zufriedenstellend.

Folgends hab ich gemacht, damit es klappt:

public class SignalMonitor extends Frame
	implements Runnable
{

private boolean interruptflag = false;
boolean threadfertig=false;

public SignalMonitor()
{
					
		addWindowListener( new WindowAdapter() 
		{
				    
			public void windowClosing( WindowEvent e ) 
			{ 
				    	  
				
				interruptflag = true;
				Thread.currentThread().interrupt();
						  
						  
				while(true)
				{
					 if(threadfertig)
					{
						e.getWindow().setVisible(false);
						e.getWindow().dispose(); 
						break;
					}
							  
				}
			}
								
			});

....
}

public void run()
{
			
       do
       {
		    	   
	     try
	     {
		      Thread.sleep(100);
		       repaint();
	     }
	     catch(InterruptedException interruptedexception)
	     {
		       interruptflag=true;
	      }
		            
      }
      while(!interruptflag);    
			
      threadfertig=true;
}

Das es jetzt funzt lag wohl daran, dass ich vorher immer direkt nach dem ich das flag oder den interrupt in der Windowclosing Methode gesetzt habe, das dispose aufgerufen hatte. Der Thread war aber zu diesem Zeitpunkt noch nicht richtig beendet und es gab den Fehler. Das hab ich jetzt mit der Hilfsvariable threadfertig gelöst, auf deren true zustand ich in WindowClosing warte, um danach zu disposen

Okay das mag ne Lösung sein, aber das ist sicher nicht im Sinne des Erfinders. Also kann mir vielleicht einer weiterhelfen, und zeigen wies richtig geht?

 

[Mag1c]

Aeh,

ich vermisse noch den Thread. Wo erzeugst du den ?

Mit Thread.currentThread().interrupt() unterbrichst du den gerade laufenden Thread, was IMHO keinen Sinn ergibt, da der gerade laufende Thread dabei ist sich selbst zu unterbrechen :bae:
Du mußt dem SignalMonitor eine Referenz auf den erzeugten Thread geben, damit dieser beim Schließen des Fensters beendet werden kann. Das threadfertig Flag kannst du wieder rausschmeißen. Das kann man mit Thread#join erledigen.

Und: Design-technisch würde ich diesen Thread (oder ein Runnable) als inner Class implementieren und den Thread per Methode setAutoUpdate(boolean) oder ganz automatisch bei setVisible(boolean) aktivieren bzw. deaktivieren.

Gruß
Mag1c

 

[Starbuck1865]

Ich erzeuge den Thread in einem anderen Frame. Wenn ich den entsprechenden Button drücke. Also die Erzeugung liegt im Ereignisbehandler des Buttons. Wie übergebe ich den jetzt eine Referenz an die andere Klasse bzw. den Thread. Und ist nicht currentThread das selbe, als wenn ich ne Referenz von dem Thread übergebe. Ich will ja gerade, über die WindowClosing() des Threads, den selben aus seinem Schlaf holen und beenden.

 

[Mag1c]

Hi,

dein erzeugter Thread tut genau das, was du ihm in der run()-Methode vorgegeben hast, nämlich in einer Schleife eine kurze Pause einlegen und anschließend das Frame neu zeichnen. Sonst nix!
Das Fenster schließen bzw. der Aufruf des WindowListeners wird durch den AWTEventThread durchgeführt.

Du könntest per set-Methode nach dem Erzeugen den Thread setzen. Aber das wäre nur eine weitere Krücke. Ich habs ja im letzten Post schon geschrieben. Mach den Thread komplett in den SignalMonitor rein. Nach außen hin ist das völlig uninteressant, wie du die Aktualisierung der Anzeige vornimmst.

Gruß
Mag1c

 

[Starbuck1865]

Okay es läuft. Ich hab den Thread, wie du es mir geraten hast, in einer inneren Klasse definiert. Damit kam der Interrupt an und alles ist wunderbar.