Frage zu Labyrinth

[Barthsimpson]

import java.awt.Color;

public class Labyrinth {
	int anz_zeilen = 6,
	    anz_spalten = 10;
	
	
	Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
	
	public static void main (String args[]) {
		
		Labyrinth Lab = new Labyrinth();
		
		System.out.println();
		
		for(int i =0;i<(Lab.labfeld.length);i++){
			for(int j=0;j<(Lab.labfeld[0].length);j++){
				if (Lab.labfeld[i][j].isbelegt())System.out.print("*");
				}
				
			System.out.println();
		}
	}
}

class Zelle {
    public boolean belegt;
    Color farbe;
    public int zellid;
    
    static int lfdnr = 24726;						
    
    public Zelle (boolean belegt) {
	    this.belegt=belegt;
	    
	    if (belegt) farbe = Color.red;					
	    else farbe = Color.green;						
	    zellid = lfdnr++;								
	    
    }
    boolean isBelegt() {return belegt;}
}

Kompilieren lässt es sich, aber als Ausgabe erhalte ich dauernd:

Exception in thread "main" java.lang.NullPointerException
at Labyrinth.main(Labyrinth.java:19)


Wäre nett wenn mir jemand helfen könnte...

MfG
Barthsimpson

 

[SlaterB]

Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
erzeugt ein Array mit lauter null-Objekten, aber zumindest der richtigen Anzahl,

jedes Array-Feld muss zunächst noch mit new Zelle(); initialisiert werden,
in deinem Falle gibt es anscheinden nichtmal einen Konstruktor new Zelle();,
sondern es muss ein Parameter mitangegeben werden,

das zeigt, dass es unmöglich ist, in
Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
die Objekte automatisch anzulegen,
welcher Parameter sollte gewählt werden?

 

[Barthsimpson]

Sorry, bin noch neu hier, sollte vielleicht noch dazuschreiben was das Ding überhaupt machen soll:

Es geht darum ein Feld aus Zellen zu erstellen, dass dann irgendwann über belegte und nichtbelegte Zellen verfügen soll, was dann auch abrufbar und darstellbar sein soll...
Alle belegten Zellen sollen eben durch ein "*" dargestellt werden...

ich komm bei diesem Fehler gerade einfach nicht weiter....

 

[Barthsimpson]

Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
erzeugt ein Array mit lauter null-Objekten, aber zumindest der richtigen Anzahl,

jedes Array-Feld muss zunächst noch mit new Zelle(); initialisiert werden,
in deinem Falle gibt es anscheinden nichtmal einen Konstruktor new Zelle();,
sondern es muss ein Parameter mitangegeben werden,

das zeigt, dass es unmöglich ist, in
Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
die Objekte automatisch anzulegen,
welcher Parameter sollte gewählt werden?

Du meinst den bool'schen Wert belegt?

Der soll nachher mithilfe einer Zufallsfunktion festgelegt werden, das heisst, dass das Feld bei jedem Aufruf anders aussieht...

Würde mir jetzt schon helfen, wenn du mir sagst, wie ich es hinbekomme, dass der Wert z.Bsp. mit true an die Zellen übergeben wird!

Mit dem Rest was du schreibst, kann ich nicht wirklich was anfangen?! Sorry, bin halt da absoluter Noob :?

Danke auf jeden Fall schonmal!!!

 

[Barthsimpson]

Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
erzeugt ein Array mit lauter null-Objekten, aber zumindest der richtigen Anzahl,

jedes Array-Feld muss zunächst noch mit new Zelle(); initialisiert werden,
in deinem Falle gibt es anscheinden nichtmal einen Konstruktor new Zelle();,
sondern es muss ein Parameter mitangegeben werden,

das zeigt, dass es unmöglich ist, in
Zelle labfeld [][] = new Zelle [anz_zeilen][anz_spalten];
die Objekte automatisch anzulegen,
welcher Parameter sollte gewählt werden?

Ich glaube ich habe inzwischen verstanden, was du gemeint hast, und hab jetzt mal den Schleifeninhalt abgeändert:

for(int i =0;i<(Lab.labfeld.length);i++){
         for(int j=0;j<(Lab.labfeld[0].length);j++){
		 new Zelle (true);
		 if (Lab.labfeld[i][j].isBelegt())System.out.print("*");
                 }
}

Allerdings hilft das nichts, denn der Fehler bleibt nach wie vor derselbe...

(NullPointerException)

 

[DocRandom]

..klar, Du hast ja auch das Array der Zellen nich befüllt!

Ich habe mir erlaubt (diesmal ohne Main-Klasse) Deinen Code nachzubilden.
Allerdings so, das er auch lauffähig wäre.
Er besteht aus 2 Klassen:

• Zelle
• Labyrinth

Zelle.java

package barthsimpson;
import java.awt.Color;
import java.util.*;

public class Zelle {

	private boolean belegt;
	private Color farbe;
	private int zellID;
	private static int lfdNr = 24726;
	
	public Zelle () {
		Random rnd = new Random();
		belegt = rnd.nextBoolean();
		init();
	}
	public Zelle ( boolean belegt){
		this.belegt = belegt;
		init();
	}
	public boolean isBelegt(){
		return belegt;
	}
	public Color getFarbe(){
		return farbe;
	}
	public int getZellID() {
		return zellID;
	}
	private void init() {
		zellID = lfdNr++;
		if (belegt)
			farbe = Color.green;
		else
			farbe = Color.red;
	}
}

Labyrinth.java

package barthsimpson;

public class Labyrinth {
	
	private int zeilen;
	private int spalten;
	private Zelle labfeld[][] = null;
	
	public Labyrinth() {
		zeilen = 6;
		spalten = 10;
		initLabyrinth();
	}
	public Labyrinth( int zeilen, int spalten) {
		this.zeilen = zeilen;
		this.spalten = spalten;
		initLabyrinth();
	}
	public Zelle[][] getLabyrith() {
		return labfeld;
	}
	public int getZeilen() {
		return zeilen;
	}
	public int getSpalten() {
		return spalten;
	}
	private void initLabyrinth(){
		labfeld = new Zelle[zeilen][spalten];
		for (int zeile = 0; zeile < zeilen; zeile++) {
			for (int spalte = 0; spalte < spalten; spalte++){
				labfeld[zeile][spalte] = new Zelle();
			}
		}
	}
}

In der Funktion initLabyrinth wird das Array mit Zellen-Objekten gefüllt, sodass es eben zu keiner NullPointerException kommen kann.
Und genau das hat SlaterDB gemeint!

mfg
DocRandom

 

[Guest]

@DocRandom: Erstmal vielen Dank, du hast mir schonmal sehr geholfen, ich habe deine Lösung etwas abgewandelt in meinem Schleifkörper verwendet, wens interessiert:

for(int i =0;i<(Lab.labfeld.length);i++){
			for(int j=0;j<(Lab.labfeld[0].length);j++){
				
				boolean Rnd = (Math.random()>=0.3? false : true);  					
				Lab.labfeld[i][j]  = new Zelle (Rnd);					
				
				if(Lab.labfeld[i][j].isBelegt()) System.out.print("*");
					else System.out.print ("-");
			}
				
			System.out.println();
		}
[quote]
Und das funktioniert!!!

Das einzige was also im Endeffekt gefehlt hat war das Füllen des Feldes mit Zellen...  :oops: 


Vielen Dank an alle die hier geposted haben  :toll: 

MfG
Barthsimpson[/quote]

 

[Barthsimpson]

@DocRandom: Erstmal vielen Dank, du hast mir schonmal sehr geholfen, ich habe deine Lösung etwas abgewandelt in meinem Schleifkörper verwendet, wens interessiert:

for(int i =0;i<(Lab.labfeld.length);i++){
			for(int j=0;j<(Lab.labfeld[0].length);j++){
				
				boolean Rnd = (Math.random()>=0.3? false : true);  					
				Lab.labfeld[i][j]  = new Zelle (Rnd);					
				
				if(Lab.labfeld[i][j].isBelegt()) System.out.print("*");
					else System.out.print ("-");
			}
				
			System.out.println();
		}

Und das funktioniert!!!

Das einzige was also im Endeffekt gefehlt hat war das Füllen des Feldes mit Zellen...


Vielen Dank an alle die hier geposted haben :toll:

MfG
Barthsimpson

P.S.: Sorry 4 Doppelpost[/code]