[gelöst]Zwei Ausgaben, die eigentlich gleich sein sollten

[Agent]

Hallo zusammen,

Das folgende Programm gibt zwei Arrays aus (a und b). Ich frage mich, wieso die beiden Ausgaben verschieden sind. (Programmbeschreibung siehe unten)

Die Ausgabe ist:

###
4 3
2 1
###
###
1 3
2 4
###

import java.util.*;
import java.lang.*;
import java.io.*;

public class Test {
  public static void main(String[] args) {
    int[][] a=new int[2][2];
    a[0][0]=1;  //initialisiere Array a
    a[0][1]=2;
    a[1][0]=3;
    a[1][1]=4;

    int[][] b=new int[2][2];
    b[0][0]=8;  //initialisiere Array b
    b[0][1]=8;
    b[1][0]=8;
    b[1][1]=8;

    b=swap(a,0,0,1,1);
    print(b);
    print(swap(a,0,0,1,1));
  }

  public static void print(int[][] bd){
    System.out.println("###");
    System.out.println(bd[0][0]+" "+bd[1][0]);
    System.out.println(bd[0][1]+" "+bd[1][1]);
    System.out.println("###");
  }

  public static int[][] swap(int[][] bd, int xpos1, int ypos1, int xpos2, int ypos2){  //Vertauscht zwei Elemente
    int sto=bd[xpos1][ypos1];
    bd[xpos1][ypos1]=bd[xpos2][ypos2];
    bd[xpos2][ypos2]=sto;
    return bd;
  }
}

Programmbeschreibung:
Zunächst werden die zwei Arrays a und b initialisiert.
Die swap() Methode vertauscht einfach zwei Elemente aus dem ihr übergebenen Array. Ihm wird quasi ein Array und die beiden x- & y-Koordinaten der zu tauschenden Arrays übergeben.
Die Methode print() gibt einfach das Array in der Konsole aus.

Frage:
Meiner meinung nach müssten beide Ausgaben gleich sein. Der Unterschied liegt doch nur darin, dass bei der ersten Ausgabe, die Rückgabe von swap in der Variable (im Array) b zwischengespeichert wird. :bahnhof:

 

[Michael...]

Du änderst in der Methode swap(...) die die Inhalte von a[][]

 

[MarcB]

Arrays und Objekte werden per call-by-reference übergeben.

Folgender Link erklärt die Sache ganz gut:
Referenzparameter ? Wikipedia

 

[Agent]

Du änderst in der Methode swap(...) die die Inhalte von a[][]

In welcher der beiden Ausgabe-Varianten meinst du? Mein Problem ist, dass ich den Grund nicht verstehe, weshalb beide Ausgaben unterschiedlich sind. Das Array a[][] wird meines Erachtens nicht verändert.
Wie beschrieben vertauscht die Methode swap() zwei Elemente in dem übergebenen Array.

Der Punkt hier sine die zwei verschiedenen Varianten der Ausgabe, die meiner Meinung nach das gleiche tun sollten.

Variante 1 (Mit Zwischenspeichern):

b=swap(a,0,0,1,1);
print(b);

Variante 2 (Ohne Zwischenspeichern):

print(swap(a,0,0,1,1));

 

[Michael...]

swap(a,0,0,1,1)

hier übergibst Du die Referenz (siehe Post von MarcB) auf a[][] (Arrays sind Objekte)
Dieses Array heißt in der Methode swap zwar bd[][] ist aber das selbe Objekt wie a[][]
Zu Deinem Zweck müsstest Du eine Kopie von a[][] erstellen und diese dann manipulieren.

 

[Agent]

hier übergibst Du die Referenz (siehe Post von MarcB) auf a[][] (Arrays sind Objekte)
Dieses Array heißt in der Methode swap zwar bd[][] ist aber das selbe Objekt wie a[][]
Zu Deinem Zweck müsstest Du eine Kopie von a[][] erstellen und diese dann manipulieren.

Wenn ich swap() folgendermaßen umschreibe funktionierts aber leider dennoch nicht. Was mache ich denn nur falsch?

  public static int[][] swap(int[][] blub, int xpos1, int ypos1, int xpos2, int ypos2){  //Vertauscht zwei Elemente
    int[][] bd = new int[2][2];
    bd=blub;
    int sto=bd[xpos1][ypos1];
    bd[xpos1][ypos1]=bd[xpos2][ypos2];
    bd[xpos2][ypos2]=sto;
    return bd;
  }

 

[Beni]

Auch hier machst du keine Kopie:

    int[][] bd = new int[2][2];
    bd=blub;

Eine Zuweisung "bd=blupp" bedeutet eben, dass bd und blupp danach auf dasselbe Objekt zeigen. Du muesstest (in einer Schleife) jeden Wert einzeln von einem Array in den anderen schieben.

for( int x = ... ){
  for( int y = ... ){
    bd[x][y] = blup[x][y];
  }
}

 

[Michael...]

Wenn ich swap() folgendermaßen umschreibe funktionierts aber leider dennoch nicht. Was mache ich denn nur falsch?

bd=blub;

Hier weist Du der Variable bd wieder die Referenz auf a[][] zu, dh. a=blub=bd.
Du musst die Inhalte aus blub in bd kopieren. Entweder manuell oder mit

System.arraycopy(blub, 0, bd, 0, blub.length);

 

[Agent]

Danke!
Mit

for (int j=0; j<2; j++){
  for (int i=0; i<2; i++){
    bd[i][j]=blub[i][j];
  }
}

funktioniert es jetzt!

Der Befehl

System.arraycopy(blub, 0, bd, 0, blub.length);

scheint aber nur für eindimensionale Arrays zu funktionieren Gibts da auch was für mein zweidimensionales Array?

 

[faetzminator]

wenn das nicht funktioniert, dann sollte es mit folgendem:

System.arraycopy(blub, 0, bd, 0, blub.length * blub[0].length);