Zuweisung von gesicherten Werten funktioniert nicht

[JRookie]

Guten Abend zusammen,
ich möchte einen array von Objekten in einem "Sicherungsobjekt" speichern und dann wieder zurückholen ... und das funktioniert nicht.
Ganz einfach:

public class Autos {

	public static void main(String[] args) {
	
	kfz[] a1				= new kfz [3] ;	
	kfz[] a2				= new kfz [3] ;	
	Werkstatt [] spst		= new Werkstatt [99] ;

     	a1[0] = new kfz() ;
		a1[0].setName("Audi");
		a1[0].setAnz_raeder(4);	
		
	    a1[1] = new kfz() ;
		a1[1].setName("VW");
		a1[1].setAnz_raeder(5);	


		for (int i = 0; i<2;i++)  {
			System.out.println("Hier Autos: " + a1[i].getName() + "   Raeder: " + a1[i].getAnz_raeder() ); 

		}
		
		spst[0]	= new Werkstatt() ;	// der Versuch, alles in Werkstatt zu sichern
		spst[0].setUps(a1);
		
		a1[0].setName("XXX");		// ändere die bisherigen Werte
		a1[0].setAnz_raeder(14);	
		
	     a1[1] = new kfz() ;
		a1[1].setName("YYY");
		a1[1].setAnz_raeder(15);	
		
		a2	= spst[0].getUps();		// der Versuch, die alten Werte aus Werkstatt zurückzuholen
		
		for (int i = 0; i<2;i++)  {
			System.out.println("Hier Autos: " + a2[i].getName() + "   Raeder: " + a2[i].getAnz_raeder() ); 

		}
	}
}

die Klasse kfz:

public class kfz {
	private String name;
	private int	anz_raeder;


	public String getName() {
		return name;
	}

	public void setName(String name) {
		this.name = name;
	}

	public int getAnz_raeder() {
		return anz_raeder;
	}

	public void setAnz_raeder(int anz_raeder) {
		this.anz_raeder = anz_raeder;
	}

}

und die "Sicherungsklasse:

public class Werkstatt {
	 kfz[] ups	= new kfz [3] ;	
	 
	    public Werkstatt()
	    {

	    }

		public kfz[] getUps() {
			return ups;
		}

		public void setUps(kfz[] ups) {
			this.ups = ups;
		}
}

Als output gibt es dieses:

Hier Autos: Audi   Raeder: 4
Hier Autos: VW   Raeder: 5
Hier Autos: XXX   Raeder: 14
Hier Autos: YYY   Raeder: 15

Warum kann ich die "alten" Werte nicht wieder aus dem grade erzeugten Objekt spst[0] wieder rauszaubern ?? Echt simpel eigentlich ... aber geht nicht.
Weiß jemand warum ?? Und wie geht's ??
Danke im voraus
JRookie

 

[Gast2]

Du setzt nur Referenzen. ups in Werkstatt und a1 zeigen auf der selbe Array. Änderst du am Array was über die Referenz a1, dann sind die Änderungen auch bei ups vorhanden.
Wenn du da was "sicher" willst dann musst du ne Kopie erzeugen.

 

[JRookie]

Ich denk mal, du hast recht, weil das Fehlerbild so erklärbar ist.
Aber verstehen tu ich es nicht: ich erzeuge zwei kfz Objekte in der main Klasse (a1 und a2) und ein kfz Objekt in der Werkstatt Klasse (ups). Eigentlich haben die doch nix miteinander zu tun ... so wie 3 int Werte verteilt auf 2 Klassen.
Wodurch kommt die Verbindung von a1 und ups zustande ????

 

[Gast2]

In der Klasse Werkstatt erstellst du nur ein Array, dass 3 kfz Objekte halten kann.
Die kfz Objekte (2 Stück) erstellst du in der Klasse Autos. Am ende deines Programms zeigen a1, a2 und ups alle auf die SELBEN beiden kfz Objekte.

 

[JRookie]

Und so gehts auch nicht : Hab jetzt mal a2 als "Aufbewahrungsort" genommen:

public class Autos {

	public static void main(String[] args) {
	
	kfz[] a1				= new kfz [3] ;	
	kfz[] a2				= new kfz [3] ;	
	Werkstatt [] spst		= new Werkstatt [99] ;

     	a1[0] = new kfz() ;
		a1[0].setName("Audi");
		a1[0].setAnz_raeder(4);	
		
	    a1[1] = new kfz() ;
		a1[1].setName("VW");
		a1[1].setAnz_raeder(5);	


		for (int i = 0; i<2;i++)  {
			System.out.println("Hier Autos: " + a1[i].getName() + "   Raeder: " + a1[i].getAnz_raeder() ); 

		}
		
		spst[0]	= new Werkstatt() ;	// der Versuch, alles in Werkstatt zu sichern

		spst[0].setUps(a1);
		a2	= a1 ;
		a1[0].setName("XXX");		// ändere die bisherigen Werte
		a1[0].setAnz_raeder(14);	
		
	     a1[1] = new kfz() ;
		a1[1].setName("YYY");
		a1[1].setAnz_raeder(15);	
		
	//	a2	= spst[0].getUps();		// der Versuch, die alten Werte aus Werkstatt zurückzuholen
		
		for (int i = 0; i<2;i++)  {
			System.out.println("Hier Autos: " + a1[i].getName() + "   Raeder: " + a1[i].getAnz_raeder() ); 

		}
		
		a1	= a2 ;
		for (int i = 0; i<2;i++)  {
			System.out.println("Hier Autos: " + a1[i].getName() + "   Raeder: " + a1[i].getAnz_raeder() ); 

		}
	}
}

liefert als Output:

Hier Autos: Audi   Raeder: 4
Hier Autos: VW   Raeder: 5
Hier Autos: XXX   Raeder: 14
Hier Autos: YYY   Raeder: 15
Hier Autos: XXX   Raeder: 14
Hier Autos: YYY   Raeder: 15

Wie schaffe ich dann 2 kfz Objekte, die voneinander unabhängig sind ??

 

[Gast2]

Wenn du unterschiedliche Objekte haben willst, dann darfst du nicht die SELBEN nehmen.
a1 und a2 sind bei dir ein und das selbe Objekt. Änderungen bei a1 siehst du auch an a2.

Wie oben schon geschrieben musst du eine Kopie erzeugen.

 

[irgendjemand]

Da ist nichts unabhängig von einander.
Was du machst ist lediglich noch mehr Referenz auf EIN UND DASSELBE Objekt zu erzeugen ... anstatt neue eigenständige Objekte zu erzeugen.

Wenn du also z.B. ein Array in einem "Sicherungsobjekt" "sichern" willst musst du vom INHALT des Arrays eine Kopie erzeugen und diese Kopie *viel mehr : die neue Referenz auf dieses neue Kopie-Objekt* speichern ... und nicht einfach eine weitere Referenz auf das original-Objekt verwenden.

 

[JRookie]

Tja, also irgendwie sollte es dann mit " .... implements Cloneable " und dann " a2 = (kfz)a1.clone(); " gehen. Das klappt noch nicht so ganz ...
Ich hab aber hier noch eine Möglichkeit entdeckt, das entspricht evt. dem was "irgendjemand" schreibt, um den Inhalt zu kopieren:

public class Autos   {

	public static void main(String[] args) {
	
	kfz[] a1				= new kfz [3] ;	
	kfz[] a2				= new kfz [3] ;	
	Werkstatt [] spst		        = new Werkstatt [99] ;

     	a1[0] = new kfz() ;
	a1[0].setName("Audi");
	a1[0].setAnz_raeder(4);	
		
        a1[1] = new kfz() ;
	a1[1].setName("VW");
	a1[1].setAnz_raeder(5);	


	for (int i = 0; i<2;i++)  {
		System.out.println("Hier Autos: " + a1[i].getName() + "   Raeder: " + a1[i].getAnz_raeder() ); 

	}
		
		
	for (int i = 0; i<2;i++)  {			// so gehts
	    a2[i]		= (kfz) a1[i];
	}
	
        a1[0] = new kfz() ;
	a1[0].setName("XXX");		// ändere die bisherigen Werte
	a1[0].setAnz_raeder(14);	
		
	a1[1] = new kfz() ;
        a1[1].setName("YYY");
	a1[1].setAnz_raeder(15);	
		
	for (int i = 0; i<2;i++)  {
		System.out.println("Hier Autos: " + a1[i].getName() + "   Raeder: " + a1[i].getAnz_raeder() ); 

	}
		

	for (int i = 0; i<2;i++)  {
		System.out.println("Hier Autos: " + a2[i].getName() + "   Raeder: " + a2[i].getAnz_raeder() ); 

	}
   }
}

Aber was mach ich da ?

a2[i]		= (kfz) a1[i];

Ich kopiere die einzelnen array Elemente? Der Output sieht so aus:

Hier Autos: Audi   Raeder: 4
Hier Autos: VW   Raeder: 5
Hier Autos: XXX   Raeder: 14
Hier Autos: YYY   Raeder: 15
Hier Autos: Audi   Raeder: 4
Hier Autos: VW   Raeder: 5

 

[JRookie]

Aha, Variablen gibts nur bei elementaren Datentypen (int, double,char etc.). Bei Objekten gibt es nur Objektreferenzvariablen, die die Adresse des Objektes enthalten.
zu Cloneable:

public class Autos  implements Cloneable {
.......
.......
		a2		= (kfz[]) a1.clone() ;

So gehts !
Danke an alle! Ich mach hier mal zu.
JRookie

 

[Gast2]

Verwende statt dem Cloneable lieber nen Copy-Konstruktor oder ne Copy-Factory.
Und ich meine auch, dass clone keine deep copy erzeugt, die Referenzen sollten also nach dem Klon immernoch aus die selben kfz Objekte zeigen.