[Greenfoot] java.lang.NullPointerException

[VintageSucks]

Hallo,

ich arbeite mit Greenfoot und erstelle gerade ein simples Spiel.

Nun wollte ich eine Pausier-Funktion einbauen, und habe in der World Class Levels

public boolean isRunning;

public void act()  
    {  
        checkPause();  
    }

und

public void checkPause()  
    {  
       String keyPressed = Greenfoot.getKey();  
        if(keyPressed.equals("p"))  
        {  
            if(isRunning)  
            isRunning = false;  
            else if(!isRunning)  
            isRunning = true;  
        }  
    }

eingebaut. In der Actor Class Fee dementsprechend

public void act()  
{  
    if(((Levels) getWorld()).isRunning)  
    {  
        ...  
    }  
}

Das Kompilieren funktioniert einwandfrei, aber wenn ich das ganze dann ausführe erscheint diese Fehlermeldung:

java.lang.NullPointerException
	at Levels.checkPause(Levels.java:28)
	at Levels.act(Levels.java:22)
	at greenfoot.core.Simulation.actWorld(Simulation.java:504)
	at greenfoot.core.Simulation.runOneLoop(Simulation.java:447)
	at greenfoot.core.Simulation.runContent(Simulation.java:197)
	at greenfoot.core.Simulation.run(Simulation.java:187)

Was habe ich falsch gemacht? Ich habe schon gegoogelt, aber nicht herausgefunden woran es liegen könnte.

 

[akimoon]

kleine Frage:
Was steht denn in:
" at Levels.checkPause(Levels.java:28)"?

Eigentlich ist eine Nullpointer-Exception doch relativ aussagekräftig? 🙂

 

[Tomate_Salat]

Ich nehme mal an:
keyPressed null oder
Greenfoot ist null.
Wenn Greenfoot keine falsch geschrieben Variable ist, wird wohl ersteres der Fall sein. D.h. du musst erst noch auf [c]null[/c] prüfen.

 

[Marco13]

Greenfoot (Greenfoot_DE_2_0) -> "Liefert die zuletzt gedrückte Taste zurück (seit dem letzten Aufruf). Wenn seit dem letzten Aufruf der Methode keine Taste gedrückt wurde, wird null zurückgegeben."

->
if (keyPressed != null && keyPressed.equals("p"))
oder
if("p".equals(keyPressed))

 

[VintageSucks]

if (keyPressed != null && keyPressed.equals("p"))

hat wunderbar funktioniert, vielen dank!

weiß jemand, warum das spiel beim ausführen nun pausiert startet, und ich es mit p starten muss? wie bekomme ich es hin, dass es so startet dass es unpausiert ist?

 

[akimoon]

was für einen Wert hat denn "isRunning" am Anfang? 😀

Kenn mich leider mit Greenfoot nicht aus, und weiß nicht, wo checkPause() aufgerufen wird..

 

[VintageSucks]

isRunning

ist am Anfang scheinbar auf

false

(genau weiß ich es aber auch nicht), und

checkPause()

wird ab dem Levelstart ununterbrochen ausgeführt um auf ein p zu warten.

 

[Gast2]

public void checkPause()  
    {  
       String keyPressed = Greenfoot.getKey();  
        if(keyPressed.equals("p"))  
        {  
            if(isRunning)  
            isRunning = false;  
            else if(!isRunning)  
            isRunning = true;  
        }  
    }

Nur so am Rande: Setz die KLammern sonst hast du irgendwann ohne Ende Fehler die du suchst...

public void checkPause()  
    {  
       String keyPressed = Greenfoot.getKey();  
        if(keyPressed.equals("p"))  
        {  
            if(isRunning) { 
                isRunning = false;  
            } else {
                isRunning = true;  
            }
        }  
    }

 

[akimoon]

Also...
Variante A: Debuggen/Wert ausgeben
Variante B: Variable einfach mal auf true setzen????:L???:L

Und wenn mein Vorredner daran schon was aussetzt:
Du kannst dir das if beim else-Zweig sparen. Ein boolscher Wert ist entweder true oder false. Eine dritte Möglichkeit gibt es nicht (if(true) xxx else yyy ) .... edits sind fies! :lol:

 

[Tomate_Salat]

Noch so ein Trick am Rande: Man kann das ganze verkürzen.
Ausgangssituation:

if(isRunning)  
      isRunning = false;  
else if(!isRunning)  
      isRunning = true;

Verkürzen:
Kürzer wäre wohl das [c]else if[/c] durch ein else zu ersetzen, aber noch kürzer geht es, wennn du einfach folgendes schreibst:

isRunning=!isRunning;

 

[VintageSucks]

das verkürzen war ein guter tipp, danke. 🙂

Variante B: Variable einfach mal auf true setzen????:L???:L

wenn ich das mache, startet das ganze zwar endlich unpausiert, lässt sich aber nicht pausieren.

kann man nicht irgendwie unpausiert starten, und erst beim drücken von p pausieren? ein spiel das pausiert startet ist ja irgendwie nicht schön. 😀

 

[akimoon]

wo hast du die Variable denn auf "true" gesetzt?

ich meinte statt

public boolean isRunning;

ein

public boolean isRunning = true;

Eigentlich sollte das pausieren dann immernoch gehen.


Achja, noch etwas: Statt dem if-else ginge auch ein:

isRunning = !isRunning;

was den Wert ja "herumdreht"

 

[VintageSucks]

oh, jetzt funktioniert es ja doch. keine ahnung was ich eben gemacht habe.

vielen dank für eure hilfe! 🙂