Threads - Beenden/ Wieder starten

[Underworld]

Hi,
wir programmieren in der Schule momentan Game of Life.
Ich habe ein bisschen weiter gemacht und bin momentan an einem Punkt angelangt wo ich nicht mehr direkt weiter komme.
Ich bin dabei den Start und den Stop Button zu programmieren.

Mein Code sieht so aus:

	boolean laeuft = true;
	Thread thread = new Thread() {
		public void run() {
			    while(laeuft)
 {    	
				try {
				    berechneNeuenZustand(); //neuer Zustant der arrays/Zellen
				repaint(); // malt halt mein Panel neu, muss nicht weiter erläutert werden
				Thread.sleep(100); // Anzahl in Millisekunden
				}
				catch (InterruptedException ie) {
				}}}};
int test = 0;
public void start() {
if(test ==0){
thread.start();
 }
	   test = 1;
	   if(test ==1){
	       laeuft = true;
	   }
	//   System.out.println(laeuft); Die war einfach nur da um zu überprüfen, ob sich läuft auch wirklich verändert hat
			}
	public void stop() {
		laeuft = false;
		// System.out.println("Hat sich geändert"); ist auch nur zur Überprüfung da
	}

Mein Problem ist nun folgendes:

Ich kann das "Programm" mit der Startmethode starten, sogar das stoppen geht.
Nur das erneute Starten funktioniert noch nicht so ganz.
Wenn ich die Methode "Start" nocheinmal aufrufe passiert einfach nichts.

Habt ihr den Fehler oder eine Idee?

lg Flo

 

[Cola_Colin]

Du solltest die Klammern ein wenig ordentlicher setzen, dann wird das Problem deutlicher.
Das Problem, soweit ich das sehe ist, dass die run Methode verlassen wird sobald läuft false ist.
Wenn diese nun wieder auf true gesetzt wird, startet das den Thread nicht, weil der längst beendet ist(Er ist eben durch die schleife durch und dann ist die Methode run zu ende).
Ich würde die schleife im Thread so aufbauen:

while(true) { 
while(laeuft) {
// tu was du auch immer tun musst ;)
}
Thread.sleep(500);
}

Das wäre so die einfachste Art das zu realisieren denk ich. Ansonsten sehe dir mal die Methoden wait und notify an. Mit denen kann man das eleganter implementieren.

 

[Murray]

Wenn du zum zweiten Mal start() aufrufst, ist laeuft immer noch false, so dass die while-Schleife in der run-Methode sofort fertig ist.

 

[Underworld]

Du solltest die Klammern ein wenig ordentlicher setzen, dann wird das Problem deutlicher.
Das Problem, soweit ich das sehe ist, dass die run Methode verlassen wird sobald läuft false ist.
Wenn diese nun wieder auf true gesetzt wird, startet das den Thread nicht, weil der längst beendet ist(Er ist eben durch die schleife durch und dann ist die Methode run zu ende).
Ich würde die schleife im Thread so aufbauen:

while(true) { 
while(laeuft) {
// tu was du auch immer tun musst ;)
}
Thread.sleep(500);
}

Das wäre so die einfachste Art das zu realisieren denk ich. Ansonsten sehe dir mal die Methoden wait und notify an. Mit denen kann man das eleganter implementieren.

Hi
dein Tipp funktioniert!

So ganz Ahnung warum es jetzt nur mit dem zugefügten while(true) funktioniert, hab ich jetzt nicht.

Das werd ich wann anders mal versuchen nachzuvollziehen.

vielen Dank

 

[Cola_Colin]

na "wann anders mal" bringt dich nicht weiter.

Stelle dir das so vor:
1. Du startest den Thread. Nun wird also in diesem Thread die Methode run ausgeführt.
2. Diese geht in die while schleife solange läuft true ist.
3. Setzt du nun läuft auf false, so wird die while(läuft) schleife nicht länger ausgeführt.
4. Ohne eine weitere Schleife ist damit die Methode run am Ende angekommen, was den Thread beendet. Danach ist der zustand von laeuft vollkommen egal.
5. Nur mit der 2. Schleife fragt er regelmäßig ab, ob läuft wieder true geworden ist.

Nachteil daran ist, dass er das eben entweder durch ein thread.sleep nur in gewissen abständen macht(im worstcase läuft das prog erst 500ms nach Klick auf den "Weiter" Button wieder an) oder aber ohne ein Thread.sleep die CPU Auslastung auf 100% treibt, was natürlich komplett Sinnfrei ist.

Um das zu verbessern bietet sich wie gesagt eine Kombination aus Thread.wait() und Thread.notify an.

 

[Murray]

... oder eben einfach in der start-Methode vor Aufruf von thread.start() einfach laeuft wieder auf true setzen.

 

[Underworld]

na "wann anders mal" bringt dich nicht weiter.

Stelle dir das so vor:
1. Du startest den Thread. Nun wird also in diesem Thread die Methode run ausgeführt.
2. Diese geht in die while schleife solange läuft true ist.
3. Setzt du nun läuft auf false, so wird die while(läuft) schleife nicht länger ausgeführt.
4. Ohne eine weitere Schleife ist damit die Methode run am Ende angekommen, was den Thread beendet. Danach ist der zustand von laeuft vollkommen egal.
5. Nur mit der 2. Schleife fragt er regelmäßig ab, ob läuft wieder true geworden ist.

Nachteil daran ist, dass er das eben entweder durch ein thread.sleep nur in gewissen abständen macht(im worstcase läuft das prog erst 500ms nach Klick auf den "Weiter" Button wieder an) oder aber ohne ein Thread.sleep die CPU Auslastung auf 100% treibt, was natürlich komplett Sinnfrei ist.

Um das zu verbessern bietet sich wie gesagt eine Kombination aus Thread.wait() und Thread.notify an.

Hab ich mir gedacht, dass sich gleich jemand über das "mit wann anders" beschwert
Hab damit gemeint, dass versuch ich heut abend oder morgen zu verstehen!

Dank deiner Erklärung ist mir das jetzt aber auch verständlicher geworden.

Ich bin davon ausgegangen, dass die while-Schleife den Wert laeuft einfach immer abfrägt, auch wenn er einmal false gewesen ist.

Ich werd mir mal das Thread.wait() und Thread.notify anschauen.

Wenn wir schon dabei sein. Ich bin auch schon auf das thread.stop gestoßen. Das scheint aber kein eleganter, wenn nicht sogar gefährlicher Weg zu sein.

lg Flo

EDIT: Murray
ich schau mir mal deine Lösung an. Ich glaube aber, das hab ich schon so ähnlich versucht und da hats leider auch nicht geklappt.

 

[Murray]

EDIT: Murray
ich schau mir mal deine Lösung an. Ich glaube aber, das hab ich schon so ähnlich versucht und da hats leider auch nicht geklappt.

Versuch's nochmal, gern auch ohne die test-Variable, die das nur komplizierter macht. Dann sollte es klappen. Wenn nicht, dann poste nochmal den neuen Code.

 

[Underworld]

Versuch's nochmal, gern auch ohne die test-Variable, die das nur komplizierter macht. Dann sollte es klappen. Wenn nicht, dann poste nochmal den neuen Code.

leider funktioniert es nicht, hier der Code

		boolean laeuft = true;
	Thread thread = new Thread() {
		public void run() {
				while (laeuft) {
				try {
					berechneNeuenZustand();
					repaint();
					Thread.sleep(100); // Anzahl in Millisekunden
				}
				catch (InterruptedException ie) {
				}
			}
		}
	};
	
	
public void start () {
laeuft = true;
thread.start();
}

	public void stop() {
		laeuft = false;		
	}

	public void step() {
		berechneNeuenZustand();
		repaint();
	}

und hier die Fehlermeldung, wenn ich das Programm benutze:

Exception in thread "AWT-EventQueue-0" java.lang.IllegalThreadStateException
	at java.lang.Thread.start(Thread.java:595)
	at GOLPanel.start(GOLPanel.java:225)
	at GOLFenster$1.actionPerformed(GOLFenster.java:50)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
	at java.awt.Component.processMouseEvent(Component.java:6263)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:6028)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
	at java.awt.Container.dispatchEventImpl(Container.java:2085)
	at java.awt.Window.dispatchEventImpl(Window.java:2475)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

 

[Murray]

OK, jetzt sehe ich das erst: du versuchst ja, die gleiche Thread-Instanz erneut zu starten; das geht natürlich nicht.

It is never legal to start a thread more than once. In particular, a thread may not be restarted once it has completed execution.

Wenn man zulässt, dass der Thread einmal aus der run-Methode herauskommt, dann muss man für den nächsten Start einen neuen Thread erzeugen.

Also entweder jedesmal einen neuen Thread nehmen, oder wie in Cola_Colins Lösung in der run-Methode die eigentliche Aktion in eine Endlosschleife kapseln (dann aber wirklich besser mit wait/notify).

 

[Atze]

du kannst auch nur einmal thread.start() aufrufen, das zweite mal führt immer zu dieser exception. start() versetzt den state eines threads nur einmal in runnable / not runnable.

*edit*
murray war schneller