Unityでオブジェクトの切り替えをしたい

Question

実現したいこと

Unityで2Dゲームを開発中で、aというGameObject(アニメーション付き)とbというGameObject(アニメーションなし)を、あるif文の条件式で表示または非表示にうまく切り替えることが目的です。

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

エラー表示は特になく、Unity上でゲームを再生することは可能です。なので、原因はプログラミングによるエラーではないと考えられますが(Debug.Logで指示を書きたい領域が機能しているかをすでに確認済)、原因の箇所が全く分からずにいます。
<具体的な症状>
・以下のスクリプト参照
if(割愛)
{
ToBchange();
}
において、aはGame画面上に表示されたままアニメーションはなぜか静止する。bはGame画面上に表示されない。
else
{
ToAchange();
}
において、aはGame画面上に表示されたままアニメーションが再び動き出す。bはGame画面上に表示されない

<理想>
if(割愛)
{
ToBchange();
}
において、aはGame画面上から消え、bはGame画面上に表示される。
else
{
ToAchange();
}
において、aはGame画面上に表示され、bはGame画面上から消える。

該当のソースコード

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

public class PlayerManager : MonoBehaviour
{
    public GameObject migimigiba;
    public GameObject hidarihidariba;

    [SerializeField] GameObject a;
    [SerializeField] GameObject b;


    void Start()
    {
        migimigiba = GameObject.Find("migiba");
        hidarihidariba = GameObject.Find("hidariba");
    }

    // Update is called once per frame
    void Update()
    {
        for (int i = 0; i <= 8; i++)
        {
            for (int j = 0; j <= 8; j++)
            {
                if (migimigiba.GetComponent<BluebarMove>().rightbar[i] == true && hidarihidariba.GetComponent<RedbarMove>().leftbar[j] == true)
                {
                    ToBchange();　//Debug.Logで確認済　問題なし
                }
                else 
                {
                    ToAchange();   //Debug.Loでg確認済  問題なし             
                }
            }
        }


    public void ToBchange()
    {
        a.SetActive(false);
        b.SetActive(true);
    }

    public void ToAchange()
    {
        a.SetActive(true);
        b.SetActive(false);
    }

}             

試したこと

Debug.Logで指定の領域がちゃんと機能していることは確認済みです。エラーもないです。 [SerializeField] で書けば非アクティブ状態のオブジェクトも取得できるとの情報があったので、そうしましたが、目的は達成できませんでした。となると、SetActiveの使い方が間違っているのでしょうか？

追記

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

public class PlayerManager : MonoBehaviour
{
    public GameObject migimigiba;
    public GameObject hidarihidariba;

    GameObject a;
    GameObject b;


    void Start()
    {
        migimigiba = GameObject.Find("migiba");
        hidarihidariba = GameObject.Find("hidariba");
        a = GameObject.Find("A");
        b = GameObject.Find("B");
    }

    // Update is called once per frame
    void Update()
    {
        if(migimigiba.GetComponent<BluebarMove>().rightbar[0] == true && hidarihidariba.GetComponent<RedbarMove>().leftbar[0] == true)
        {
            a.transform.position = new Vector3(-4.2f, -0.19f, 0);
            b.transform.position = new Vector3(-1.6f, -0.19f, 0);
        }
        else
        {
            a.transform.position = new Vector3(-1.63f, -0.19f, 0);
            b.transform.position = new Vector3(-4.13f, -0.19f, 0);
        }

        if (migimigiba.GetComponent<BluebarMove>().rightbar[0] == true && hidarihidariba.GetComponent<RedbarMove>().leftbar[1] == true)
        {
            a.transform.position = new Vector3(-4.2f, -0.19f, 0);
            b.transform.position = new Vector3(-1.6f, -0.19f, 0);
        }
        else
        {
            a.transform.position = new Vector3(-1.63f, -0.19f, 0);
            b.transform.position = new Vector3(-4.13f, -0.19f, 0);
        }
　　　　　というような感じで同じような処理が続いていきます。
    }
}

というような感じで、for文ではなく上記のように(rightbar[0]かつleftbar[0]のとき,rightbar[0]かつleftbar[1]のとき, rightbar[0]かつleftbar[2]のとき ・・・と個別に処理を書くと目的の動作が実行されました。for文だとi=8, j=8の最後の処理しか実行されないとの指摘を受け、まさにその通りでした。

ただ、これだと(rightbar[0]かつleftbar[0]～rightbar[8]かつleftbar[8]の計81個の処理をコピペする必要があり、もっとスマートに書ける処理を考え中です。)
この記載しているスクリプト内で処理が完結できればいいなと思っています。

Comments

[YAmaGNZ]

SetActiveの使い方については知りませんが、ループ内で行うとループの最後の条件の処理（i=8、j=8の時の結果）となりますがいいのですか？

[退会済みユーザー]

再び回答して頂き、ありがとうございます。おっしゃる通り、forだとi=8, j=8の処理しか実行されませんでした。なので、for以外のベストな方法を探っています。詳細は質問内容の編集で記載しました。

[YAmaGNZ]

追記されたソースでも
migimigiba.GetComponent<BluebarMove>().rightbar[0]=true
hidarihidariba.GetComponent<RedbarMove>().leftbar[0]=rue
hidarihidariba.GetComponent<RedbarMove>().leftbar[1]=false
の場合
a.transform.position = new Vector3(-4.2f, -0.19f, 0);
が実行された直後に
a.transform.position = new Vector3(-1.63f, -0.19f, 0);
が実行され、最終結果が
a.transform.position = new Vector3(-1.63f, -0.19f, 0);
となりますがいいのですか？

というかソースを出すたびになんで処理が変わるんでしょう？

[YAmaGNZ]

なんとなくaやbの表示・非表示を切り替えるための条件の考え方がよくないように感じます。
追記されたソースを見る限り、
migimigiba.GetComponent<BluebarMove>().rightbarとhidarihidariba.GetComponent<RedbarMove>().leftbarの中にそれぞれtrueがあれば
a.transform.position = new Vector3(-4.2f, -0.19f, 0);
b.transform.position = new Vector3(-1.6f, -0.19f, 0);
を実行し全部falseであれば
a.transform.position = new Vector3(-1.63f, -0.19f, 0);
b.transform.position = new Vector3(-4.13f, -0.19f, 0);
を実行するという処理を作りたいように読めます。

一旦この配列の組み合わせ81通りを全部チェックする必要があるのかから考え直してはどうでしょうか？

[退会済みユーザー]

確かにそうですね。もう一度根本の部分から考え直して見ます。

[退会済みユーザー]

無事解決に至りました。いろいろアドバイス頂き、ありがとうございました。

Answer 1

for (int i = 0; i <= 8; i++)
{
    if(migimigiba.GetComponent<BluebarMove>().rightbar[i] == true) right = true;
    if(hidarihidariba.GetComponent<RedbarMove>().leftbar[i] == true) left = true;
}
if(right && left)
{
    a.transform.position = new Vector3(-4.2f, -0.19f, 0);
    b.transform.position = new Vector3(-1.65f, 0.03f, 0);
}
else
{
    a.transform.position = new Vector3(-1.63f, -0.19f, 0);
    b.transform.position = new Vector3(-4.13f, -0.19f, 0);
}

で終わりだったりしません？

Comments

[YAmaGNZ]

どのような条件時にダメだったのでしょうか？

Answer 2

rightbar と leftbar の型が不明ですが List や配列であれば Linq の Any を使うのがスマートでしょう。

using System.Linq;

if (migimigiba.GetComponent<BluebarMove>().rightbar.Any(n => n == true) && hidarihidariba.GetComponent<RedbarMove>().leftbar.Any(n => n == true)) {
    a.transform.position = new Vector3(-4.2f, -0.19f, 0);
    ...
}

Answer 3

    void Update()
    {
        bool flag = false;
        
        for (int i = 0; i <= 8; i++)
        {
            for (int j = 0; j <= 8; j++)
            {
                if (migimigiba.GetComponent<BluebarMove>().rightbar[i] == true
                    && hidarihidariba.GetComponent<RedbarMove>().leftbar[j] == true)
                {
                    flag = true;
                }
            }
        }
        
        if (flag)
        {
            a.transform.position = new Vector3(-4.2f, -0.19f, 0);
            b.transform.position = new Vector3(-1.65f, 0.03f, 0);
        }
        else
        {
            a.transform.position = new Vector3(-1.63f, -0.19f, 0);
            b.transform.position = new Vector3(-4.13f, -0.19f, 0);
        }
    }

Comments

[退会済みユーザー]

回答ありがとうございます。
確かにこのコードでも動作しましたので、これを参考にさせて頂きます。「配列とboolの組み合わせ」ということですね！　大変勉強になりました。

Answer 4

以下のコードに変更することで、目的を達成できました(動作確認済)。1度だけ81個の処理をif文の()に書くことを我慢すれば、{}内での処理を気軽にまとめて変更できるようになりました。if((A&&B)||(A&&C)||(A&&D)||・・・)と無限に使えることを知らなかったのが原因でした。

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

public class PlayerManager : MonoBehaviour
{
    public GameObject migimigiba;
    public GameObject hidarihidariba;

    GameObject a;
    GameObject b;


    void Start()
    {
        migimigiba = GameObject.Find("migiba");
        hidarihidariba = GameObject.Find("hidariba");
        a = GameObject.Find("A");
        b = GameObject.Find("B");
    }

    // Update is called once per frame
    void Update()
    {
        if((migimigiba.GetComponent<BluebarMove>().rightbar[0] == true && hidarihidariba.GetComponent<RedbarMove>().leftbar[0] == true) || (migimigiba.GetComponent<BluebarMove>().rightbar[0] == true && hidarihidariba.GetComponent<RedbarMove>().leftbar[1] == true) || (migimigiba.GetComponent<BluebarMove>().rightbar[0] == true && hidarihidariba.GetComponent<RedbarMove>().leftbar[2] == true)・・・全部で81個の条件を書く)
        {
            a.transform.position = new Vector3(-4.2f, -0.19f, 0);
            b.transform.position = new Vector3(-1.65f, 0.03f, 0);
        }
        else
        {
            a.transform.position = new Vector3(-1.63f, -0.19f, 0);
            b.transform.position = new Vector3(-4.13f, -0.19f, 0);
        }
    }
}

Comments

[fiveHundred]

> 1度だけ81個の処理をif文の()に書くことを我慢すれば、

ということになっていること自体、とてもいい方法だとは思えません。
代案を回答しましたので、ご確認ください。

[YAmaGNZ]

その条件って
migimigiba.GetComponent<BluebarMove>().rightbarのどれかがtrue
かつ
hidarihidariba.GetComponent<RedbarMove>().leftbarのどれかがtrue
って単純な条件に思えます
9x9のループではなく
9回のループを1つで済むんじゃないですかね？

[退会済みユーザー]

YAmaGNZさん
回答頂いたコードを試したのですが、elseの部分の処理がうまく動作しませんでした。ほかに回答いただいたfiveHundredさんのコードではうまく実行されました。