Aufruf von Klasse verändert ungewollt Werte

[deeuu]

Hallo,

ich habe eine Matrix

          double A[][] = {{-1,0,0},{0,-1,0},{0,0,-1},{1,1,1}};
	       double B[]   = {0,0,0,1};

gegeben. Ich möchste nun dass mein Programm daraus alle Kombinationen von 3x3 Matrizen macht.
Aus der Matrix A wird also:
-1.0 0.0 0.0
0.0 -1.0 0.0
0.0 0.0 -1.0

-1.0 0.0 0.0
0.0 -1.0 0.0
1.0 1.0 1.0

-1.0 0.0 0.0
0.0 0.0 -1.0
1.0 1.0 1.0

0.0 -1.0 0.0
0.0 0.0 -1.0
1.0 1.0 1.0

Das funktioniert auch wunderbar. Nun will ich jede dieser Teilmatritzen an eine Klasse übergeben welche mir diese Gleichungssysteme löst und die Lösung dann zurückgibt. Die Klasse LGS das hab ich von
http://www.java-forum.org/de/viewtopic.php?p=435704 übernommen.
Das funktioniert leider aber nicht. Nachdem LGS aufgerufen wird verändert sich meine Matrix a, nämlich in:

-1.0 0.0 0.0
-0.0 -1.0 0.0
-0.0 -0.0 -1.0

-1.0 0.0 0.0
0.0 -1.0 0.0
-1.0 -1.0 1.0

-1.0 0.0 0.0
0.0 0.0 -1.0
1.0 -1.0 1.0

0.0 -1.0 0.0
0.0 -0.0 -1.0
1.0 -1.0 1.0

Ich will eigentlich nur dass ich die Matrix a an LGS übergebe, LGS dann die Lösung zurückgibt und die Schleife weiterläuft, ohne das in der Klasse "Schnittpunkt" irgendwas verändert wird. Wie mach ich das?

static  Punkt[] Schnittpunkt(double[][] array,double [] erg,int i)
	{
		Punkt Schnittpunkte []= new Punkt[20];
		double a [][]=new double[3][3];
		double b []=new double[3];
		int zaehler=-1;
		
		for(int j=0;j<=i-3;j++)
		{
		    for(int k=j+1;k<=i-2;k++) 
		    {
		    	for(int l=k+1;l<=i-1;l++)             //Diese 3 Schleifen erzeugen die Teilmatrizen.
			    {                                           // Das funktioniert soweit auch  :)
		           a[0]=array[j];
		           a[1]=array[k]; 
		           a[2]=array[l];
		      
		           b[0]=erg[j];
		           b[1]=erg[k];
		           b[2]=erg[l];
		                
		      double[] Loesung = LGS.solve(a, b);                 //Hier ist das Problem: Nachdem LGS aufgerufen wird
		                                                          // werden die Teilmatrizen nicht mehr richtig erzeugt.
		         double x=Loesung[0];
		         double y=Loesung[1];
		         double z=Loesung[2];
		        
		    zaehler++;

		    Schnittpunkte[zaehler] = new Punkt(x, y, z);
			    }
		    }
	    }
		return Schnittpunkte;
	}

 

[SlaterB]

übergib nicht a und/ oder b an LGS.solve(), sondern Kopien deiner Matrizen,
oder ändere die solve()-Methode, so dass sie sich selber die Parameter kopiert

 

[diggaa1984]

public static double[] solve(double[][] A, double[] b) 
   { 
      double[][] B = A.clone(); 
      double[] x = b.clone();
      <...>
   }

sollte das damit nicht schon geschehen? Hierbei reicht ja auch ne flache Kopie, sind ja nur double drin

 

[SlaterB]

oh, gar nicht gesehen,

leider wird nur auf erster Ebene flach kopiert, A und B sind zwar unterschiedliche Objekte,
aber A[0], das double[] der Länge 3, ist gleich B[0], A[1] == B[1], usw

 

[diggaa1984]

aso aber quasi A[0][1] wird nicht mitkopiert? Würde dabei eine default-initialisierung mit 0.0 erfolgen? denke mal schon das java es mitbekommt das A 2 dimensionen hat. nur clone orientiert sich nur an der 1. :?:

 

[SlaterB]

> aso aber quasi A[0][1] wird nicht mitkopiert?

da A[0] nur ein Objekt ist, welches in beiden Arrays drinsteckt,
macht deine Aussage keinen rechten Sinn,

A[0][1] bleibt wie es ist und ist trotzdem auch in B drin, denn B entspricht weitgehend A

> denke mal schon das java es mitbekommt das A 2 dimensionen hat.

nicht wirklich, es wird gar nicht nachgeschaut

 

[deeuu]

Mit clone() hatte ich schonmal alles mögliche probiert ... hat alles nicht funktioniert.
Ich habs jetzt einfach damit gemacht:

for(int m = 0; m <= 2; m++) 
{
  for(int n = 0; n <= 2; n++) 
	 {
		A[m][n]=a[m][n];
		B[m]=b[m];
	 }
		   	      
}

[/code]

Die Kopien übergeb ich dann an LGS ... und siehe da es funktioniert

Das ist bestimmt nicht der eleganteste Weg aber für meine Zwecke funktionierts.
Danke für die schnellen Antworten.

Mfg

 

[diggaa1984]

leider wird nur auf erster Ebene flach kopiert, A und B sind zwar unterschiedliche Objekte,
aber A[0], das double[] der Länge 3, ist gleich B[0], A[1] == B[1], usw

aso ich habs nu glaube ... also A[0] und B[0] sind verschieden, aber B[0][1] zeigt auf das selbe Objekt wie A[0][1] .. und daher eine Änderung in B = Änderung am original

 

[SlaterB]

A[0] und B[0] sind exakt dasselbe Objekt, deshalb ist auch B[0][1] dasselbe wie A[0][1],
in diesem Fall allerdings kein Objekt sondern eine double-Variable

 

[diggaa1984]

aso, also bewirkt clone gar nicht das was ich mir die ganze zeit darunter verstehe, nämlich das der inhalt von A[0] damit an B[0] kopiert wird, und somit doppelt vorliegen würde.

 

[SlaterB]

so isses

 

[diggaa1984]

warum sagst das nich gleich ... hm muss ich also nochma kieken was da genau passiert.
danke, bin ich wieder schlauer

 

[SlaterB]

leider wird nur auf erster Ebene flach kopiert, A und B sind zwar unterschiedliche Objekte,
aber A[0], das double[] der Länge 3, ist gleich B[0], A[1] == B[1], usw

 

[diggaa1984]

ja is mir schon klar, mir fehlte nur das gedankliche verständnis

:### Note that all arrays are considered to implement the interface Cloneable. Otherwise, this method creates a new instance of the class of this object and initializes all its fields with exactly the contents of the corresponding fields of this object, as if by assignment; the contents of the fields are not themselves cloned. Thus, this method performs a "shallow copy" of this object, not a "deep copy" operation. ???:L

also .. da das ganze n 2-dimensionales feld ist .. und A[0] damit kein einfacher Typ sondern wiederum ein quasi komplexes Array .. wird dieses Array an Stelle A[0] nicht weiter kopiert, sondern der Wert von A[0] landet in B[0] und damit referenziert B[0] das selbe Array wie A[0]??
Wären A und B eindimensionale Felder vom Typ double, wären sie exakte Kopien und A würde sich durch B net verändern. :meld:

 

[SlaterB]

stimmt