Unity 3D オブジェクト リスト 表示/非表示 一括切り替え SetActve

Question

依然としてUnity 3Dでゲームを開発しています。

UIのボタンを押して、複数のオブジェクトの表示/非表示を一括で切り替えたいです。

シーン上には非表示のオブジェクトと表示しているオブジェクトが混在していて、
ボタンを押すと非表示のオブジェクト→表示に、表示中のオブジェクトは非表示にしたいです。

ボタンはゲーム中何回も押すことができ、そのたびに表示/非表示を交互に入れ替えたいです。

表示/非表示にはenable、SetActiveなどいろいろありますが、今回はSetActiveで行きたいです。

以下のスクリプトを試しましたが、一度非表示にしてしまったものが、再度ボタンを押してもうまくいきません。

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

public class SwitchBotton : MonoBehaviour
{
    public GameObject[] targetObjects;



    // Start is called before the first frame update
    void Start()
    {

    }

    // Update is called once per frame
    void Update()
    {

    }

    public void OnClick()
    {
        foreach (GameObject targetObject in targetObjects)
        {
            if (targetObject.activeSelf == false)
            {
                targetObject.SetActive(true);
            }

            if (targetObject.activeSelf == true)
            {
                targetObject.SetActive(false);
            }
        }
    }
}

なお、対象のオブジェクトを二つに絞った場合、またボタンを一回だけ押せる場合には以下のスクリプトでうまくいきました。

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

public class SwitchButton : MonoBehaviour
{
    public GameObject targetObject1;
    public GameObject targetObject2;



    // Start is called before the first frame update
    void Start()
    {

    }

    // Update is called once per frame
    void Update()
    {

    }

    public void OnClick()
    {
        targetObject1.SetActive(false);
        targetObject2.SetActive(true);
    }
}

アドバイスよろしくお願い致します。

Answer 1

foreach内のif文が別れてしまってるので、

●最初のif → falseのものをtrueに変える。
●次のif → 最初のifでtrueにしたものをfalseに変える。

となってしまってます。

elseで繋げば、多分うまくいくと思います。

public void OnClick()
    {
        foreach (GameObject targetObject in targetObjects)
        {
            if (targetObject.activeSelf == false)
            {
                targetObject.SetActive(true);
            }else{
                targetObject.SetActive(false);
            }
        }
    }

Comments

[911RSR]

シンプルかつ的確、スピーディな訂正を誠にありがとうございます！
自分で無限ループに陥っていたのですね！ありがとうございます。