kann methode nicht ausführen

[Tom1101]

ich muss für die schule ein projekt machen und habe keine möglichkeit mehr meine lehrerin zu fragen was für einen fehler ich programmiert habe.
Wenn ich die Methode s2setzen() der Klasse SPIELER2 aufrufe steht da immer "java.lang.NullPointerException:null" ich weiß, dass es hier öfter schon erklärt wurde, aber unter verwendung eines für mich zu komplizierten quellcodes. Hier mein Quellcode:

public class SPIELER2
{
    //Attribute
    private ZEIGER zeiger;
    private SPIELFELD spiel;

    protected String name2;

    public SPIELER2(String benName){
        name2 = benName;
    }

    //Methoden
    public void s2setzen(){
        if(zeiger.zeigerZustand == 1 && spiel.status[0] == 0){
            spiel.spieler1steine[0].sichtbarMachen();
            spiel.status[0] = 2;
        }else{
            if(zeiger.zeigerZustand == 2 && spiel.status[1] == 0){
                spiel.spieler1steine[1].sichtbarMachen();
                spiel.status[1] = 2;
            }else{
                if(zeiger.zeigerZustand == 3 && spiel.status[2] == 0){
                    spiel.spieler1steine[2].sichtbarMachen();
                    spiel.status[2] = 2;
                }else{
                    if(zeiger.zeigerZustand == 11 && spiel.status[3] == 0){
                        spiel.spieler1steine[3].sichtbarMachen();
                        spiel.status[3] = 2;
                    }else{
                        if(zeiger.zeigerZustand == 12 && spiel.status[4] == 0){
                            spiel.spieler1steine[4].sichtbarMachen();
                            spiel.status[4] = 2;
                        }else{
                            if(zeiger.zeigerZustand == 13 && spiel.status[5] == 0){
                                spiel.spieler1steine[5].sichtbarMachen();
                                spiel.status[5] = 2;
                            }else{
                                if(zeiger.zeigerZustand == 21 && spiel.status[6] == 0){
                                    spiel.spieler1steine[6].sichtbarMachen();
                                    spiel.status[6] = 2;
                                }else{
                                    if(zeiger.zeigerZustand == 22 && spiel.status[7] == 0){
                                        spiel.spieler1steine[7].sichtbarMachen();
                                        spiel.status[7] = 2;
                                    }else{
                                        if(zeiger.zeigerZustand == 23 && spiel.status[8] == 0){
                                            spiel.spieler1steine[8].sichtbarMachen();
                                            spiel.status[8] = 2;
                                        }else{
                                            System.out.println("Dieses Feld ist bereits besetzt.");
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }

}

public class SPIELFELD  {
    //Attribute
    private RECHTECK linie1;
    private RECHTECK linie2;
    private RECHTECK linie3;
    private RECHTECK linie4;

    //Spielsteine
    protected RECHTECK [] spieler1steine;
    protected KREIS [] spieler2steine;

    //Status
    protected int [] status;

    public SPIELFELD (){
        status = new int[9];
        linie1 = new RECHTECK(150, 50, 2, 300, "schwarz", true);
        linie2 = new RECHTECK(250, 50, 2, 300, "schwarz", true);
        linie3 = new RECHTECK(50, 150, 300, 2, "schwarz", true);
        linie4 = new RECHTECK(50, 250, 300, 2, "schwarz", true);

        spieler1steine= new RECHTECK[9];
        spieler2steine= new KREIS[9];

        for(int i=0; i<3; i++){
            spieler1steine[i] = new RECHTECK((i+1)*90, 75, 50, 50, "lila", false);
            spieler1steine[i+3] = new RECHTECK((i+1)*90, 175, 50, 50, "lila", false);
            spieler1steine[i+6] = new RECHTECK((i+1)*90, 275, 50, 50, "lila", false);

            spieler2steine[i] = new KREIS(50, (i+1)*90, 75, "gruen", false);
            spieler2steine[i+3] = new KREIS(50, (i+1)*90, 175, "gruen", false);
            spieler2steine[i+6] = new KREIS(50, (i+1)*90, 275, "gruen", false);
        }
    }
    
    public void auf0setzen(){
        for(int i=0; i<9;i++){
            status[i]=0;
            System.out.println(status[i]);
        }
    }

   
}

sry falls ich was falsch gemacht hab wegen forum Regeln ... bin neu hier

 

[Anti-Banane]

zu deiner ersten klasse : switch-case
generell : Conevtions !
zum fehler : nur "NPE" reicht nicht ... wenn dann brauchen wir den ganzen stacktrace ... oder zumindest die zeile (sollte zum code passen) in der die NPE geworfen wird

 

[Tom1101]

switch-case und conevtions sagen mir garnichts, bin ja anfänger.

[JAVA=15]if(zeiger.zeigerZustand == 1 && spiel.status[0] == 0){[/code]

da tritt die NPE in der klasse SPIELER2 auf.

das ist alles was da steht:

java.lang.NullPointerException
at SPIELER1.s1setzen(SPIELER1.java:15)

 

[Phash]

zeiger und spieler werden nie erzeugt, du musst irgendwo mal ein
zeiger = new Zeiger() und ein spieler = new Spieler(); machen
In deinem SPIELER2 Konstruktor

Klassen schreibt man aber bitte
UpperCamelCase anstatt ALLESGROSS

 

[trez]

switch-case und conevtions sagen mir garnichts, bin ja anfänger

Switch case geht in diesem Fall wohl nicht einfach so und "Conventions" alleine sagt auch nicht gerade viel aus, aber sieht das nicht eleganter aus?

	public void s2setzen() {
		if (zeiger.zeigerZustand == 1 && spiel.status[0] == 0) {
			spiel.spieler1steine[0].sichtbarMachen();
			spiel.status[0] = 2;
		} else if (zeiger.zeigerZustand == 2 && spiel.status[1] == 0) {
			spiel.spieler1steine[1].sichtbarMachen();
			spiel.status[1] = 2;
		} else  if (zeiger.zeigerZustand == 3 && spiel.status[2] == 0) {
			spiel.spieler1steine[2].sichtbarMachen();
			spiel.status[2] = 2;
		} else  if (zeiger.zeigerZustand == 11 && spiel.status[3] == 0) {
			spiel.spieler1steine[3].sichtbarMachen();
			spiel.status[3] = 2;
		} else  if (zeiger.zeigerZustand == 12 && spiel.status[4] == 0) {
			spiel.spieler1steine[4].sichtbarMachen();
			spiel.status[4] = 2;
		} else  if (zeiger.zeigerZustand == 13 && spiel.status[5] == 0) {
			spiel.spieler1steine[5].sichtbarMachen();
			spiel.status[5] = 2;
		} else  if (zeiger.zeigerZustand == 21 && spiel.status[6] == 0) {
			spiel.spieler1steine[6].sichtbarMachen();
			spiel.status[6] = 2;
		} else  if (zeiger.zeigerZustand == 22 && spiel.status[7] == 0) {
			spiel.spieler1steine[7].sichtbarMachen();
			spiel.status[7] = 2;
		} else  if (zeiger.zeigerZustand == 23 && spiel.status[8] == 0) {
			spiel.spieler1steine[8].sichtbarMachen();
			spiel.status[8] = 2;
		} else {
			System.out.println("Dieses Feld ist bereits besetzt.");
		}
	}

 

[njans]

Oder mal kürzer:

	int[] zustaende = {1,2,3,11,12,13,21,22,23};
		
		for (int i = 0; i < zustaende.length; i++)
		{
			if (zeiger.zeigerZustand  == zustaende[i] && spiel.status[i] == 0)
			{
				spiel.spieler1steine[i].sichtbarMachen();
	            spiel.status[i] = 2;
                    break; //return;
			}
		}