ArrayListの参照の代わりに値を格納する

Question

私は過去4日間Javaコードに問題があります。私の人生では、その根本原因を見つけることができません。

基本的には、オブジェクトのリストを属性として持つオブジェクトのリストのArrayListを作成します。しかし、リストの最初の要素にメソッドを適用しようとすると、他のオブジェクトも同様に更新されます！

public class Matrix { 
    public double[][] A; 

    public Matrix(double[][] A) { 
     this.A = A; 
    } 
} 
public class Model { 
    public Matrix M; 

    public Model(Matrix M) { 
     this.M = M; 
    } 

    public static Model(Model m) { 
     // modifying model a bit 
     return newModel; 
    } 
} 

public class MyProgram { 
    public ArrayList<Model> models; 

    public main() { 
     this.models = new ArrayList<Model>(NUM); 
     Matrix A = randomMatrix(); 
     for (int i = 0; i < NUM; i++) { 
      this.models.add(new Model(A)); 
     } 
    } 

    public otherMethod() { 
     this.models.set(0, changeModel(this.models.get(0))); 
    } 
} 

問題は、私はotherMethodを起動するたびに、リスト内のすべてのモデルが同様に変更を取得している：

は、ここに私のオブジェクト構造の基本的な概要です！

Answer 1

あなたは2-D arrayそれを割り当てないcloneする必要があります。

ので、代わりのthis.A = A;

あなたはコピーのために、このようなものを使用する必要があります。

public static int[][] deepCopyIntMatrix(int[][] input) { 
    if (input == null) 
     return null; 

    int[][] result = new int[input.length][]; 

    for (int r = 0; r < input.length; r++) { 
     result[r] = input[r].clone(); 
    } 
    return result; 
} 

Answer 2

どうもありがとう、問題がされたが実際には深いコピー：

public class Matrix { publ ic double [] [] A;

public Matrix(double[][] A) { 
    A = new double[n][m]; 
    for(int i=...) 
     for(int j=...) 
       this.A[i][j] = A[i][j]; 
} 

そして、私の行列を追加するには、私はこれをしなかった：

this.models.add(new Model(new Matrix(A.A))));