JAVAでLISTの中身が書き換わってしまう

Question

お世話になっております
皆様のお知恵を拝借できたら幸いでございます

JAVA(SpringBoot)にてファイルアップロードのシステムを作成しており動作途中で
Listの中身が書き換わってしまう事象です。

前提・実現したいこと

WEB側より複数のファイルアップロードを行い、既に登録がされているファイルを
除いた新規登録分だけを受け付ける仕組みを想定しています

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

エラーは出ないのですが後述コード内のList変数の中身が後から登録されたデータに書き換わってしまいます

tnpFileNewdata.add(addData);

プログラム実行後のデータ
[fileName=tnpfile_03.pdf, size=700, serial=003]
[fileName=tnpfile_03.pdf, size=700, serial=003]

同じレコードが二つ出来ます

該当のソースコード

Web側から受けるデータは変数tnpListに格納
中身は3ファイル分

ファイル一つ目

kye	value
alradyFlg	1
fileName	tnpfile_01.pdf
fileSize	800
fileSerialNo	001

ファイル二つ目

kye	value
alradyFlg	0
fileName	tnpfile_02.pdf
fileSize	650
fileSerialNo	002

ファイル三つ目

kye	value
alradyFlg	0
fileName	tnpfile_03.pdf
fileSize	700
fileSerialNo	003

二つ目と三つ目が登録される想定です

pabulic class tnpFileBean{
	/**登録済みフラグ 0=新規登録 1=登録済み.*/
	private int alradyFlg;
	/**ファイル名.*/
	private String fileName;
	/**ファイルサイズ.*/
	private String fileSize;
	/**ファイル管理番号.*/
	private String fileSerialNo;

	public int getAlradyFlg(){
		return alradyFlg
	}
	public String getFileName(){
		return fileName
	}
	public String getFileSize(){
		return fileSize
	}
	public String getFileSerialNo(){
		return fileSerialNo
	}
//setterは略
}

	List<Map<String, Object>> tnpFileNewdata = new ArrayList<Map<String, Object>>();
        Map<String, Object> addData = new HashMap<String, Object>();
        for (tnpFileBean registTnpFile : tnpList) {
            if (registTnpFile.getAlradyFlg() == 0) {
                addData.clear();
                addData.put("fileName", registTnpFile.getBtrFilePath());
                addData.put("size", registTnpFile.getFileSize());
                addData.put("serial", registTnpFile.getFileSerialNo());
                tnpFileNewdata.add(addData);
            }
        }

補足説明

デバッグモードでステップ実行したところ
ループの初回に到達したaddの時点では
tnpFileNewdataの中身は
[fileName=tnpfile_02.pdf, size=650, serial=002]

2回目のループではaddの行に到達する前に
[fileName=tnpfile_03.pdf, size=700, serial=003]
上記に下記変わってました

参照値でもListに渡した後のList内でも変わってしまうものなのでしょうか

期待している値は
[fileName=tnpfile_02.pdf, size=650, serial=002]
[fileName=tnpfile_03.pdf, size=700, serial=003]
になります。

Answer 1

回答：
同一のHashMapインスタンスに対して[clear]を行って格納情報を削除しているため。
全て別の要素にするならば、For文内で利用する都度インスタンス化しなければならない。
後蛇足ですが、JAVAの場合クラス名の1文字目は基本的に大文字にした方がいいです。

解説：
前提として、tnpList には tnpFileBean 型のインスタンスが3つ格納されているとします。

for文の外でHashMapをインスタンス化してます。

Map<String, Object> addData = new HashMap<String, Object>();

すると、HashMapのインスタンスが1つ生成されます。
コードの説明を入れますと、

  Map<String, Object> addData = new HashMap<String, Object>();
        for (tnpFileBean registTnpFile : tnpList) {
            if (registTnpFile.getAlradyFlg() == 0) {
                // addDataには2週目以降、1つ前にputメソッドで追加した内容が含まれますが、
　　　　　　　　 // 2週目の場合、tnpFileNamedataの0番目に格納したaddDataのインスタンスは
　　　　　　　　　// 同じインスタンスです。
                // そのインスタンスの格納情報をclearメソッドにてクリアします。
　　　　　　　　　// tnpFileNamedataの0番目に格納したaddDataのインスタンスは同じなので格納情報が消えます。
                addData.clear();
                addData.put("fileName", registTnpFile.getBtrFilePath());
                addData.put("size", registTnpFile.getFileSize());
                addData.put("serial", registTnpFile.getFileSerialNo());
                tnpFileNewdata.add(addData);
            }
        }

全部べつにしたければ、全て別のHashMapインスタンスを代入し直さなくてはならないので以下のような状態にしなくてはなりません。

  Map<String, Object> addData = null;
        for (tnpFileBean registTnpFile : tnpList) {
            if (registTnpFile.getAlradyFlg() == 0) {
                addData =  new HashMap<String, Object>();
                addData.put("fileName", registTnpFile.getBtrFilePath());
                addData.put("size", registTnpFile.getFileSize());
                addData.put("serial", registTnpFile.getFileSerialNo());
                tnpFileNewdata.add(addData);
            }
        }

Comments

[koppe]

ご回答ありがとうございます。
無事動作いたしました。

デバッグ中不思議だったのが変数addDataがeclipseの「変数」ビューに表示されなかったのです

ブレークポイントを置いてaddDataの記載箇所にマウスカーソルを近づけても「Map<String, Object> addData ～（以下略）」と表示され、実際に入っているはずの値が確認できませんでした。

今回　Map<String, Object> addData = null;　を通過したところで変数ビューにも表示されるようになり、正常に動作いたしました

本当に助かりました。
ありがとうございました。

Answer 2

変数の内容が上書きされてしまっています。

    List<Map<String, Object>> tnpFileNewdata = new ArrayList<Map<String, Object>>();
	for (tnpFileBean registTnpFile : tnpList) {
		if (registTnpFile.getAlradyFlg() == 0) {
			//addData.clear();
			Map<String, Object> addData = new HashMap<String, Object>();
			//↑同じ器を使いまわすのではなく、新たに用意しないと上書きされる
			addData.put("fileName", registTnpFile.getBtrFilePath());
			addData.put("size", registTnpFile.getFileSize());
			addData.put("serial", registTnpFile.getFileSerialNo());
			tnpFileNewdata.add(addData);
		}
	}

Comments

[koppe]

ご回答ありがとうございます。
ご教示頂いた内容を試したところ
処理結果は変わらずでした。

[mar-kn]

Service の tnpList の中身には、ファイル1つ目、ファイル2つ目、ファイル3つ目 の中身が入ってますか？

[koppe]

度々のご回答ありがとうございます。
tnpListの中身はファイル3つ分それぞれの中身がはいっております

[mar-kn]

tnpList の中身はデバッガなどで確認されましたか？
また解決済みになったということは、kanimaruさんの提示されたコードでは正しく動いたということでしょうか？

[koppe]

ご回答ありがとうございます
kanimaru様のコードで正しく動作いたしました。
tnpList の中身はデバッガで確認しましたそれぞれfor文のループごとに変化がないかどうかも監視してましたが、変わることはありませんでした。
ただ懸念点がaddDataの中身が確認ができなかったことです。変数ビューに表示されず、ブレークポイントを置いてマウスカーソルを近づけても変数の中身が確認できませんでした

少々不明な点が残りましたが、無事動きました。ご協力ありがとうございました