array frage

[Beginner007]

public class Karte {
	String farbe, rang;
	int wert;
	boolean istAktivImSpiel;
}



public class Game {

	public static void main(String[] args) {

		Karte[] karten = new Karte [31];
		String farben[] = {"Kreuz", "Pik", "Herz", "Karo"};
		String rang[] = {"Sieben", "Acht", "Neun", "Zehn", "Bube", "Dame", "Koenig","Ass"};
		
		karten[0].farbe = "Kreuz";

	}

}

java.lang.NullPointerException
Game.java:14

Ich kann nicht verstehen, warum Fehler zeigt. hilfe bitte
Danke im Voraus.

MfG
Beginner007

 

[SlaterB]

karten[0] ist null,

nirgendwo steht
karten[0] = new Karte();

Java macht das nicht von allein

 

[Guest]

public class Karte { 
   String farbe, rang; 
   int wert; 
   boolean istAktivImSpiel; 
}

public class Game {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Karte[] karten = new Karte [31];
		int temp;
		String farben[] = {"Kreuz", "Pik", "Herz", "Karo"};
		String rang[] = {"Sieben", "Acht", "Neun", "Zehn", "Bube", "Dame", "Koenig","Ass"};
		
		karten[0].rang = "Kreuz"; 

   } 

}

 

[jPat]

Es steht immer noch nicht drin, oder?

 

[Niki]

Wenn du eine Array mit new Karte[31] anlegst. Werden die einzelnen Felder des Arrays mit "null" initialisiert. Du greifst aufs erste Element mit karten[0] zu, obwohl das Element auf kein Objekt zeigt. mach:

 karten[0] = new Karte();
 karten[0].rang = "was auch immer";

 

[Guest]

danke euch für schnelle antwort, dank euch hab den fehler korrigiert.

Die richtige Ist:

public class Game {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Karte[] karten = new Karte [31];
		
		int temp;
		String farben[] = {"Kreuz", "Pik", "Herz", "Karo"};
		String rang[] = {"sieben", "acht", "neun", "zehn", "bube", "dame", "koenig","ass"};
		
		for(int k=0; k<karten.length; k++){
			karten[k] = new Karte();			
			if (0 == k) temp = 0; else temp = (int)(k/8);			
			karten[k].farbe	= farben[temp];			
			karten[k].rang	= rang[k%8];
			System.out.println(karten[k].farbe + karten[k].rang);			
		}
		
	}

}

 

[Niki]

danke euch für schnelle antwort, dank euch hab den fehler korrigiert.

Die richtige Ist:

			if (0 == k) temp = 0; else temp = (int)(k/8);

etwas angenehmer zu lesen ist:

			temp = k == 0 ? 0 : k/8;

 

[SlaterB]

temp = k/8;
dürfte auch hinkommen

 

[Murray]

Noch ein paar Tipps dazu:

1. den Code

if (0 == k) temp = 0; else temp = (int)(k/8);

kann man mit dem ternären Operator etwas kürzer schreiben:

temp = (k == 0) ? 0 : (k/8);

//EDIT: viel zu langsam - und Slater hat natürlich recht; in diesem Fall braucht man die Fallunterscheidung eigentlich überhaupt nicht.



2. Mach die Member der Klasse Karte lieber private und definiere für den Zugriff eigene get- und (ggfs.) set-Methoden. Überlege dabei, ob sich die jeweiligen Werte an einer Instanz jeweils während ihrer Lebensdauer ändern können (würde ich für Farbe und Rang eine Karte jetzt erstmal bezweifeln). Ist ein Member seiner Natur nach unveränderlich, setze es auf final und weise es direkt im Konstruktor zu.

public class Karte {
   private final String farbe;
   private final String rang;
   /* ... */

  public Karte( final String farbe, final String rang) {
     this.farbe = farbe;
     this.rang = rang;
  }

  public String getFarbe() {
    return farbe;
  }

  public String getRang() {
    return farbe;
  }

  /* ... */
}

3. Wenn ein Feld einen festen Wertebereich haben kann, dann bietet es sich meistens an, dafür einen enum-Typ zu verwenden. Damit ist man wesentlich typsicherer, als wenn man beliebige String zulässt; dann prüft nämlich bereits der Compiler, ob z.B. der Paramter für die Frabe einen erlaubten Wert hat.

 

[Guest]

Erstmal vielen dank für die Antworten! :toll:
hab umgeschrieben, wie ihr geraten habt.

public class Game {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		KartenSet neuSet = new KartenSet(400);
		for (int i=0; i< neuSet.karten.length; i++){

			System.out.println(neuSet.karten[i].getFarbe() + neuSet.karten[i].getRang());
		}
		
		
		
	}

}

public class Karte {
	private final String farbe;
	private final String rang;
	private int wert;
	private boolean istAktiv;
	
	
	//Der Konstruktor Karte(farbe, rang) setzt Kartenfrabe und Kartenrang.
	public Karte( final String farbe, final String rang) { 
	     this.farbe = farbe; 
	     this.rang = rang; 
	}
	
	//Die Methode getFarbe() gibt die kartenFarbe zurück.(als String)	
	public String getFarbe(){
		return this.farbe;
	}
	
	//Die Methode setRang() gibt der Rang der Karte zurück.(als String) 
	public String getRang(){
		return this.rang;
	}
	
	//Die Methode setWert(int) setzt wert.
	public void setWert(int wert){
		this.wert = wert;
	}
	
	//Methode getWert() gibt wert zurück.(als int)
	public int getWert(){
		return this.wert;
	}
	
	//Methode setIstAktiv setzt istAktiv.
	public void setIstAktiv(boolean istAktiv){
		this.istAktiv = istAktiv;
	}
	
	//Methode getIstAktiv gibt istAktiv zurück.(als boolean)
	public boolean getIstAktiv(){
		return this.istAktiv;
	}
}

import java.util.Random;


public class KartenSet {
Karte karten []=  new Karte[31];
	
	KartenSet(int mixFactor){
		
		String farben[] = {"Kreuz", "Pik", "Herz", "Karo"};
		String rang[] = {"sieben", "acht", "neun", "zehn", "bube", "dame", "koenig","ass"};
		
		for(int k=0; k<karten.length; k++){			
			karten[k] = new Karte(farben[k/8], rang[k%8]);			
						
		}
		this.mixKarten(mixFactor);
		
		
	}
	public void mixKarten(int mixFactor){
		Random generator = new Random();
		
		for (int i=0; i < mixFactor; i++){
			int r1 = generator.nextInt(31);
			int r2 = generator.nextInt(31);
			//Kartentaush
			Karte temp 	= this.karten[r1];
			this.karten[r1] 	= this.karten[r2];
			this.karten[r2] 	= temp;
		}
		
	}
}

3. Wenn ein Feld einen festen Wertebereich haben kann, dann bietet es sich meistens an, dafür einen enum-Typ zu verwenden. Damit ist man wesentlich typsicherer, als wenn man beliebige String zulässt; dann prüft nämlich bereits der Compiler, ob z.B. der Paramter für die Frabe einen erlaubten Wert hat.

Ich habe das Kapitel "Aufzählungen(enum)" noch nicht gelesen. :###

 

[Murray]

Ich habe das Kapitel "Aufzählungen(enum)" noch nicht gelesen. :###

Das ist kein Hexenwerk, aber man muss sich natürlich auch nicht sofort damit belasten; funktionieren wird es auch so.
Grundsätzlich würde es nur darum gehen, statt der String-Arrays für Farben und Rang jeweils einen Aufzählungstyp zu definieren:

enum Farbe {Kreuz, Pik, Herz, Karo};
enum Rang {sieben, acht, neun, zehn, bube, dame, koenig,ass};

Dann würde man die Member und natürlich auch die Zugriffsmethoden bzw. Konstruktorparameter mit diesen Typen statt mit Strings definieren - das ändert am Programm nicht viel, erlaubt es aber eben, bereits zur Compile-Zeit zu prüfen, ob nur erlaubte Werte verwendet werden.