C# activeselfで表示非表示を取得したあと、同じボタンを押して切り替えたい

Question

実現したいこと

スイッチをEキーで押して、ライトをつけたり消したりしたいです。

アニメーションでのやり方がよくわからなかったので、activeselfで分岐を作って、trueならボタンを押したときSetactiveでfalseに、falseならその逆、としたいです。

発生している問題・分からないこと

ついている状態→Eキーを一度押すと消える
(ここまでは想定どおりの挙動)

この後Eキーを押してもライトがつかず消えたままです。

エラーメッセージ

エラーは出ていません。

該当のソースコード

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

public class LightSwitch : MonoBehaviour
{
    public GameObject light;
   
    void Update()
    {

        // もしライトがついていたらEキーを押して消す
        if (light.activeSelf == true)
        {
            if (Input.GetKeyDown(KeyCode.E) )
            {
                // ライトをノン・アクティブ状態にする・・・＞ライトが消える
                light.SetActive(false);
            }

            // ライトがついていない場合
            else
            {
                if (Input.GetKeyDown(KeyCode.E) )
                {
                    // ライトをアクティブ状態にする・・・＞ライトがつく
                    light.SetActive(true);
                }
            }
        }
    }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

activeSelfの分岐文は検索で見つけられたのですが、その後にさらに条件文を入れるかたちのものを見つけられませんでした。

補足

unityのバージョンは2022.3.14です。

Answer 1

if文を入れ子にした際の書き方は

if (条件１) 
{
    // 条件1がtrueの時
    if (条件２)
    {
        // 条件２がtrueの時
    }
}
else
{
    // 条件１がfalseの時
}

となります。

インデントを見れば分かりやすいと思うのですが、提示されたコードは

if (条件１) 
{
    // 条件1がtrueの時
    if (条件２)
    {
        // 条件２がtrueの時
    }
    else
    {
        // 条件２がfalseの時
    }
}

になっています。

単純に

    void Update()
    {
        if (Input.GetKeyDown(KeyCode.E) )
        {
            // ライトのアクティブ状態を反転する
            light.SetActive(!light.activeSelf);
        }
    }

とかでもいけそうですけど

Comments

[petaden12120412]

ありがとうございます！
構造をシンプルに説明してくださって、理解が早かったです。
また「アクティブ状態を反転する」を知らなかったので、よりシンプルにすることができ助かりました。

Answer 2

カッコの対応関係がおかしいです。

コメントを書き換えると以下になります。

        // もしライトがついていたら
        if (light.activeSelf == true)
        {
            // 『Eキーが押されているとき』
            if (Input.GetKeyDown(KeyCode.E) )
            {
                // ライトをノン・アクティブ状態にする・・・＞ライトが消える
                light.SetActive(false);
            }

            // 『Eキーが押されていないとき』（ライトがついていない場合ではなくなっている）
            else
            {
                // 『なおかつ、Eキーが押されているとき』
                if (Input.GetKeyDown(KeyCode.E) )
                {
                    // ライトをアクティブ状態にする・・・＞ライトがつく
                    light.SetActive(true);
                }
            }
        }

if (light.activeSelf == true)に対してelse文を書かないといけないので、正しくは以下になります。

        // もしライトがついていたらEキーを押して消す
        if (light.activeSelf == true)
        {
            if (Input.GetKeyDown(KeyCode.E) )
            {
                // ライトをノン・アクティブ状態にする・・・＞ライトが消える
                light.SetActive(false);
            }
        }
        // ライトがついていない場合
        else
        {
            if (Input.GetKeyDown(KeyCode.E) )
            {
                // ライトをアクティブ状態にする・・・＞ライトがつく
                light.SetActive(true);
            }
        }

Comments

[petaden12120412]

ありがとうございます！
初歩的なミスでお恥ずかしいです。。参考になりました！