Unerklärliche NullPointerException

[lowbrain]

Moin Moin!

Ich habe ein kleines Problem und stehe grade voll auf dem Schlauch...
Das Problem ist folgendes:
Ich möchte das Smess-Spiel implementieren. Dazu habe ich ein Array aus Spielfeldern erzeugt (court[][]). Die Klasse Spielfeld besitzt eine Spielfigur, bzw. wenn keine Figur darauf ist, dann ist es null. Die Spielfigur wiederum hat eine Position, also ein Spielfeld auf dem sie steht.
Jetzt möchte ich die Figur umsetzen. Das macht folgender Programmabschnitt:

[Java]
public void zugAusfuehren(int startZ, char startS, int endZ, char endS){
int sZ = startZ;
int eZ = endZ;
int sS = startS - 'a' + 1;
int eS = endS - 'a' + 1;

if(this.court[eZ][eS].getFigur() != null){
// Geschlagene Figur löschen
this.court[eZ][eS].getFigur().getBesitzer().deleteFigur(this.court[eZ][eS].getFigur());
}
// Figur auf neues Feld setzen und altes Feld auf null
this.court[eZ][eS].setFigur(this.court[sZ][sS].getFigur());
Spielfigur aushilfe = new Spielfigur();
aushilfe = court[sZ][sS].getFigur();
aushilfe.setPosition(court[eZ][eS]);
this.court[sZ][sS].setFigur(null);

}
[/Java]

Das Problem ist, das in Zeile 13 manchmal eine NullPointerException auftritt. Dabei durchläuft jeder Spielzug zuerst ein Schiedsrichterprogramm, bevor er ausgeführt wird. Der Spielzug ist also korrekt. Es existiert sowohl das von-Feld, als auch das nach-Feld, und das Feld von dem gestartet wird hat eine Spielfigur.
Das komische ist, manchmal dauert es 2 Züge, bis der Fehler auftritt, aber ein anderes Mal kann er erst 10 Züge machen, und dann ist Feierabend.

Vielleicht hat jemand eine Idee? Manchmal sieht man ja den Wald vor lauter Bäumen nicht...

Vielen Dank schonmal im voraus!

 

[jgh]

[c] Spielfigur aushilfe = new Spielfigur();[/c] dieser Codeabschnitt schmeißt eine NPE?

Dann zeig mal bitte den Konstruktor, bzw. den Stacktrace.

 

[Antoras]

Der Spielzug ist also korrekt.

Offenbar nicht, sonst würde ja keine NPE fliegen.

Du greifst in deiner Methode immer wieder auf unterschiedliche Felder des Arrays court zu. Wenn eines dieser Felder null ist, dann fliegt natürlich eine NPE sobald du darauf zugreifen möchtest.

In Zeil 13 kann im übrigen keine NPE fliegen, es sei denn die NPE fliegt im Konstruktor von Spielfigur. Im übrigen ist diese Zeile sinnlos, da du die Referenz auf das erzeugte Objekt in der nächsten Zeile sofort wieder überschreibst.

 

[lowbrain]

Ahh, sry,

Zeile 15 schmeißt die Exception:

Exception in thread "main" java.lang.NullPointerException
at lowBrainKI.LowBrain.zugAusfuehren(LowBrain.java:68)
at lowBrainKI.LowBrain.naechsterSpielzug(LowBrain.java:41)
at lowBrainKI.TestBrett.main(TestBrett.java:34)

----------------------------------------------
Natürlich ist die Zeile sinnlos, wie auch zum Beispiel die int nochma neu zuzuweisen. Das habe ich gemacht, um halt im Debug-Modus zu sehen, ob er alles richtig macht

 

[Cola_Colin]

aushilfe ist null.
Wieso, das kann man aus dem Codeabschnitt nicht heraus erkennen. Aber es ist null.

 

[Antoras]

Zeile 15 ist sehr suspekt. Wenn du der Figur eine neue Position zuweisen willst, warum übergibt du dann der Methode das Object auf dieser Position? Was passiert wenn hier null übergeben wird?

 

[lowbrain]

Das ganze Ding ist wie eine riesige doppelt und dreifach verkette Liste...
Also es gibt zwei Spieler, die haben Figuren, und ein Spielbrett aus Feldern.
Und ich habe halt immer Referenzen von
Spieler<--> Figur <-->Spielfeld

Naja, das ich halt das Objekt übergebe funktioniert aber, da gab es noch nie Probleme...

Joa, nicht das beste, aber ist halt über die Wochen gewachsen, hat kleiner angefangen...

Naja, auf jeden Fall wird aushilfe nie null, da ja vorher geschaut wird, durch das Schiedsrichterprogramm, ob auf dem Feld ein Spieler ist...

 

[Cola_Colin]

Füge mal das vor Zeile 15 ein:

if (aushilfe == null) System.out.println("Aushilfe ist NULL !");

und teste.

 

[lowbrain]

jo, stimmt

Na dann kann ich mich ja jetzt mal auf Fehlersuche machen, wieso das Ding doch ausgeführt wird...
Danke

 

[tagedieb]

Ich finde den ganzen Code recht verwirrend.

Versuch mal den Code aufzuraeumen und einen mehr Objektorientierten Ansatz zu waehlen.
Zum Beispiel spendier deiner Spielfigur die Methoden:

set(..)
moveTo(...)
remove()

Ausserdem denke ich wie gegenseitige Abhaengigkeit von Spielfigur und Spielfeld kann zu Rekursiv- und NullPointerFehlern fuehren.
Z.B. Loesche Figur aus Spielfeld -> loesche Spielfeld aus Figur -> Loesche Figur aus Spielfeld (already deleted) -> NullpointerException


Wieso erstellst du hier eine neue Spielfigur?

    Spielfigur aushilfe = new Spielfigur();
    aushilfe = court[sZ][sS].getFigur();

Sollte es nicht ehrer so lauten?

    Spielfigur aushilfe = court[sZ][sS].getFigur();