Methode in der Bedingung

[Endymion]

Ich habe folgendes Problem:
Ich habe eine Methode mit Rückgabewert in der einen Klasse, und will diesen Wert in die Bedingung in einer Methode der anderen Klasse setzen. Wenn ich diese Methode aber anwenden will, bekomme ich nur eine Fehlermeldung. Die Methode sieht folgendermaßen aus:

  void NachOstenBewegen()
    {
        if(symbol.positionX < labyrinth.BreiteGeben()-1)
        {
            positionX = positionX+1;
            symbol.PositionXSetzen(positionX);
            NachOstenBlicken();
        }

Bei der Fehlermeldung ist Zeile3 markiert.
Das Objekt labyrinth existiert und ist sowohl bei den Attributangaben als auch beim Konstruktor gegeben. Wenn ich anstett "labyrinth.BreiteGeben()-1" einfach nur eine Zahl eingebe, funktioniert es aber.

 

[LadyMilka]

Welche Fehlermeldung erhälst du denn?

Gib dir doch mal nur den Wert von labyrinth.BreiteGeben() aus.

 

[SlaterB]

Fehlermeldung NICHT gepostet,
korrekte Objektdeklaration und Initialisierung beteuert, aber NICHT gepostet

Gratulation, genau so macht man es NICHT

----

Methoden unbedingt klein schreiben

 

[Endymion]

Die Fehlermeldung lautet:

java.lang.NullPointerException
at MAMPFI.NachOstenBewegen(MAMPFI.java:82)


Das bedeutet meines wissens nach aber nicht mehr, als dass etwas mit Zeile 82 nicht stimmt, und zu dem Problem noch keine korrekte Fehlermeldung existiert oder?

Als Anhang noch die beiden Quelltexte:

public class MAMPFI
{
    // Attribute
    int positionX;
    int positionY;
    boolean verwundbar;
    char blickrichtung;
    
    // Referenzattribute
    MAMPFISYMBOL symbol;
    LABYRINTH labyrinth;
    
    // Konstruktor für Objekte der Klasse MAMPFI
    MAMPFI (LABYRINTH labyrinthNeu)
    {
        verwundbar = true;
        blickrichtung = 'N';
        positionX = 0;
        positionY = 0;
        
        symbol = new MAMPFISYMBOL();
        symbol.RadiusSetzen(25);
        symbol.StartWinkelSetzen(120);
        symbol.BogenWinkelSetzen(300);
        symbol.BogenArtSetzen(2);
        symbol.FuellFarbeSetzen("gelb");
        symbol.PositionXSetzen(0);
        symbol.PositionYSetzen(0);
        
        
    }
    
    // Methoden
    void NachOstenBlicken ()
    {
        blickrichtung = 'O';
        symbol.StartWinkelSetzen(30);
    }
    void NachNordenBlicken ()
    {
        blickrichtung = 'N';
        symbol.StartWinkelSetzen(120);
    }
    void NachWestenBlicken ()
    {
        blickrichtung = 'W';
        symbol.StartWinkelSetzen(210);
    }
    void NachSüdenBlicken ()
    {
        blickrichtung = 'S';
        symbol.StartWinkelSetzen(300);
    }
    void VerwundbarSetzen(boolean verwundbarNeu)
    {
        verwundbar = verwundbarNeu;
        if (verwundbar == true)
        {
         symbol.FuellFarbeSetzen("gelb");
        }
        else
        {
            symbol.FuellFarbeSetzen("rot");
        }
    }
    void VerwundbarMachen ()
    {
        verwundbar = true;
        symbol.FuellFarbeSetzen("gelb");
    }
    void UnverwundbarMachen ()
    {
        verwundbar = false;
        symbol.FuellFarbeSetzen("rot");
    }
    void MundBewegtSichSetzen(boolean bewegtSichNeu)
    {
        symbol.MundBewegtSichSetzen(bewegtSichNeu);
    }
    void NachOstenBewegen()
    {
        if(symbol.positionX < labyrinth.BreiteGeben()-1)
        {
            positionX = positionX+1;
            symbol.PositionXSetzen(positionX);
            NachOstenBlicken();
        }
    }
    void NachSüdenBewegen ()
    {
        if(symbol.positionY < labyrinth.HoeheGeben()-1)
        {
            positionY = positionY+1;
            symbol.PositionYSetzen(positionY);
            NachSüdenBlicken();
        }
    }
    void NachWestenBewegen ()
    {
        if(symbol.positionX > 0)
        {
            positionX = positionX-1;
            symbol.PositionXSetzen(positionX);
            NachWestenBlicken();
        }
    }
    void NachNordenBewegen ()
    {
        if(symbol.positionY >0)
        {
            positionY = positionY-1;
            symbol.PositionYSetzen(positionY);
            NachNordenBlicken();
        }
    }
}

public class LABYRINTH
{
    //Attribute
    int breite;
    int hoehe;
    ZELLE[][] spielFlaeche;
    
    //Referenzattribute
    
    
    //Konstruktor
    LABYRINTH (int breiteNeu, int hoeheNeu)
    {
        breite = breiteNeu;
        hoehe = hoeheNeu;
        spielFlaeche = new ZELLE[breiteNeu][hoeheNeu];
        for (int zaehlerY = 0; zaehlerY<hoeheNeu; zaehlerY = zaehlerY+1)
        {
            for (int zaehlerX = 0; zaehlerX<breiteNeu; zaehlerX = zaehlerX+1)
            {
                spielFlaeche[zaehlerX][zaehlerY] = new ZELLE(zaehlerX,zaehlerY);
            }
        }
    }
    
    //Methoden
    void GaengeErstellen (int positionX, int positionY)
    {
        spielFlaeche[positionX][positionY].IstMauerSetzen(true);
    }
    int BreiteGeben()
    {
        return breite;
    }
    int HoeheGeben()
    {
        return hoehe;
    }
}

 

[LadyMilka]

edit: hat sich erledigt und Slater war dann schneller

 

[SlaterB]

das ist immerhin für manche ein sehr sehr klarer Hinweis dass du dich nun auch um das zweite der beiden geforderten Dinge dringend kümmern musst:

korrekte Objektdeklaration und Initialisierung beteuert, aber NICHT gepostet

edit:
schon gepostet,
labyrinth ist immer null, nie wird dieser Variablen ein Wert zugewiesen, der Konstruktor-Parameter labyrinthNeu bleibt ungenutzt

 

[Endymion]

Das versteh ich jetzt nicht ganz. labyrinth ist doch kein wert, sondern ein Objekt oder? Und dem Konstruktorparameter labyrinthNeu wird sehr wohl ein Wert zugewiesen, nämlich wenn ich ein Objekt der Klasse MAMPFI erstelle. (In dem Fall muss ich auch immer den Namen eines bereits existierenden LABYRINTHs angeben.) Also entweder verstehe ich etwas falsch oder ihr habt etwas missverstanden. Wenn ich falsch liege, könnt ihr es mir bitte genauer erklären?

 

[SlaterB]

> Und dem Konstruktorparameter labyrinthNeu wird sehr wohl ein Wert zugewiesen, nämlich wenn ich ein Objekt der Klasse MAMPFI erstelle.

das mag sein, die Variable ist gut und wäre hilfreich, wird nur dann in MAMPFI nirgendwo verwendet

------

zu labyrinth:
vergleiche

public class MAMPFI
{
    // Referenzattribute
    MAMPFISYMBOL symbol;
    LABYRINTH labyrinth;
    
    // Konstruktor für Objekte der Klasse MAMPFI
    MAMPFI (LABYRINTH labyrinthNeu)
    {
        symbol = new MAMPFISYMBOL();
    }

symbol und labyrinth werden beide als Referenzattribute angelegt, sind da noch leere Variablen,
symbol befüllst du mit einem Objekt, mit labyrinth machst du dagegen überhaupt nichts, bleibt immer null/leer

dabei wird von außen extra ein anscheinend fertiges LABYRINTH-Objekt übergeben,
du musst es nur auch dem lokalen Referenzattribut zuweisen.., von alleine passiert da nix

 

[bone2]

Die Fehlermeldung lautet:

java.lang.NullPointerException
at MAMPFI.NachOstenBewegen(MAMPFI.java:82)


Das bedeutet meines wissens nach aber nicht mehr, als dass etwas mit Zeile 82 nicht stimmt, und zu dem Problem noch keine korrekte Fehlermeldung existiert oder?

NullPointerException ist eine sehr genaue fehlermeldung und besagt das deine variable keine referenz auf ein objekt enthält

 

[Endymion]

Das verstehe ich nicht komplett. könntest du mir als quelltext schreiben, wie es richtig funktionieren würde?

 

[SlaterB]

MAMPFI (LABYRINTH labyrinthNeu)
    {
        labyrinth = labyrinthNeu;
        symbol = new MAMPFISYMBOL();
    }

 

[Endymion]

ui, danke, es funktioniert