二次元配列に配列を格納する方法について教えてください。

Question

実現したいこと

現在、エクセルファイルを読み込んでセルを一つ一つ取り出して
一次元配列に格納し、その配列を二次元配列のListに格納したいと考えています。

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

ループの中でエクセルの１レコード（行）を配列に格納して１レコード
読み終わると、二次元配列に格納させようとしているのですが、実行結果は
すべて、最後に読み込んだレコードになってしまいます。

例）下記のようなエクセルファイルがあったとき
１１１　ああああ　大阪
２２２　いいいい　東京
３３３　うううう　神戸

下記を１レコード終わるたびに登録させていく。
xlsxData[0] 　←　１１１　ああああ　大阪
に格納してcsvData.add(xlsxData)で登録していきたい。

実行結果）
３３３　うううう　神戸
３３３　うううう　神戸
３３３　うううう　神戸
になってしまいます。

１レコード二次元配列に格納して、次の要素([0][2])に格納していきたいのですが、
格納できず困っています。

どなたかお力添え頂けないでしょうか。

該当のソースコード

xlsxData[] : セルの情報を格納していく配列。
csvData ：xlsxData配列を格納していく二次元配列。

// 格納する二次元配列に
**List<String[]> csvData = new ArrayList<>();**

						// List<String[]> xlsxData = new ArrayList<>();
						// エクセルファイルのとき
						/** ファイルのあるフォルダ */
						final String FILE_DIR = "C:/pleiades/workspace/DataConvertSystem/src/main/WebContent/xlsx/";
						ExelFileRead EFR = new ExelFileRead();
						List<String> filePathList = ExelFileRead.getFilePathList(FILE_DIR);
						for (String filePath : filePathList) {
							//Excelを開く
							File file = new File(filePath);
							Workbook workbook;
							// シートの全列をループで表示する処理
							try {
								workbook = WorkbookFactory.create(new FileInputStream(file));
								//シート１枚目を開く
								Sheet sheet = workbook.getSheetAt(0);								
								// ヘッダ行だけの二次元配列を作成
								// ヘッダ行のみ取得
								Row rowhead = sheet.getRow(0);
								System.out.println("★：" + rowhead.getLastCellNum());
								int element = rowhead.getLastCellNum();
								// セル情報格納用の配列を作成
								String[] xlsxData = new String[element];
								// すべての行のセルを確認しに行っている。
								int loopcnt=0;
								int rowcnt = 0;
**								for (Row row : sheet) {
									System.out.println("row:" + row);
									for (Cell cell : row) {
										// セルの文字列か数値の確認を行って値を取得
									
										//セルの型を判定する
										CellType cType = cell.getCellType();
									      switch(cType) {
									        case STRING:
									        	// 	文字型
									          System.out.println("文字型です。");
									          System.out.println(cell.getStringCellValue() + "\t");
									          xlsxData[loopcnt] = cell.getStringCellValue();
									          
									          break;
									        case NUMERIC:
									        	//	数値型、日付型 ※日付もNUMERICと判定されます
									        	// 小数点の判定が必要
									          System.out.println("数値型です。");
									          System.out.println((int)cell.getNumericCellValue() + "\t");
									          String Intcell = String.valueOf((int)cell.getNumericCellValue());
									          System.out.println("文字列変換:" + Intcell);
									          xlsxData[loopcnt] = Intcell;
									          //xlsxData.add(cell.getNumericCellValue());
									          break;
									        case FORMULA:
									        	// 	Excel関数型 ※例）NOW()、SUM()などのExcelの関数
									          System.out.println("関数です。");
									          break;
									        case BOOLEAN:
									        	// 真偽型 ※例）TRUE、FALSE
									          System.out.println("真偽型です。");
									          break;
									        case BLANK:
									        	//空 ※セルに値がセットされていない場合の型です
									          System.out.println("空です。");
									          break;
									          
									          // セルタイプを確認してもじれつが
									      }
									      
									      loopcnt ++;
									      
									}									

　　　　　　　　　　　　　　　　　　csvData.add(xlsxData);
									rowcnt ++;
									loopcnt=0;**
								}

試したこと

直接格納させる二次元配列（csvData ）にエクセルから取得したセルの値を格納するなど
試みたのですが、型の不一致のエラーが発生してうまく動作させることができませんでした。

Answer 1

String[] xlsxData = new String[element];
の書く場所を行のループが始まる場所にしてください。

**								for (Row row : sheet) {
ここ								String[] xlsxData = new String[element];`
									System.out.println("row:" + row);

行ごとにnewをしないと同じオブジェクトを使いまわすので、同じデータが複数個かつ
データ自体は最後のデータになってしまいます。

異なるオブジェクトを格納すればよいので、
csvData.add(xlsxData);
を
csvData.add(xlsxData.clone());
にしてもうまくいくはずです

Comments

[magmag123]

ありがとうございます！
無事にすべてのレコードを取得し、二次元配列に格納させることができました！
ループするごとに、格納させる配列はnewしないといけないんですね。

また、別回答の「clone()」もありがとうございます。
こちらでも実行できるか確認してみます。