Methode aus einer anderen Klasse ausführen

[acevik94]

Hallo Community!
Ich programmiere ein Spiel mit Java und habe folgendes Problem.
Ich habe 2 Klassen, Main und GoAnim. In der Main Klasse habe ich JFrame, labels, buttons usw. initialisiert und für den ersten Spieler habe ich ein Label(PlayerOne) erzeugt. Nun rufe ich die Methode ChangePos1(...Parameter...) in der GoAnim Klasse auf, lasse das Programm etwas ausrechnen und anschliessend rufe ich die Methode setPosition1(...Parameter...) in der Main Klasse auf, kurz danach bekomme ich java.lang.NullPointerException Fehler. Hier ist mein Quelltext:

public class Main extends javax.swing.JFrame{
        ButtonClickEvent(){ // ich klicke auf ein Button und passiert folgendes
              GoAnim objectGoAnim = new GoAnim();
              //X,Y sind Koordinaten, Roll und Pos sind beliebige int Werte
              objectGoAnim.ChangePos1(int Roll, int Pos, int X, int Y);
        }
              .............. //Hier sind andere Methoden, die unwichtig sind 
        //Diese Methode werde ich spaeter aufrufen
          public void setPosition1(String PosName1, int PosNumber, int X, int Y){
               Position1 = PosName1; //Name der Position
               PosNumber1 = PosNumber; //wird in GoAnim ausgerechnet
               //PlayerOne ist ein Label, neue Y-Koordinate wird berechnet
               PlayerOne.setLocation(X,Y);
        }
}
public class GoAnim extends Main{
         Main objectMain = new Main();
        public void ChangePos1(int Roll, int Pos, int x, int y){
                   int newPos = Roll + Pos; //rechne neue Position
                   y -= 20; //Y Koordinate wird um 20 verringert
                   objectMain.setPosition1("PositionX", newPos, x, y);
        }
}

Die Fehlermeldung ist :

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
   at GoAnim.ChangePos1(GoAnim.java:60)
   at Main$33.actionPerformed(Main.java:651)

Ich verstehe nicht warum ich NullPointer Fehler bekomme. Ich habe in der Methode
setPosition1(...Parameter...) die übergegebenen Parameter ausgegeben lassen und
die Werte waren richtig und "null" war nicht dabei.
Wie kann ich diesen Fehler beheben ?
Ich bedanke mich im Voraus.

 

[VfL_Freak]

Moin,

die Exception tritt hier auf:
at GoAnim.ChangePos1(GoAnim.java:60)

Leider weiß keiner, welche Zeile in Deinem Code das ist !!
Zudem fehlt ggf. mehr Code (Aufrufe etc.) !!

Gruß Klaus

 

[Elenteria]

das problem ist dass objectMain zu dem Zeitpunkt null ist.
Wofür brauchst du überhaupt ein seperates Main-Object in der Klasse? GoAnim erweitert doch Main also steht setPosition1 auch GoAnim zur verfügung. probiers mal mit this.setPosition1("PositionX", newPos,x,y);

 

[JStein52]

Wieso ist objectMain null ? Ausserdem schreibt er ja auch dass setPosition1 ausgeführt wird. Allerdings ist die gegenseitige Objekterzeugung schon nicht schön und sicher nicht notwendig. Man müsste einfach mal sehen wo die Exception denn nun geworfen wird.

 

[Joose]

Ich glaub nicht das "objectMain" null ist und daher diese Exception fliegt. Das Objekt wird ja erstellt sobald ein GoAnim Objekt erstellt wird.

Wir können dir zwar gerne sagen wo dein Problem liegt (dazu musst du uns aber genau sagen was Zeile 60 in der GoAnim Klasse ist), aber es fehlen dir wirkliche Grundlagen und du wirst bald wieder wo Probleme haben.

Ich gebe Elenteria recht: Deine Klasse GoAnim leitet von der Klasse Main ab und ist somit selbst ein "Main" Objekt. Diese Ableitung ergibt für überhaupt keinen Sinn, ebensowenig das bei einem Buttonklick jedesmal ein neue GoAnim Objekt erstellt wird und in diesem GoAnim Objekt wird wieder ein Main Objekt erstellt.
Bevor du groß an dem Spiel weiterarbeitest solltest du dir anhand eines einfachen Konsolenprogramms die Grundlagen der OOP nochmals anschauen! Die sind eben eine der Grundvorraussetzungen für Spiele

 

[acevik94]

Vielen dank für die schnellen Antworten.

Moin,

die Exception tritt hier auf:
at GoAnim.ChangePos1(GoAnim.java:60)

Leider weiß keiner, welche Zeile in Deinem Code das ist !!
Zudem fehlt ggf. mehr Code (Aufrufe etc.) !!
Gruß Klaus

Zeile 60 ist : "objectMain.setPosition1("PositionX", newPos, x, y);"
Nein es fehlen keine Aufrufe , da die 3 Methoden zusammenhaengen. Also Es gibt keine andere Methode, die diese Methoden aufruft oder die Werte in diesen Methoden aendert.

das problem ist dass objectMain zu dem Zeitpunkt null ist.
Wofür brauchst du überhaupt ein seperates Main-Object in der Klasse?
GoAnim erweitert doch Main also steht setPosition1 auch GoAnim zur verfügung.
probiers mal mit this.setPosition1("PositionX", newPos,x,y);

Wie kann ein Objekt null sein ? Ich sage ja mainObject soll = new Main() sein (Tut mir leid für die Frage, ich bin neu in java )
Ich möchte ja eine Methode in der Klasse Main aufrufen und daher brauche ich ein Objekt. Ich habe gerade getestet und habe gesehen, dass man mainObject nicht braucht .
this.setPosition1(); klappt leider auch nicht. Ich bekomme wieder den selben Fehler.
Mit dem Befehl this.setLocation(); bekomme ich zwar keinen Fehler aber das Label bewegt sich nicht

 

[Joose]

Zeile 60 ist : "objectMain.setPosition1("PositionX", newPos, x, y);"

Dann ist "objectMain" null und daher auch die NullPointerException.

Ich möchte ja eine Methode in der Klasse Main aufrufen und daher brauche ich ein Objekt. Ich habe gerade getestet und habe gesehen, dass man mainObject nicht braucht .

Diese Objekt hast du ja theoretisch schon, daher müsstest du kein neues erzeugen.
Ja das brauchst du nicht weil deine Klasse GoAnim von Main ableitet. Sprich dein GoAnim Objekt ist auch ein Main und enthält somit auch alle Methoden die Main anbietet.

this.setPosition1(); klappt leider auch nicht. Ich bekomme wieder den selben Fehler.
Mit dem Befehl this.setLocation(); bekomme ich zwar keinen Fehler aber das Label bewegt sich nicht

Weil du ein komplett neues Objekt hast welches nicht angezeigt wird und an diesem deine Bewegung durchführst. Das Objekt was angezeigt wird bleibt von dir unverändert.

Wie schon oben gesagt fehlen dir die Grundlagen der OOP und die sind für ein Spiel notwendig!

 

[acevik94]

Für diejenigen die den selben Fehler haben. Nach einer Recherche habe ich endlich die Lösung gefunden .
Innerhalb oder ausserhalb der Klick-Methode erzeugt man zunaechst ein Objekt der Klasse. In meinem Fall
GoAnim objectGoAnim = new GoAnim();
Nun ruft man die Methode in der GoAnim-Klasse auf. !!!!!Wichtig !!!!! ist hier, dass man das Label, das man erzeugt hat (also playerOne) als Parameter mitgibt ! sonst funzt. das ganze nicht. Also zb:
objectGoAnim.rufeDieseMethode(JLabel playerOne);
In der rufeDieseMethode schreibt man dann einfach :
playerOne.setLocation(x,y);