ジェネリッククラスに要素を追加すると、最後のファイル名のみ反映されてしまう

Question

###発生している問題・エラーメッセージ

ディレクトリ下にあるファイル名（rtf）を列挙 (button1)
┗「Data」フォルダ以下に Form1.rtf ~ Form4.rtf までを用意

上記で取得したファイル名をXMLとして保存 (button2)

という処理の流れで、最後に取得したファイル名のみ反映され、出力結果が <FileName>Form4.rtf</FileName> のみとなってしまいます。

上書きされてしまうようですが、正常に反映させる方法が分かりません。

参考にした記事
C#で付箋紙を作る（第三回目） XMLファイルからパラメーターを読み込んで反映する

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

###該当のソースコード

 
        private void Form1_Load(object sender, EventArgs e)
        {
            setCurrent = Directory.GetCurrentDirectory();
           
        }



        /// <summary>
        /// //カレントディレクトリ下にある”Data”フォルダ内のファイル名を列挙
        /// </summary>
        string setCurrent;
        static StickyList SL;
        static StickyData SD;
        private void button1_Click(object sender, EventArgs e)
        {
            SD = new StickyData();
            SL = new StickyList();
            SL.DataList = new List<StickyData>();
       ///インスタンス化

            string[] files = Directory.GetFiles(setCurrent +"\\Data\\", "*.rtf", SearchOption.TopDirectoryOnly);
         //”Data”フォルダ下のフルパスを取得
        
                foreach (string filename in files)
                {
                    string s = Path.GetFileName(filename);
             //フルパスをファイル名に変換
               

           SD.FileName = s;

                    SL.DataList.Add(SD);  //この時点で、SD内にはForm4.rtfしか存在しない
                      //上書きされる

                    listBox1.Items.Add(SD.FileName);  //結果のプレビュー



                }

        }

      private void button2_Click(object sender, EventArgs e)
        {


            var serializer = new XmlSerializer(typeof(StickyList));
            //XMLシリアライズクラスを初期化（インスタンス化）


            var sw = new StreamWriter(@"Sticky.xml", false, new UTF8Encoding(false));
            serializer.Serialize(sw, SL);
            //シリアライズして書き込む

            sw.Close();
        }
    
     #region ジェネリッククラス

        /// <summary>
        /// XMLとして保存するクラス
        /// </summary>
        [XmlRoot("Sticky")]
        public class StickyData
        {
            [XmlElement("FileName")]
            public string FileName { get; set; }           
        }

        /// <summary>
        /// StickyDataをジェネリックリスト化するクラス
        /// </summary>
        [XmlRoot("Stickys")]
        public class StickyList
        {
            [XmlElement("Sticky")]
            public List<StickyData> DataList { get; set; }
        }

        #endregion

###出力結果

<?xml version="1.0" encoding="utf-8"?> <Stickys xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema"> <Sticky> <FileName>Form4.rtf</FileName> </Sticky> <Sticky> <FileName>Form4.rtf</FileName> </Sticky> <Sticky> <FileName>Form4.rtf</FileName> </Sticky> <Sticky> <FileName>Form4.rtf</FileName> </Sticky> </Stickys>

###補足情報(言語/FW/ツール等のバージョンなど)
listbox1内の出力は正常でした。

変数
SLがbutton1 クリック時にFileName”Form4”の重複が見られました。

Answer 1

こんにちは。

たぶん、「ListにAddする再、何故か全てのデータがAddしたデータに上書きされてしまいます。」と同じ原因です。

ListへのaddはSDのデータではなく、SDへのポインタがaddされるだけです。
SD = new StickyData();をforeachループの中に入れれば解決する筈です。

List<T>.Add メソッド (T)のサンプルが参考になるかも。

Comments

[sheephuman]

あ、なるほど・・・　foreachループ内でその都度newすればよかったんですね！
関連の質問みつけたのはいいものの、外側でnewしても単に初期化されちゃうだけなんで、どうしたものかと思ってたんですが・・・・
　目から鱗でした(;´･ᴥ･)

Answer 2

XmlRoot属性が複数になっているからじゃないですか？

Comments

[sheephuman]

よくわかりません。　[XmlRoot("Stickys")]　を削除するなどでしょうか？

[hihijiji]

そちらはルートっぽいので、削除するなら[XmlRoot("Sticky")]のほうです。

[sheephuman]

すいません。削除しても出力結果が変わりません。

　試しに要素を全部削除して実行したら、
　 <DataList>
<StickyData>
<FileName>Form4.rtf</FileName>
</StickyData>　×4ループ　　となりました。

　この場合、ルート要素の変更は処理に影響を与えないのでは。

[hihijiji]

そこじゃなかったですm(__)m
static StickyData SD;
これは使わずに都度 new StickyData() として新しいインスタンスを作ってListに追加してください。

[sheephuman]

うーん・・・ほかにジェネリック配列を定義して移したりもしたんですが、リストの追加でつまずいてしまいます。
　もう少し勉強してみます。