javaプログラミング

質問

Javaを勉強中の者です。
以下のソースを実行すると
-------------------------------------------
item[0] = {10, true, true}
item[1] = {11, true, true}
-------------------------------------------
となります。
しかし、
-------------------------------------------
item[0] = {10, true, false}
item[1] = {11, false, true}
-------------------------------------------
が期待する結果です。

public class test{

public static void main(String[] args) {
boolean[] b = {false,false};

Item[] item = new Item[2];
item[0] = new Item(10 ,b);
item[1] = new Item(11 ,b);

item[0].setCandidate(0, true);
item[1].setCandidate(1, true);

b = item[0].getCandidate();
System.out.println("item[0] = {" + item[0].getNumber() + ", " + b[0] + ", " + b[1] + "}");
b = item[1].getCandidate();
System.out.println("item[1] = {" + item[1].getNumber() + ", " + b[0] + ", " + b[1] + "}");
}
}

public class Item{
private int number;
private boolean[] candidate = new boolean[2];

public Item(int number, boolean[] candidate){
this.number = number;
this.candidate = candidate;
}

public void setNumber(int num){
number = num;
}

public int getNumber(){
return number;
}

public boolean[] getCandidate(){
return candidate;
}

public void setCandidate(int num , boolean candi){
this.candidate[num] = candi;
}

}
おそらくcandidate[]が上書きされてる？
説明がうまくできません…

期待通りの結果にするにはどうすればよいのでしょうか？

回答No.2

nda23 · ベストアンサー

item[0] = new Item(10 ,b);
item[1] = new Item(11 ,b);
これで、item[0]とitem[1]のcandidateには独立した値があると
思われたのでしょうが、両方とも同じオブジェクトbを使っています。
b、item[0]のcandidate、item[1]のcandidate は全て同じデータです。
従って、どれを表示しても同じ値です。
因みに、bを変更すると、item[0]やitem[1]を修正しなくても、中の
candidate は変化しています。尚、プリミティブ型で配列でない
numberは各インスタンスで独立しています。

コンストラクタを変更して、以下のようにします。
public Item(int number, boolean[] candidate){
　　this.number = number;
　　this.candidate[0] = candidate[0];
　　this.candidate[1] = candidate[1];
}
配列の各要素はプリミティブ型なので、値はコピーされます。
コンストラクタに与えた変数bとクラス内のcandidateは別の
データになります。

お礼

配列もintやstringと同じように使えると考えてました。
勉強になりました。
ありがとうございます。

回答No.1

ProKaseifu

Item[] item = new Item[2];
item[0] = new Item(10 ,new boolean[]{false,false});
item[1] = new Item(11 ,new boolean[]{false,false});

Itemのインスタンス生成部分を↑のように変更してみてください。
元のソースがなぜうまくいかないのかというと、配列は参照型だからです。

参考URL：

http://www.javadrive.jp/start/array/index3.html

お礼

うまくいきました！
ありがとうございました。