複数ファイルの一括データ読み込みで配列がnullになる

Question

###実現したいこと
Androidアプリでローカルのとあるフォルダの全ファイルのデータを読み出し、Fragmentで指定したファイルのデータを取得してグラフを描く。(MPAndroidChart使用)

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

public class ReadData {
    private String path = パス;
    private File file;
    private File[] fileList;
    private File dataFile;

    private int numFiles;

    private float[][] array1;
    private float[][] array2;
    private float[][] array3;

    public void readDataFile(Context context) {
        String text;
        fileAccess();

        for(int i = 0; i < numFiles; i++){
            int j = 0;
            dataFile = fileList[i];
            //ファイル名取得
            String fileName = dataFile.getName();

            try{
                FileInputStream fileInputStream = context.openFileInput(fileName);
                BufferedReader reader = new BufferedReader(new InputStreamReader(fileInputStream,"UTF-8"));

                String lineBuffer;

                while ((lineBuffer = reader.readLine()) != null){
                    text = lineBuffer;
                    String[] data = text.split(",", -1);

                    array1[i][j] = Float.parseFloat(data[0]);
                    array2[i][j] = Float.parseFloat(data[1]);
                    array3[i][j] = Float.parseFloat(data[2]);

                    j++;
                }
                fileInputStream.close();
                reader.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }

    public void fileAccess(){
        file = new File(path);
        fileList = file.listFiles();
        numFiles = fileList.length;
    }

    public int getNumFiles() {
        return numFiles;
    }

    public File[] getFileList() {
        return fileList;
    }

    public float[][] getArray1() {
        return array1;
    }

    public float[][] getArray2() {
        return array2;
    }

    public float[][] getArray3() {
        return array3;
    }
}

public class GraphFragment extends Fragment {
    private DataViewModel dataViewModel;
    private ReadData readData;

//省略

    @Override
    public void onViewCreated(View root, Bundle bundle){
        dataViewModel = ViewModelProviders.of(getActivity()).get(dataViewModel.class);

        mChart = root.findViewById(R.id.line_chart);

        setUpLineChart();

        //別画面で選択したファイルがfile.listFiles()の何個目かわたす。
        final Observer<Integer> observer = new Observer<Integer>() {
            @Override
            public void onChanged(Integer integer) {
                setData(integer);
            }
        };
        dataViewModel.getItemNumber().observe(getViewLifecycleOwner(), observer);
    }

    private void setData(int i){
        readData = new ReadData();
        readData.readDataFile(getContext());
        float[][] data = readData.getArray2();

        ArrayList<Entry> values = new ArrayList<>();

        for(int j = 0; j < data[i].length; j++){
            values.add(new Entry(j, data[i][j], null, null));
        }

//省略

Fragmentの方、全て載せると長いのでデータセットに関する部分のみ抜粋しました。

###発生している問題
データの格納の仕方が悪いのか、Fragmentでの配列の要素の取得がわるいのかわかりませんが、data[i][j]がnullになってしまいます。nullになってしまう原因、対処がわからないので教えていただきたいです。

Android StudioのLogCatのエラーを確認したところ、setDataのfor文の中のdata配列で
java.lang.NullPointerException: Attempt to read from null array
と出ていました。

Comments

[hoshi-takanori]

array1 などを宣言だけして、実際の配列の生成をしてないのでは。
https://eng-entrance.com/java-array-twodimensional

[3mp]

ご指摘ありがとうございます。
float[][] array1　= new float[20][100]
のように適当な要素数で初期化してみましたが、データが格納されずにすべて0がはいってしまっていました

[hoshi-takanori]

ファイルはどこに置いてますか？ また、適切なパーミッションを設定していますか？

[3mp]

ファイルは外部ストレージのAndroid/data/にあるアプリのフォルダです。
パーミッションはREAD_EXTERNAL_STORAGE, WRITE_EXTERNAL_STORAGEを設定しています。
また、別画面でReadDataクラスのfileAccessメソッドを用いてrecyclerViewでファイルの一覧を表示しているのですが、そこは問題なく実装できています。

[hoshi-takanori]

dataFile.getName() でファイル名だけにして context.openFileInput() してるからでは。dataFile にフルパスが入ってるのでそれを直接開けばいい気がします。あと、LogCat のエラーを確認しましょう。

[3mp]

ありがとうございます。LogCatのエラーを解読してみます

Answer 1

解決が重複投稿されてしまっていたので削除しました

Answer 2

本来ならModelであるReadDataクラスにファイル読み込み処理を書きたいのですが、どうしても多次元化してしまい面倒なのでいったんGraphFragmentに読み込み処理を書きました。

FileInputStream in = context.openFileInputStream(fileName);

ファイルを開く際に上の書き方をやめ、下の書き方にしました。

FileInputStream in = new FileInputStream(フォルダのパス + fileName);

これでちゃんとファイルのデータ読み込みができました。

ご指摘いただきありがとうございました。

Comments

[3mp]

書き方を上記のように修正し、配列ではなくリストを使用したところ、ReadDataクラスに読み込み処理を任せることができました。
データの数がファイルによって異なるため、可変長であるリストを使用するのが適切だったようです。