Unityでtxtファイルを読み込んで配列に格納したい

Question

Unityでtxtファイルに保存した数字の羅列を読み込みたい

Unityのスクリプトをc#で書いています。c#を使うのは今回が初めてです。
あらかじめ保存してあるtxtファイルの中の数字を読んで配列に格納する関数を作っています。
ReadLine を使ってtxtファイルの中身を読んできた後、float型に変換してから配列に格納したいと思いfloat.Parseを使っています。

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

"Index was outside the bounds of the array"と出ます。
配列まわりで怒られているのはわかるのですが具体的になにが悪いのかわかりません。
エラーメッセージ内のcs:226は後述のソースコードのfloat.Parseの行を指しています。

IndexOutOfRangeException: Index was outside the bounds of the array.
AGXUnity_Excavator.Scripts.ExcavatorInputController.ReadFile () (at Assets/AGXUnity_Excavator/AGXUnity_Excavator_Assets/Scripts/ExcavatorInputController.cs:226

該当のソースコード

public string boomin ;
public float[] boominput = new float[150];

void ReadFile()
    {
                using (TextReader tr1 = File.OpenText("../TextData.txt"))
                {
                  for(int i = 0;i<150;i++)
                    {
                      boomin = tr1.ReadLine();
                      Debug.Log(boomin);
                      boominput[i] = float.Parse(boomin);
                    }
                 }
    }

###該当のtxtファイル
0
0
0
0
0
0
0
0
0
0.3
0.3
のような形で1500行ほど続きます
今回のソースコードでは試しに上150個の数字を読んでみています

実行結果

for文の一回目の処理でエラー文を吐いて止まります。
Debug.logの出力はtxtファイルの一行目通りにしっかり0がログに出ますが、その次のfloat.Parseの処理ができないようでエラーがでます。

試したこと

float.Parseの行をコメントアウトすると150行分の数字がしっかりデバックログに表示されるのでreadlineは機能していると思います。

Answer 1

このエラーだと配列の数が足りてないとかそのへんかなーと思います。

まずboominputの配列はちゃんと数が確保できてますか？
publicで宣言しているのでインスペクター側から数値を変えたりとかしてるとそっちが使われちゃいます。念の為、ちゃんと宣言されているか確認してみてください。あとは読み込む前に再初期化しても良いです。

public string boomin ;
public float[] boominput = new float[150];

void ReadFile()
    {
        Debug.Log("配列の数: " + boominput.Length); // デフォルトの配列数
        boominput = new float[150]; // 配列の初期化し直し

                using (TextReader tr1 = File.OpenText("../TextData.txt"))
                {
                  for(int i = 0;i<150;i++)
                    {
                      boomin = tr1.ReadLine();
                      Debug.Log(boomin);
                      boominput[i] = float.Parse(boomin);
                    }
                 }
    }

#追記
長さが決まってないなら、配列じゃなくてListを使うほうが楽だと思います。
ReadLineがnullってことは行末のはず＝それまで読み込めばファイルの全行を読み込む という考えからwhile文で回します

using System.Collections;
using System.Collections.Generic;
using System.IO;
using UnityEngine;

public class Test : MonoBehaviour {

    public string boomin;
    public List<float> inputDataList = new List<float>();

    private void Start() {
        ReadFile();
    }

    void ReadFile() {
        using (TextReader tr1 = File.OpenText("../TextData.txt")) {
            while((boomin = tr1.ReadLine()) != null) {
                inputDataList.Add(float.Parse(boomin));
            }
        }
    }
}

Comments

[ryosan0201]

配列の初期化をしたら問題なく動きました！ありがとうございます
ただ、特にほかの関数やupdate処理の中でこの配列に書き込んだりもしていないので、なぜエラー吐いてたのかはよくわかりませんでした,,,

後々には配列の個数を150などの固定値ではなく読む先のファイルの長さによって変えたいのですが難しいのでしょうか,,,?

[hogefugapiyo]

回答追記しました

[ryosan0201]

ありがとうございます！
リスト使っても問題なく動きました！

Answer 2

boominput[i]のiが配列の範囲を越えているようですがコードからおかしそうな箇所がありません。
記載していない箇所でboominputを上書きしていませんか？

以下のようにログ出力と配列の長さでループするとどうなりますか？

Debug.Log($"boominput.Length: {boominput.Length}");
for(int i = 0;i<boominput.Length;i++)

また、TextReader.ReadLine()は最後まで読み込むとnullを返すのでbreakで抜けてしまった方がよいです。

Comments

[ryosan0201]

boominput.lengthが０と出てしまいました
他の回答者様から教えていただいた通りReadFile関数に入ってから配列を宣言しなおすと問題なく動きました！
breakを使ったほうが良いとおっしゃっていましたがfor文の最後にbreakをつけるという認識で合ってますか？