ListにAddする再、何故か全てのデータがAddしたデータに上書きされてしまいます。

Question

C#でプログラムを作成しています。

// サンプルデータ格納クラス
private class data_sample
{
    public string data1 { get; set; }
    public string data2 { get; set; }
}

List<data_test> SampleList = new List<data_test>();

このように定義したリストに対して、

data_sample sample;
sample = new data_sample();

//データセット
sample.data1 = (ここになんらかのデータをセット);
sample.data2 = (ここになんらかのデータをセット);

//リストに追加
SampleList.Add(sample);

このようにデータをセットしているのですが、何故か追加するたびに、全てのデータが、最後に追加したデータに上書きされたようになってしまいます。

例えば、最初に

//データセット
sample.data1 = "aaa";
sample.data2 = "111";

//リストに追加
SampleList.Add(sample);

このようにデータをセットすると、SampleListには、data1に"aaa"、data2に"111"がセットされるのですが、その後、

//データセット
sample.data1 = "bbb";
sample.data2 = "222";

//リストに追加
SampleList.Add(sample);

とすると、SampleListの配列は2つになるのですが、data1は2つとも"bbb"、data2は２つとも"222"になってしまい、最初にAddした、"aaa"と"111"は無くなってしまいます。

その後、例えばもうひとつデータを追加して、

//データセット
sample.data1 = "ccc";
sample.data2 = "333";

//リストに追加
SampleList.Add(sample);

とすれば、配列数は3つになりますが、データは3つとも最後に追加したデータになってしまいます。

なぜこのようになってしまうのか、それがわかりません。

どうして、このような事になってしまうのでしょうか？

Answer 1

こんにちは。

C#のクラスは参照型ですので単純に代入した場合、単にそのアドレスが代入されるだけです。
List<T>のaddは代入と同じ動作をするようですので、単にsampleのアドレスがリストへ追加されるだけなのです。

従って、SampleList.add(sample);を3回実行した場合、sampleのアドレスを3つSampleListへ追加されます。つまり、全部同じインスタンスへの参照(ポインタ)が設定されます。
従って、SampleListの要素を全て表示した場合、最後にsampleに設定した値が常に表示されます。

MSDNのサンプルのように、newしたものをadd()しましょう。それにより異なるdata_sampleのインスタンスが追加されます。

---

【余談ですが】
C#は、Javaと同じくポインタを意識しないで済む文法になっていますが、実装はポインタを使ってますので、ポインタを理解していないと今回のようなトラブルを解決することが厳しいです。

Comments

[GiveAHand]

Chironian様

おぉ！これはわかりやすい説明！！！

なるほど、そういう事なんですね！！！

ありがとうございました！
よく理解できました！

Answer 2

こんにちは。

もう少しコードの前後を精査する必要はありますが、
最も単純な可能性としてはAddしたインスタンスsampleが同一であることでしょうか。
毎回newしたインスタンスをAddしても発生しますか？

Comments

[GiveAHand]

Tak1wa様

毎回newしたら、うまくいきました！
ありがとうございました！

Answer 3

上記プログラムですと、sampleがすべて同じインスタンスを参照してしまっているかもしれません。

都度インスタンスを生成してListにセットするとどうでしょうか？

data_sample sample1 = new data_sample();
sample1.data1 = "aaa";
sample1.data2 = "111";

//リストに追加
SampleList.Add(sample1);


data_sample sample2 = new data_sample();
sample2.data1 = "bbb";
sample2.data2 = "222";

//リストに追加
SampleList.Add(sample2);

Comments

[GiveAHand]

takafumin様

上記のやり方で、うまくいきました！
ありがとうございました！

[takyafumin]

動いたようでよかったです。

ただ、この事象は「どうしてそうなるのか？」をきちんと理解しておくことをお勧めします。
Chironianさまの回答をよく読むと理解の助けになるかと思います。