Thread verhält sich seltsam

[java-jim]

Hallo!

Ich habe einen Thread in einem Applet, der sobald er gestartet wird einfach die Sekunden von 10 runterzählen soll. So sieht das bei mir aus:

public void run() {
		int zeit = 10;
		laufend = true;
		
		while (laufend) {
			infoPanel.setZeit(zeit);
			if (zeit == 0) {
		                verwaltung.zeitUeberschreitung();
				break;
			}
			try {
				Thread.sleep(1000);
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
			zeit--;
		}

	}

Bei ca. 4 von 5 Versuchen läuft das ganz normal. Aber manchmal läuft die Zeit "kreuz und quer", also z.B. 10, 8, 4, 9, usw. Der Thread ist der einzige im gesamten Programm.

Weiß jemand warum das ganze so unregelmäßig läuft?

 

[André Uhres]

Und du bist sicher, daß bei jedem "Versuch" die Zeit vom vorigen Versuch ganz abgelaufen ist?
Obwohl das der einzige "Thread" ist, könnte er aber theoretisch mehrmals gleichzeitig
(oder fast gleichzeitig) gestartet werden!

 

[java-jim]

Es handelt sich um ein kleines Spiel. Eine Runde wird gestartet und damit auch der Thread. Wenn der Benutzer in diesen 10 Sekunden seine Eingabe macht, wird "laufend" auf "false" gesetzt. Damit ist der Thread doch dann beendet oder nicht?
Wenn keine Eingabe gemacht wird, wird die Schleife in run() ja auch verlassen und damit der Thread beendet.

Was ich noch vergessen hatte zu erwähnen: Das "Kreuz- und Quer-Zählen" erfolgt auch wesentlich schneller als jede Sekunde und mit unregelmäßigen Zeitabständen.

 

[deleted]

entferne mal das break.
Das gehört NICHT in if Konstrukte sondern ist lediglich bei switch-Anweisungen zu nutzen!

 

[java-jim]

Hab's ohne break probiert, funktioniert leider nicht.

 

[java-jim]

Hab's selbst hinbekommen, danke für eure Mühe!

Das Problem war tatsächlich dass der Thread noch im sleep() war als ein neuer Thread gestartet wurde und sich erst danach beendet hat.

 

[HoaX]

wieso soll er es nicht verwenden? ich würde statt laufend lieber Thread.isInterrupted() und Thread.interrupt() verwenden, aber dennoch ist das break nicht wirklich falsch. man könnte auch statt break einfach laufend=false setzen. es bezieht sich sowieso auf die schliefe und nicht auf die if.