Anfänger-Frage

[jochen_234]

Hallo,
bin absoluter Neuling wenns um Java geht, und verstehe nicht warum folgendes nicht läuft;
Hab 3 Klassen:

public class test {
	
	public static void main(String[] args) {
		Auto neuesAuto = new Auto();
		String farbe = neuesAuto.getLnkKarosserie().getFarbe();
		System.out.println(farbe);
		
}
}

public class Auto {
	
	
		private Karosserie lnkKarosserie;

		public Auto() {
			Karosserie lnkKarosserie = new Karosserie();
	}

		public Karosserie getLnkKarosserie() {
			return lnkKarosserie;
		}
}

public class Karosserie {
	
	private String farbe = "rot";

		public String getFarbe() {
		return farbe;
	}

	public void setFarbe(String farbe) {
		this.farbe = farbe;
	}

		public Karosserie() {
	}
}

Bekomme als Fehler :
Exception in thread "main" java.lang.NullPointerException
at test.main(test.java:6)
Also in der Zeile: String farbe = neuesAuto.getLnkKarosserie().getFarbe();

Warum geht das nicht, und vielleicht: wie kann man das geschickter anstellen, wenn man Karosserie als Attribut von Auto vewendet und die Farbe an die Klasse test übergeben will. Test soll sich halt die Farbe holen.

Danke im Voraus

[Edit by Beni: Codetags]

 

[Beni]

Stichwort "Überlagerung". Du definierst zwei verschiedene Variablen "lnkKarosserie" die miteinander soviel wie das Huhn und der Apfel zu tun haben:

      private Karosserie lnkKarosserie; // << einmal

      public Auto() {
         Karosserie lnkKarosserie = new Karosserie();  // << zweimal
   }

Besser ist:

      private Karosserie lnkKarosserie; // << definieren

      public Auto() {
         lnkKarosserie = new Karosserie(); // << zuweisen
   }

 

[bygones]

public Auto() { 
         Karosserie lnkKarosserie = new Karosserie(); 
   }

hiermit erstellst du eine lokale Variable namens lnkKarosserie. Diese hat nichts mit der instanzvariable zu tun....

public Auto() { 
         lnkKarosserie = new Karosserie(); 
   }

nunja - beni war schneller....

was man machen kann ist z.b. in der Klasse auto eine Methode setFarbe

public void setFarbe(String farbe) {
   lnKarosserie.setFarbe(farbe);
}

 

[jochen_234]

Ist mir jetzt klar, vielen Dank !!!

 

[MPW]

Und noch ein bisschen klugscheissern: Klassennamen schreibt man gross, also Test, nicht test;-)