Game Loop Problem

[GuestG]

Hi,

ich hatte und hab gerade ein sehr komisches Problem mit meinen GameLoop.

	while(!terminated)														
{
 if(running)																
   {
	long startTime = System.currentTimeMillis();
	update();	
	long sleeptime = (startTime + 20) - System.currentTimeMillis();	
		
		 if(sleeptime < 0 )
		{
			sleeptime = 0;
		}
		try {gameController.sleep(sleeptime);}				
		catch (InterruptedException e) {System.out.println("Unterbrechungsfehler");}
		}
}

Problem war, dass mein Spiel nicht ausgeführt wurde (die schleife nicht lief). Erst nachdem ich zwischen while(!terminated) und if(running) noch einen Befehl (System.out.println()) eingefügt hatte ging es. Aber woran es lag konnte ich nicht sagen.
Das selbe Problem tritt momentan noch auf wenn ich das if in ein while ändere.
Wird da irgendwie was zu schnell abgetestet? so wirklich verstehe ich das Problem nicht.
while(!terminated) läuft ja ständig und bei jeden Durchlauf testet er ja ab ob running == true ist.

mfg

 

[Gast2]

Schwer zu sagen ohne zu wissen wo running und terminated gesetzt werden!

 

[GuestG]

Spielt es eine große Rolle wo running gesetzt wird?
Es wird mit dem Pause Button gesetzt vorher die Instanzvariable running auf false.

pauseButton.addActionListener(new ActionListener() 
		{
			public void actionPerformed(ActionEvent e) 
	         { 
				requestFocus(); 
				if(game.isRunning() == false)
				{
					pauseButton.setIcon(pauseIcon);
					game.setRunning(true); 
				}
				else
				{
					pauseButton.setIcon(spielFortsetzenIcon);
					game.setRunning(false);
				}
	         }
});

Der Button setzt running wirklich auf true aber der GameLoop reagiert dann halt aus irgendeinen Grund nicht drauf obwohl er nach meinen Verständlich permanent testen müsste ob running auf true ist und bei einer Änderung auch reagieren müsste.

 

[Gast2]

Natürlich spielt das eine Rolle!

Wenn du z.B: Threads verwendest könnte es zu Concurrencies oder Deadlocks kommen etc....

 

[Plopo]

Problem war, dass mein Spiel nicht ausgeführt wurde (die schleife nicht lief). Erst nachdem ich zwischen while(!terminated) und if(running) noch einen Befehl (System.out.println()) eingefügt hatte ging es.

Hast du den irgend eine Art von Ausgabe, ohne den Syso-Befeh zwischen der While-Schleife und if-Abfrage?
Ansonsten läuft dein Programm schon richtig.
Nur siehst du nichts, da keine Ausgabe gemacht wird.

 

[GuestG]

Der GameLoop ist genau so wie im Spiel. Zwischen While-Schleife und if-Abfragesteht nix.
Hier muss ich mich mal korrigieren mit while und if geht es. Nur wenn ich das if in ein while ändere geht es nicht mehr.

Also Deadblocks sind quasi Zustände in dem zwei Threads (in meinen Fall der Spiel-Thread und der MainThread) die gleiche Ressourche irgendwie beanspruchen.
Also könnte man eventuell mal probieren das running zu synchronisieren?

mfg

 

[Plopo]

Warum brauchst du eine While-Schleife in einer While-Schleife?
Das ist unnötig und bringt nichts.

while(!isTerminated)
{
   if(isRunning)
   {
      //Methoden
   }
}

Ist doch alles was du brauchst, in die If-Abfrage kommt dein Game Zeugs rein.

 

[Marco13]

Wenn bei dem "sleep" eine 0 übergeben wird, macht er gar nichts, und es kann sein, dass dieser Thread dann alleine läuft (SEHR vereinfacht). Mach' mal sowas wie

// WEG
//if(sleeptime < 0 )
//{
//  sleeptime = 0;
//}

// HIN:
sleepTime = Math.max(1, sleepTime);

 

[AquaBall]

wenn ich das if in ein while ändere geht es nicht mehr.

Jetzt versteh ich den Spruch: Es gibt keine if-Schleife!

Zwischen if und while ist wohl ein grandioser Unterschied?!?!

while(!isTerminated && isRunning)
{ ...
}

 

[CPU-Last]

Das Problem dürfte sein das in der while ein sleep fehlt.
Vereinfacht : wenn isTerminated false ist und gleichzeitig runnig ebenfalls false ist dann hast du eine leere Endloseschleife. Und die frisst extrem viel CPU-Zeit wesshalb es dazu kommen kann das der Loop viel zu ausgelastet ist mit nichts-tun um auf die Flagänderung zu reagieren. Leere loops sollte man grundsätzlich vermeiden und stattdessen gerade bei solchen "Pausen" mit wait und notify arbeiten.

btw : while(!bool) macht man nicht (lange Form : ist es wahr das etwas falsch ist). Wenn man dierekt nur bools hat dann schreibt man immer so das man dierekt auf den bool prüft.
Also statt while(!isTerminated) halt auf while(isNotTerminated).