bool型の配列では値をそれぞれ参照できないのか | コードはこれで合っているのか

Question

//これを配列にする理由は、一つずつbool変数の名前と値を書いていくのがめんどくさいから
bool[] arrayBool = { true };

//敵を倒すとポイントが増える(1ずつとは限らない)
int point = 0;

void Point()
{
    if(arrayBool[0] == true)
    {
        if(point >= 1)
        {
            Debug.Log("arrayBool[0] == true");
            arrayBool[0] = false;
        }
    }

    if(arrayBool[1] == true)
    {
        if(point >= 2)
        {
            Debug.Log("arrayBool[0] == true");
            arrayBool[1] = false;
        }
    }

    //これが全部で7回続く  ~arrayBool[7]
}

###コードについて追記
__if(arrayBool[n] == true)__はその中のif文を一度しか処理しないために組んだものです。
順序としては、arryaBool[]の7個の変数がすべてtrueであり、ある一定のpointが貯まると__if(point >= x)__内が実行され、arrayBool[n]をfalseにすることによってそれを今後処理しないようにします。

---

これで**Point()**が呼ばれたときにデバッグのログを出したいのですが、エラーが出ます。

正直言うと配列のことをあまり理解できていなくて誤解していると自分でも思っています。

配列とは、多くなってしまう変数を一つの変数からまとめて管理できるようにしたもので、
対象物が多いときに配列を使うと便利なもの。

という認識ですがこの考え方はあってるのでしょうか？

エラーも出ています。

---

IndexOutOfRangeException: Index was outside the bounds of the array.

エラー範囲はちょうどif文のところです。

---

どなたか回答お願いします。

Comments

[ozwk]

他で配列に代入してません？

[_Beginner]

多分していないと思います。
arrayBool[]が関わっているコードは編集で付け足しました。

[ozwk]

多分エラーが出たのは[1]の方ですよね？

Answer 1

if (arrayBool[0] == true) {

の前に、

	if (arrayBool.Length < 1) {
		Debug.Log("arrayBool.Length is 0");
		//Console.WriteLine("arrayBool.Length is 0");
		return;
	}

としたら、どうなるでしょう?

多分、"arrayBool.Length is 0" が出力されるのでは?

[追記]
もしかして、

if(arrayBool[1] == true)

でエラーが起きていませんか?

bool[] arrayBool = { true };

の宣言では、配列の長さは、 1です。　arrayBool[1] の参照はエラーとなります。

Comments

[_Beginner]

コンソールのデバッグはONにしていますが
それは出力されませんでした。

arrayBoolの個数は1以上あるということになります？

[pepperleaf]

> arrayBoolの個数は1以上あるということになります？
1つも無いという事です。
手元の確認では、
> static bool[] arrayBool = {};
としました。　どこかで類似の処理をしていると思います。

> //Console.WriteLine("arrayBool.Length is 0");
これは、単に、コマンドラインで、 Debug.Log() がエラーになったのでその代用。特に意味は無いです。

[_Beginner]

試しに宣言部分のコードを
readonly bool[] IncreaseDifficultyLevel = { true, true, true, true, true, true, true };
に変えてみた(これだと個数が7個ある...?)のですが、
いつものエラーが出ます。

---

＞bool[] arrayBool = { true };　の宣言では、配列の長さは、 1です。　arrayBool[1] の参照はエラーとなります。

これはつまり arrayBool[0]の値がtrueでarray[1]は宣言されていないということですか？

語彙力＆理解力なくてごめんなさい

[pepperleaf]

変えるならば、
> bool[] arrayBool = { true, true, true, true, true, true, true, true };
ではないでしようか? (arrayBool[7] だったら、 8個)
また、
> readonly bool[] IncreaseDifficultyLevel
とすると、 IncreaseDifficultyLevel[1] = false; とかはエラーになりますが、誤解してないですか?

Answer 2

IndexOutOfRange はインデクスの範囲外にアクセスしたときに出る例外です。
この場合、インデクス 0 にアクセスしているので、何か一つでも要素が入っていたら出ません。
ということは、初期化するタイミングが悪いために配列に何も入っていないと言うことです。
もう少し広い範囲のコード見なければどのように修正すれば良いか判断ができませんが、初期化される前にアクセスされています。

#追記

if(arrayBool[1] == true)

ここで出ているのでは？

Comments

[Zuishin]

または、別のところで別の配列を入れています。null でないところを見るとこちらかもしれません。

[_Beginner]

＞別のところで配列を入れる
というのは、arrayBool[n] = true を別のところで行っているということですか？

[Zuishin]

ソースが無いのでどこで何をしているのかわかりません。可能性の話です。
その後質問が編集されたのでそれを見て追記しました。
まずどこで例外が起こっているのかを確認してください。

[pepperleaf]

> 例外が起こっているのか
例外が起きている正確な場所(コード位置)ですね。

[Zuishin]

そうですね。エラーメッセージに行番号が書いてあると思うのですが。

Answer 3

bool[] arrayBool = new bool[]{ true };

Comments

[_Beginner]

同じくIndexOutOfRangeException: Index was outside the bounds of the array.というエラーが出ます

[y_waiwai]

それ、なんかおかしなことになってますな
コード全体を提示できない？

[_Beginner]

arrayBool[]が関わっているところを書きました。
poin整数型は敵を倒したりしたときに手に入れるものです。
これらのif文はレベルアップ時の報酬みたいなもので、一度レベルアップしたらその処理は二度と起こさないようにするために組んでいます。

[y_waiwai]

arrayBool自体にどっかで代入してませんか？
VisualStudioなら、要所で止めて変数の内容モニタできるんで、どこでarrayBoolが変わるのか追いかけてみれば？

Answer 4

他の回答者さんも言ってくれたように、配列に何も入っていない状態でした。
僕の理解力が乏しくなかなか答えにたどり着けませんでしたがようやく解決できました。
本当にありがとうございました！
今後も無知な質問ばかりしてしまうと思いますがよろしくおねがいします。

どの回答もとても勉強になり解決への助けになりました。
ベストアンサーに迷ったのですが、最後はこちらで最終的なコードを添付してクローズさせていただこうと思います。

以下コード

//これを配列にする理由は、一つずつbool変数の名前と値を書いていくのがめんどくさいから
bool[] arrayBool = new bool[] { true, true, true, true, true, true, true, true, true };

//敵を倒すとポイントが増える(1ずつとは限らない)
int point = 0;

//ポイントが増えた際に呼ばれる
void Point()
{
    if(arrayBool[0] == true)
    {
        if(point >= 1)
        {
            Debug.Log("arrayBool[0] == true");
            arrayBool[0] = false;
        }
    }

    if(arrayBool[1] == true)
    {
        if(point >= 2)
        {
            Debug.Log("arrayBool[0] == true");
            arrayBool[1] = false;
        }
    }

    //これが全部で7回続く  ~arrayBool[7]
}

Comments

[y_waiwai]

グローバルとstartの中でnewしてますがこれは無駄です
start関数が必ず実行されるなら、そこだけでnewさせればいいです

[_Beginner]

なるほど
修正させていただきます！

宣言と初期化を同時にしちゃいます。

Answer 5

bool[] arrayBool = new bool[2];
又は
bool[] arrayBool = new bool[]{true,true, true,true, true,true,true };

配列はnewしたときのサイズで固定なのでサイズ以上のindexは使えません

ちなみにpublic List<bool> arrayBoolまたは[SerializeField]private List<bool> arrayBoolを使えば（newはしない）unityのインスペクターから初期値を指定できます

Comments

[Zuishin]

> //これが全部で7回続く ~arrayBool[7]

らしいです。

[bochan2]

ご指摘ありがとうございます
編集しておきます

[_Beginner]

bool[] arrayBool = new bool[7];
もしくは
bool[] arrayBool = new bool[]{true,true,true,true,true,true,true,true};

つまり

bool[] arrayBool = new Bool[7];
を宣言して
Startメソッドなどで
bool[] arrayBool = new bool[]{true,true,true,true,true,true,true,true};
を定義してやればいいのでしょうか？

[bochan2]

bool[] arrayBool = new bool[]{true,true,true,true,true,true,true,true};はstaticでなくても使えますよ

[_Beginner]

上のように最初に宣言、Startメソッドで定義した所、理由はよくわかりませんがうまくいきました！

他の回答者さんも言ってくれたように、配列に何も入っていない状態でした。
僕の理解力が乏しくなかなか答えにたどり着けませんでしたがようやく解決できました。

本当にありがとうございました！