Arrays clonen mit clone() scheint nicht zu funktionieren

[maatze]

Hallo allerseits,

ich habe eine Klasse, deren Konstruktor folgendermassen lautet:

...

int[][] quadrat;
int magicsum;

public MagicSquare(int dimension,int[][] quadrat)
	{
		magicsum = dimension*((dimension*dimension)+1)/2;
		this.dimension = dimension;
		this.quadrat = (int[][])quadrat.clone();
						
	}

An einer anderen Stelle erstelle ich wiederum ein Array aus eben Objekten dieser Klasse und verändere mit Hilfe eigener public - Methoden (von MagicSquare) das Array "quadrat". Das mache ich, indem ich eine Schleife durchlaufen lasse und somit die einzelnen Objekte anspreche. Problem ist, dass nach dem Schleifendurchlauf alle Objekte dieselben Werte im Array haben.

Ich denke das liegt daran, dass das Array nicht richtig geclont wurde, sondern die einzelnen Objekte jeweils nur einen Verweis auf das übergebene Array besitzen. Ich dachte allerdings, dass man richtige Kopien mit clone() (oder arraycopy() )erstellt. Oder wie soll das funktionieren.

Ich hoffe ich konnte mich verständlich machen. Danke für eventuelle Antworten.

gruss maatze

 

[Beni]

"clone" ist ein bisschen schwammig definiert, und macht mal das eine, mal das andere...
"arraycopy" erstellt keine neuen Objekte, sondern kopiert nur den Link.

Du musst selbst eine Schlaufe machen, die dir einen neuen Array erstellt. (Du kannst ja "clone" im Quadrat selber implementieren :wink: )

Quadrat[] newArray = new Quadrat[...];
for( ... )
  newArray[i] = new Quadrat( ... );

mfg Beni

 

[maatze]

ich bin mir nicht ganz sicher, ob Du mich verstanden hast. Ich galube ich habe das schon gemacht, was du geschrieben hast. Nochmal kurz: Ich habe eine Klasse MagicSquare (Konstruktor siehe oben), von der jeder ihrer Instanzen ein Array enthält. Von dieser Klasse erstelle ich mehrere Instanzen und speichere auch diese wieder in ein Array. Das mache ich so:

...

kinder = new MagicSquare[lambda];
// elter ist ebnefalls vom Typ MagicSquare und bekannt.
		
            }
	
	
            public void muLambdaStrategie()
            {	
	
            for (int i=0; i<lambda; i++)
            {
	    kinder[i]=new MagicSquare(elter.dimension,elter.quadrat);
	    kinder[i].shift2Numbers();
            }

           kinder[0].quadrat[0][0]=0;

           for (int i=0; i<lambda; i++)
           {
                kinder[i].showSquare(); //gibt einfach quadrat aus
            }

Letzte Schleife gibt mit für jedes der Kinder an der Stelle [0][0] eine 0 aus, was ja eigentlich nicht sein sollte.

 

[Beni]

Ich hab deinen Post wohl zu schnell durchgelesen :cry:

Ich hab mal

        int [][] test = new int[2][2];
        
        test[0][0] = 1;
        test[0][1] = 2;
        test[1][0] = 3;
        test[1][1] = 4;
        
        int[][] clone = (int[][])test.clone();

durch einen Debugger gejagt und

erhalten.

Nur der äussere Array wird kopiert, die inneren bleiben.

Also muss da doch noch eine Schlaufe rein:

int[][] newQuadrat = new int[ ... ][];

for( ... )
  newQuadrat[i] = (int[])oldQuadrat[i].clone();

Passt die Antwort besser zur Frage? :roll: ???:L :bae:

mfg Beni

 

[Guest]

Die Antwort passt perfekt.

Vielen Dank!