csvファイルを外部から読み取るメソッドの作成

Question

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
外からメソッドを呼び出したいのですが、上手くいきません

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

java.lang.NullPointerException

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

	public static void main(String[] args) {

		XXXBean bean = new XXX7Bean();
		ArrayList<String[]> A = bean.setId();
		bean.getId(A);

	}

=====================================
public class XXX7Bean {

	private ArrayList<String[]> id;

	public ArrayList<String[]> setId(){
		String filepath = "読み込むCSVファイル";
	    File file = new File(filepath);

	    try {
	      FileInputStream fis = new FileInputStream(file);
	      InputStreamReader isr = new InputStreamReader(fis);
	      BufferedReader br = new BufferedReader(isr);

	      String line;
	      ArrayList<String[]> id = new ArrayList<String[]>();
	      while ( ( line = br.readLine()) != null ) {
	        String[] cols = line.split(",");
	        id.add(cols);
	      }
	      br.close();

	    } catch(Exception e) {
	      e.printStackTrace();

	}
		return this.id;
	}public void getId(ArrayList<String[]> list) {
		for (String[] row : list) {
		System.out.println(row[0] + "," + row[1] + "," + row[2] + "," + row[3]);
		}
		}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

XXX7BeanクラスのsetIdメソッドでthis.idをreturnしていますが、何も設定していないのでnullが返り、
getIdのfor文でNullPointerExceptionが発生しています。
setIdのローカル変数のidをthis.idに設定するか、ローカル変数のidをreturnすると動くようになると思います。
getIdで引数を受け取っているので、インスタンス変数のidはなくてもいいかもしれません。

Comments

[FNKswOG77EyJ205]

ご回答ありがとうございます。ローカル変数のidをthis.idに設定するにはどう記述すればいいでしょうか。

[ijufumi]

while文が終わった後に、
this.id = id;
を記述したら大丈夫です。

[FNKswOG77EyJ205]

ありがとうございます！
return this.id = id;
では上手くいかなったので、それぞれ別にしてあげないといけないんですね。

[ijufumi]

ローカル変数のidは、tryの中で宣言されているのでreturnのところでは無理ですね。
idをtryの前で宣言するといける気がします。
ただ、必要ではない限り、1文1命令とした方がコードの可読性は良くなるので、
this.id = id;
return this.id;
はそれぞれ別にした方がいいです。