行列データ 配列 格納

Question

###前提・実現したいこと
3行3列のテキストデータを読み込み3行3列の配列に格納
行数は不確定にする予定

お願いいたします。
###発生している問題・エラーメッセージ

1行目しか配列に入らない

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

       is = this.getAssets().open("sample.txt");
            br = new BufferedReader(new InputStreamReader(is));
            
            String str;
            while ((str = br.readLine()) != null) {
                text.add(str);

            }
            nums = new int[text.size()][3];
            for (int i = 0; i < text.size(); i++) {
                String[] s = text.get(i).split(" ");
                for (int j = 0; j < 3; j++) {
                    nums[i][j] = (int) Integer.parseInt(s[j]);
                }

###試したこと

行数は不確定になる場合のためArrayListを用意
1行目はすべて配列に入った

textView.setText(String.valueOf(nums[0][2]))は該当数字が表示され、textView.setText(String.valueOf(nums[2][0]))は0が表示されてしまいます。

Answer 1

textの型がArrayList<String>だとすると上のプログラムで動くような気がします。
１行しかできなかったということであればsample.txtが1行しかなかったのだと思います。

String str;
while ((str = br.readLine()) != null) {
  text.add(str);
}
Log.d("xxx", "line count = " + text.size());
...

これでログに"line count = 1"と出るんじゃないんでしょうか。

あるいは他の可能性として「2行目に数字が3要素未満しかなかった」「2行目に空白が2つ連続していた」等々で例外が発生していたとも考えられますが・・・

Comments

[pro]

回答ありがとうございます。
textの型がArrayList<String>で、line　countは３でした。

[KSwordOfHaste]

KiyoshiMotokiさんへのコメントでわかりました・・・
そもそも数字３要素という前提が崩れている入力ですね・・・
途中で例外が発生するということでしょう。

[pro]

KiyoshiMotokiさんに対する返信は間違えました。
ログ出力はスペースで区切られたものでした。

Answer 2

Java SE 8以降なら????

is = this.getAssets().open("sample.txt");
BufferedReader br = new BufferedReader(new InputStreamReader(is));
int[][] matrix = br.lines()
        .map(line ->
                Stream.of(line.split(" "))
                        .mapToInt(Integer::parseInt)
                        .toArray())
        .toArray(int[][]::new);

Answer 3

ソースコードが途中で途切れているように見えます。もしかして、この先にbreakが潜んでいたりしませんか？見えているコードは問題なさそうなので、sample.txtが1行しかないか、breakされて1行目しか処理されないか、出力するときに間違えて1行目のデータしか見ていないか、のどれかかと思います。

Answer 4

sample.txt

1 2 3
4 5 6
7 8 9

Main.java

import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;

public class Main {

    public static void main(String[] args) throws IOException {
        List<String> text = new ArrayList<String>();
        int[][] nums;

        InputStream is;
        BufferedReader br;

        ////////////////////////////////////////////////////////
        is = new FileInputStream("sample.txt");
        br = new BufferedReader(new InputStreamReader(is));

        String str;
        while ((str = br.readLine()) != null) {
            text.add(str);

        }
        nums = new int[text.size()][3];
        for (int i = 0; i < text.size(); i++) {
            String[] s = text.get(i).split(" ");
            for (int j = 0; j < 3; j++) {
                nums[i][j] = (int) Integer.parseInt(s[j]);
            }
        }
        ////////////////////////////////////////////////////////

        for (int[] num : nums) {
            for (int n : num) {
                System.out.print(n + ",");
            }
            System.out.println();
        }
    }

}

実行結果

1,2,3,
4,5,6,
7,8,9,

---

おそらく、sample.txtが意図するフォーマットになっていないと思われます。
以下のようにデバッグコードを追加し、logcat にどのように表示されるか確認してみてください。

Before

while ((str = br.readLine()) != null) {
    text.add(str);

}

After

while ((str = br.readLine()) != null) {
    Log.e("My App", str);
    text.add(str);
}

追記

sample.txt

1 2 3
4 5 6 0
7 8 9 
10 11 12

Main.java

import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;

public class Main {

    public static void main(String[] args) throws IOException {
        int[][] nums;

        InputStream is;
        BufferedReader br;

        // //////////////////////////////////////////////////////
        is = new FileInputStream("sample.txt");
        br = new BufferedReader(new InputStreamReader(is));

        List<int[]> table = new ArrayList<int[]>();

        String str;
        while ((str = br.readLine()) != null) {
            String[] splitted = str.trim().split(" ");
            if (splitted.length < 3) {
                throw new IllegalStateException("Too few element.");
            }

            int[] row = new int[3];
            for (int i = 0; i < 3; i++) {
                row[i] = Integer.parseInt(splitted[i]);
            }
            table.add(row);
        }
        nums = table.toArray(new int[][] {});
        // //////////////////////////////////////////////////////

        for (int[] num : nums) {
            for (int n : num) {
                System.out.print(n + ",");
            }
            System.out.println();
        }
    }
}

実行結果

1,2,3,
4,5,6,
7,8,9,
10,11,12,

Comments

[pro]

回答ありがとうございます。
ログ出力してみたところこのように出ました。。。
E/My App:1 2 3
E/My App:4 5 6
E/My App:7 8 9

[KiyoshiMotoki]

つまり、sample.txt の内容は

123
456
789

のようになっている、ということです。
しかし、ご提示のソースコードは、

1 2 3
4 5 6
7 8 9

という、各数字がスペース 1文字で区切られていることを期待したものになっています。

現状の sample.txt とソースコード、どちらがあなたの意図する sample.txt のフォーマットですか？

[pro]

スペースで区切ったものが意図しているフォーマットです。

[pro]

ログ出力はスペースで区切られてました、すみません

[KiyoshiMotoki]

> ログ出力はスペースで区切られてました、すみません

しかし、ご提示のログはスペースで区切られていません。
そのため、あなたの仰ることが本当に正しいのか、私には確証が持てません。

情報を提示する際は下手にご自分で編集したりせず、なるべくそのままコピペするようにしてください。
https://teratail.com/help/question-tips#questionTips3-4-2

とりあえず、sample.txt は本当に「スペースで区切られて」いるという前提で正しく動作するコードを回答欄に追記しました。
確認してみてください。

それでも
> 1行目しか配列に入らない

ということは、コードのどこかで例外が発生し、ループを抜けてしまっている、という可能性が考えられます。
（例えば、sample.txt の中にスペース2文字以上で区切られている箇所がある、など）

その際は、以下を参考にデバッグしてみてください。
https://developer.android.com/studio/debug/index.html?hl=ja#breakPoints

[pro]

ご指摘ありがとうございます。
提示してくださったコードを元に考えたいと思います。

[pro]

無事解決しました。
問題点は複数For分の構成でした。
ご丁寧な解説ありがとうございました。

[KiyoshiMotoki]

Good Job!