実行した後、暫くしてインデックスが配列の境界外といった内容のエラーが表示される。

Question

シーン上のタグが付いたゲームオブジェクトを取得し、配列に入れた後、
変数にランダムにオブジェクトの座標を代入し、その座標のオブジェクトを削除するスクリプトを書きました。
実行して最初の内は問題無いのですが、
シーン上のオブジェクトの数が少なくなってくると下記のエラーが表示されます。
IndexOutOfRangeException: Index was outside the bounds of the array.
BallGun.PosT () (at Assets/main/Script/BallGun.cs:150)
コードはこちらになります。

    public void PosT()
    {
        if (H_m == false)
        {
            return;
        }
        
            TargetObject = GameObject.FindGameObjectsWithTag("Target");
            Debug.Log("Target数;" + TargetObject.Length);
            if (TargetObject.Length == 0)
            {
                GameResult.clear = true;

                script.GetComponent<GameResult>().Owari();
            }
            else if (TargetObject == null)
            {
                Debug.Log("無い");
            }
            int number = Random.Range(1, TargetObject.Length);
            a_p = TargetObject[number].transform.position;
        
    }

どうかお知恵をかしていただけないでしょうか？

Answer 1

こんにちは。

ぱっと見ですが、Random.Range(1, TargetObject.Length); がおかしいです。

この定義では Range は 1 以上 Length 未満の値を返しますが、配列の index は 0 からです。
ここで、TargetObject.Length が 1 になった場合を考えると、
Random.Range(1, 1) == 1 となり、
TargetObject は 1 件、つまり [0] のみが存在するところに 1 でアクセスしており、IndexOutOfRangeException となります。

つまり、TargetObject が 2 件以上ある場合はエラーは発生しませんが、条件が常に 1 以上 Length 未満 なので、最初の一つ (index == 0) が決して選択されない状態になっています。

Answer 2

C#の配列は最初の要素は1ではなく0です。

また、Random.Rangeには

max と min が等しければ、min が返されます。

と書かれています。

つまり、TargetObject.Lengthが1のとき、numberには1が保持されます。
このとき、TargetObject[0]にはアクセスできますが、TargetObject[1]にはアクセスできません。

掲示されたコードはTargetObject[1]にアクセスしているためIndexOutOfRangeExceptionの例外が発生しています。
乱数の最小値を1ではなく0を返すように修正してください。

- int number = Random.Range(1, TargetObject.Length);
+ int number = Random.Range(0, TargetObject.Length);

Comments

[sakura_hana]

この修正を加えた場合でも配列要素数が0の場合に「TargetObject[0]」にアクセスしようとするので、if分岐の追加が必要かと思います。

[tamoto]

要素数が 0 の場合の早期脱出が既に書かれているため問題ないと思います。直後に全く意味をなさない is null とかもありますが……

>>>
と思ったら、特に中で脱出してませんでしたね。質問中で省略しているわけでは……どうなんでしょう。

[BluOxy]

sakura_hanaさん、補足ありがとうございます。

> evolsnakeさん
sakura_hanaさんの補足についてもIndexOutOfRangeExceptionを発生させないためのポイントになるので考慮してください。

[sakura_hana]

> tamotoさん
変数代入してメソッド呼んでるだけなんで脱出してないですね。（OwariメソッドからPosTメソッドを終了させるのは多分出来ないはず、出来てもエラーか警告起こりそう）
if(TargetObject == null) が全く無意味なのはその通りです。

[sakura_hana]

（すいません被りました）
if (TargetObject.Length == 0) 内の最後に return; 追加
int number = Random.Range(1, TargetObject.Length);の1を0に変更
で動くと思います。

[BluOxy]

ですね。returnするなどして、TargetObject.Lengthが0のときに要素へアクセスしないようロジックを直せば良さそうです。

Answer 3

Random-Range - Unity スクリプトリファレンス

max と min が等しければ、min が返されます。

これにより、次のコードで TargetObject.Length が 1 の時、number は 1 になります。

        int number = Random.Range(1, TargetObject.Length);

しかし、TargetObject.Length が 1 の時 TargetObject[1] は存在しないので、次の行で例外が発生します。

a_p = TargetObject[number].transform.position;

Answer 4

            int number = Random.Range(0, TargetObject.Length-1);

Random.Rangeは第２引数の値までの数値を返します
配列のインデックスの最大値は配列の要素数-1なので、最大値がランダムで生成されたときに例外がおきてます
なのでランダム値生成は配列の要素数-1までにしなければなりません

Comments

[BluOxy]

リファレンスには
> min (この値を含む) と max (この値を含まない) の範囲のランダムな整数を返します
と書かれているので、Length-1の考慮は必要なさそうに思います。