summe von 2 Matrizen

[bradig]

kann einer mir bitte sagen,was ich in der folgenden Implementierung falsch gemacht habe.
Es funktioniert .
Aber ein Bekannter von mir meinte,die Implementierung tut nicht zu 100% das,was die tun sollte.

 public static double[][] add(double[][] a, double[][] b) {

        double[][] c = new double[a.length][b.length];
        for (int i = 0; i < a.length; i++) {// Zeile  durchlaufen
            for (int j = 0; j < a[0].length; j++){// spalten  durchlaufen
                c[i][j] = a[i][j] + b[i][j];
            }
        }
        return c;
    }

Bitte um Hilfe.
Herzlich
Bradig

 

[bradig]

kann einer mir bitte sagen,was ich in der folgenden Implementierung falsch gemacht habe.
Es funktioniert .
Aber ein Bekannter von mir meinte,die Implementierung tut nicht zu 100% das,was die tun sollte.

 public static double[][] add(double[][] a, double[][] b) {

        double[][] c = new double[a.length][b.length];
        for (int i = 0; i < a.length; i++) {// Zeile durchlaufen
            for (int j = 0; j < a[0].length; j++){// spalten durchlaufen
                c[i][j] = a[i][j] + b[i][j];
            }
        }
        return c;
    }

Bitte um Hilfe.
Herzlich
Bradig

 

[JStein52]

Hast du es auch mal mit nicht quadratischen Matizen (z.B. 5x4 oder 4x5 Matrizen) getestet ?

 

[Harry Kane]

Die Matrix c muss nicht die Dimensionen [a.length][b.length] haben, sondern [a.length][a[0].length].
Ausserdem müssen die Ausgangsmatritzen dieselbe Größe haben, d. h. a.length = b.length muss gelten,
alle Arrays a[j] müssen gleich lang sein, und für alle Arrays a[j] und b[j] muss gelten a[j].length = b[j].length.

 

[bradig]

kann einer mir bitte sagen,was ich in der folgenden Implementierung falsch gemacht habe.
Es funktioniert .
Aber ein Bekannter von mir meinte,die Implementierung tut nicht zu 100% das,was die tun sollte.

 public static double[][] add(double[][] a, double[][] b) {
if (a.length!=b.length||a [0].length!=b [0].length||a==null||b==null){
throw new IllegalArgumentException ();
}
        double[][] c = new double[a.length][a [0].length];
        for (int i = 0; i < a.length; i++) {// Zeile  durchlaufen
            for (int j = 0; j < a[0].length; j++){// spalten  durchlaufen
                c[i][j] = a[i][j] + b[i][j];
            }
        }
        return c;
    }

Ist jetzt so in Ordnung?
Bitte um Hilfe.
Herzlich
Bradig

 

[Salo]

Die Matrix c muss nicht die Dimensionen [a.length][b.length] haben, sondern [a.length][a[0].length].
Ausserdem müssen die Ausgangsmatritzen dieselbe Größe haben, d. h. a.length = b.length muss gelten,
alle Arrays a[j] müssen gleich lang sein, und für alle Arrays a[j] und b[j] muss gelten a[j].length = b[j].length.

So?

public static double[][] add(double[] a, double[] b) {

    if(a.length != b.length) return null;

    double[][] c = new double[a.length][a[0].length];

    for(int i = 0; i < a.length) {
        for(int j = 0; j < a[0].length {
            c[i][j] = a[i][j] + b[i][j];
        }
    }
    return c;
}

Falls es so nicht funktioniert, dann schilder dein Problem etwas genauer.
Dann können wir dir auch besser helfen.

Gruß, Salo

 

[bradig]

ich glaube ,der einzige Fehler,der eigentlich keine Auswirkung auf die berechnung hatte, lag bei der Deklaration von c.
double[][] c = new double[a.length][a[0].length] statt double[][] c = new double[a.length][b.length];
Mein Code hat immer hin,2 Matrizen ohne Probleme addieren können,obwohl ich c nicht "korrekt" deklariert habe.
Danke an allen,die mir geholfen haben.
Mein junit-test:

     @Test
    public void testMatrixAdd() {
        double[][] m = {{1,2,3},{4,5,6},{7,8,9}};
        double[][] n = {{1,2,3},{4,5,6},{7,8,9}};

        double[][] exp = {{2,4,6},{8,10,12},{14,16,18}};
        assertArrayEquals( exp, Matrix.add(m, n));
    }

Herzlich

 

[JStein52]

der einzige Fehler,der eigentlich keine Auswirkung auf die berechnung hatte

Doch, hätte sehr wohl Auswirkungen gehabt. Du testest eben nur mit quadratischen Matrizen, da geht es gut. Aber du musst verschieden Fälle testen !

 

[bradig]

Soweit ich es verstanden habe,müssen die beiden matrizen gleich lang sein und die gleiche Anzahl von spalten haben.
Ausserdem muss jede matriz die gleiche Anzahl von zeilen und von spalten haben.
A=N×N
B=N×N
RICHTIG?

 

[JStein52]

nein.
A=nxm
B=nxm

und n und m können verschieden sein.

 

[bradig]

Wie doof von mir..ich habe echt gedacht ,die addition nur mit quadratischen Matrizen geht.
Vielen Dank jStein52.
Das heisst :
A.length==B.length && A[0].length==B[0].length gelten muss.
Oder?

 

[Cromewell]

A[0].length==B[0].length

und für alle folgenden Zeilen. Am besten wäre es wahrscheinlich, wenn du A[0].length mit allen Zeilen abgleichst (auch innerhalb der A-Matrix).

 

[JStein52]

ich habe echt gedacht ,die addition nur mit quadratischen Matrizen geht.

Das dachte ich mir und deshalb hatte ich so drauf rumgeritten dass du es auch mal mit nicht quadratischen Matrizen testest. Denn dann hättest du bei deinem Code eine IndexOutOfBounds Exception gekriegt.
Und da du in Java 2-dimensionale Arrays konstruieren kannst deren Zeile verschieden lang sind hat @Cromewell recht dass du bei jeder Zeile testen musst ob beide gleich lang sind. Ob das dann noch als als Matrix im mathematischen Sinn bezeichnet wird weiss ich aber nicht.

 

[bradig]

ja.ich habe es mit nicht quadratischen Matrizen getestet.
Ich bekomme ein IndexOutOfBounds Exception.

ich muss versuchen,so zu implementieren,dass es auch mit nicht quadratischen Matrizen funktioniert.

 

[JStein52]

Ja. Du musst das ergebnis-Array richtig dimensionieren ! Der Rest deines Codes ist erst mal richtig.
Du kannst dann im nächsten Schritt noch diverse Prüfungen einbauen.

 

[bradig]

Meine Lösung :

 public static double[][] add(double[][] a, double[][] b) {
if(a.length!=b.length || a[0].length!=b[0].length||a==null||b==null){
throw new IllegalArgumentException();
}
     double[][] c = new double[a.length[a[0].length];
        for (int i = 0; i < a.length; i++) {// Zeile durchlaufen
            for (int j = 0; j < a[0].length; j++){// spalten durchlaufen
                c[i][j] = a[i][j] + b[i][j];
            }
        }
        return c;
    }

Herzlich

 

[bradig]

die Oben stehende Lösung funktioniert mit quadratischen und nicht quadratischen Matrizen.
Also.A=m*n,B=m*n
m und n können entweder gleich sein(Quadratische Matrizen) oder verschieden sein(nicht quadratische Matrizen).
danke noch an allen.

 

[Meniskusschaden]

Das:

double[][] c = new double[a.length[a[0].length];

lässt sich kompilieren? Kann ich mir nicht vorstellen.

 

[bradig]

sorry kleiner Fehler.ich habe hier nur meinen alten Eintrag bearbeitet.
Es funktioniert sonst .

 

[bradig]

Ein Fall ist mir noch gerade beim Testen aufgefallen.
Wenn in dem Array a ,ein null-Array (null) enthalten ist (Beispiel: a[0]={null,{1,2},{3,4}})
ich habe versucht auch ein IllegalArgumentException dafür ausgeben zu lassen,aber ich bekomme immer ein NullPointerException.

for(int i=0;i<a.length;i++){

if(a[i]==null ||b[i]==null|| a.length!=b.length || a[0].length!=b[0].length||a==null||b==null){
throw new IllegalArgumentException();
}

wie kann man dieses Problem lösen?
Danke im voraus.
Herzlich
Bradig

 

[Meniskusschaden]

Wenn in dem Array a ,ein null-Array (null) enthalten ist (Beispiel: a[0]={null,{1,2},{3,4}})

Das ist nicht kompilierbar.