Unity2D StackOverflowException

Question

#pragma warning disable 0414
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Statistics : MonoBehaviour {

    public Text targetText;

    int Level;//レベル
    string Ability;//特性
    int HP;
    int Attack;//こうげき
    int Defense;//ぼうぎょ
    int Sp_Atk;//とくこう
    int Sp_Def;//とくぼう
    int Speed;//すばやさ
    int Evasion;//回避
    int Accuracy;//命中
    string Gender;//性別
    int Exp;//経験値
    int Individual_Value;//個体値
    int ATKorSPATK;

    private void Awake()
    {
        Generate_Stats();
        targetText = this.GetComponent<Text>();
    }

    void Generate_Stats()
    {
        //int Random_Level = Random.Range(1, 101);
        int Random_Ability = Random.Range(1, 11);
        int Random_HP = Random.Range(1, 256);
        int Random_ATK = Random.Range(1, 256);
        int Random_DEF = Random.Range(1, 256);
        int Random_Sp_Atk = Random.Range(1, 256);
        int Random_Sp_Def = Random.Range(1, 256);
        int Random_Speed = Random.Range(1, 256);
        int Random_Evasion = Random.Range(1, 101);
        int Random_Accuracy = Random.Range(1, 101);
        int Random_Gender = Random.Range(0, 2);
        Exp = 0;
        int Random_IV = Random.Range(100, 601);
        int Random_ATKorSPATK = Random.Range(0, 2);

        Level = 1;
        HP = Random_HP;
        Attack = Random_ATK;
        Defense = Random_DEF;
        Sp_Atk = Random_Sp_Atk;
        Sp_Def = Random_Sp_Def;
        Speed = Random_Speed;
        Evasion = Random_Evasion;
        Accuracy = Random_Accuracy;
        Individual_Value = Random_HP + Random_ATK + Random_DEF
                           + Random_Sp_Atk + Random_Sp_Def + Random_Speed;

        if (Random_Ability == 1)
        {
            Ability = "Power Boost";//ATKが二倍になる・パワーブースト
        }
        if (Random_Ability == 2)
        {
            Ability = "Poison Body";//歩いた場所が毒のマスになる・どくのからだ
        }
        if (Random_Ability == 3)
        {
            Ability = "Water Barrier";//状態異常にならない・みずのバリア
        }
        if (Random_Ability == 4)
        {
            Ability = "Levitate";//異常マスを踏んでもダメージを受けない・ふゆう
        }
        if (Random_Ability == 5)
        {
            Ability = "Shed Skin";//体力が半分以下になるととくこうが二倍になる・だっぴ
        }
        if (Random_Ability == 6)
        {
            Ability = "Insecticide";//分類・虫の相手に与えるダメージが二倍になる・さっちゅう
        }
        if (Random_Ability == 7)
        {
            Ability = "Quick Pace";//移動するスピードが二倍になる。ニマスずつ進む・はやあし
        }
        if (Random_Ability == 8)
        {
            Ability = "Damage Filter";//受けるダメージが三分の二になる。・ダメージフィルター
        }
        if (Random_Ability == 9)
        {
            Ability = "Solution";//広範囲にダメージを与える・ようかいえき
        }
        if (Random_Ability == 10)
        {
            Ability = "Aftermath";//もしも相手の攻撃によって死亡したとき、相手のＨＰの三分の一のダメージを与える・ゆうばく
        }

        if (Individual_Value != Random_IV)
        {
            Generate_Stats();
        }
        else
        {
            targetText.text = "Level      " + Level.ToString() + "\n" + "Ability    " + Ability + "\n" + "IV         " + Individual_Value.ToString() + "\n" + 
                "HP         " + HP.ToString() + "\n" + "Attack     " + Attack.ToString() + "\n" + "Defense    " + Defense.ToString() + "\n" +
                "Sp_Atk     " + Sp_Atk.ToString() + "\n" + "Sp_Def     " + Sp_Def.ToString() + "\n" + "Speed      " + Speed.ToString() + "\n" +
                "Evasion    " + Evasion.ToString() + "\n" + "Accuracy   " + Accuracy.ToString();
        }
    }
}

３５行目の「int Random_Ability = Random.Range(1, 11);」がエラーになってしまいます。

エラーメッセージは

StackOverflowException
UnityEngine.Random.Range (Int32 min, Int32 max) (at C:/buildslave/unity/build/artifacts/generated/common/runtime/RandomBindings.gen.cs:63)

です。

Random.Rangeを使わずに、イコール（１～１０）までにすると、普通に動作します。ですが、ランダムで１から１０まで数字を選びたいので、Random.Rangeを使う必要があります。

どう書き直せばよいでしょうか？　教えていただけるとありがたいです。

回答よろしくお願いします。

Answer 1

StackOverflowとは、メモリのスタック領域を、その領域よりも多く使おうとしてしまうことです。
詳しいことは自分で調べて欲しいと思いますが、その原因として考えられるのは関数の呼び出しすぎです。
現在のPCではメモリが多いので、普通にしていれば問題ないと思いますが、例えば、

void Foo()
{
    Foo();
}

のようにしてしまうと、Foo()が永遠に呼ばれ続け、しまいにはメモリがFoo()だらけになってしまいます。

同じように、同じ関数自身を呼び出している箇所があります。
それは、以下の箇所です。

        if (Individual_Value != Random_IV)
        {
            Generate_Stats();
        }

この場合、Individual_Value == Random_IVとなればこの現象から抜け出せるので、永遠ではないですが、

        int Random_IV = Random.Range(100, 601);

となっているので、単純に考えると500回に1回ぐらいの確率しか抜け出せません。
平均500回も関数を呼び出していては、メモリのスタック領域を超えてもおかしくないと思いますが、どうでしょうか？

なお、これが正しければ、int Random_Ability = Random.Range(1, 11);で起きたのはただの偶然でしょう。
何回か実行していれば、そのうち他の箇所で同じエラーが起きるでしょう。

Comments

[fiveHundred]

…と思ったけど、仮にRandom_IVの値を固定しても、Individual_Value == Random_IVが満たされる確率は限りなく低いですね。

[退会済みユーザー]

回答ありがとうございました。エラーにならずにIndividual_Value == Random_IVにするためには、どのように書けばいいのでしょうか？もしよろしければ、教えていただきたいです。

[fiveHundred]

Random_IVをランダムに生成し、そのRandom_IVの数だけ「Random_HP〜Random_Speedのいずれかを+1する」を繰り返せば、パラメータをランダムに振り分けることが出来ます。
この方法を取れば、Individual_Value == Random_IVの判定はそもそも要らないでしょう。

[fiveHundred]

あるいは、そもそもRandom_HP〜Random_Speedはランダムなので、Individual_Value == Random_IVではなく「100 <= Individual_Value && Individual_Value < 601」としても良いでしょう。
この程度であれば確率的には問題ないと思いますが、心配であれば再帰呼び出しではなくループを使うのも良いかもしれません。

[退会済みユーザー]

回答ありがとうございます。出来ました。ありがとうございました。