C# 何度も実行してしまうものを一度だけ実行したい

Question

何度も実行してしまうものを一度だけ実行したいです。

ImageのFillAmountが１以上なら、Daimahouboolをtrueにすると書き込めば、一度だけ実行されると思ったのですがtrueの内容が何度も実行されてしまいます。
1以上なら常に実行されるようになっているために、何度も実行してしまうと思うのですが、1度だけ実行する方法を教えていただきたいです。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.Playables;
using UnityEngine.UI;

public class DaimahouScript : MonoBehaviour
{
    private bool Daimahoubool;

    public PlayableDirector director;
    public Image DaimahouGage;

    public GameObject DaimahouUI;

    public GameObject Book;

    public GameObject MovieBook;
    public GameObject Touroku;

    // Start is called before the first frame update
    void Start()
    {
        DaimahouGage = DaimahouGage.GetComponent<Image>();
        Daimahoubool = false;
    }

    // Update is called once per frame
    void Update()
    {
        if (DaimahouGage.fillAmount >= 1)
        {
            Daimahoubool = true;
        }

        if(Daimahoubool == true)
        {
            Book.GetComponent<bookaction>().enabled = false;
            director.Play();
            Touroku.SetActive(true);
            MovieBook.SetActive(true);
            DaimahouUI.SetActive(false);
        }
    }
   
}

スクリプトを以下のように修正しました。
実行は一度だけになったのですが、DaimahouUIが非アクティブになってくれません。
ImageがDaimahouUIの中にあるからでしょうか？
この場合どうすればよいのか教えていただきたいです。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.Playables;
using UnityEngine.UI;

public class DaimahouScript : MonoBehaviour
{
    private bool Daimahoubool = false;

    public PlayableDirector director;
    public Image DaimahouGage;

    public GameObject DaimahouUI;

    public GameObject Book;

    public GameObject MovieBook;
    public GameObject Touroku;

    // Start is called before the first frame update
    void Start()
    {
        DaimahouGage = DaimahouGage.GetComponent<Image>();
    }

    // Update is called once per frame
    void Update()
    {
        if (DaimahouGage.fillAmount == 1)
        {
            if (!Daimahoubool)
            {
                Daimahoubool = true;

                Book.GetComponent<bookaction>().enabled = false;
                director.Play();
                Touroku.SetActive(true);
                MovieBook.SetActive(true);
                DaimahouUI.SetActive(false);
            }
        }
    }
   
}

また、スクリプトを以下のように変更すると非アクティブ化はするのですが点滅しながら非アクティブになってしまいます。

 void Update()
    {
        if (DaimahouGage.fillAmount == 1)
        {
　　　　　　 DaimahouUI.SetActive(false);
            if (!Daimahoubool)
            {
                Daimahoubool = true;

                Book.GetComponent<bookaction>().enabled = false;
                director.Play();
                Touroku.SetActive(true);
                MovieBook.SetActive(true);
            }
        }
    }

Answer 1

しょーもない回答ですが、気になったので。

まず、命名規則がよくわかりません。

private bool Daimahoubool; とありますが、これはどういう意味でしょうか。

読み方によっては『大魔法を使うことが出来る(状態)』とも読めますし、『大魔法を使った(ため、ゲージが溜まるまで使えない状態)』とも読めます。

今回はそこまで問題はありませんが、たとえば一か月後、デバッグしたり機能を追加したりすると、自分だけで開発する場合でも混乱してしまいます。

なので、AlreadyUsedDaimahou のように『使った(という状態)』なのか CanUseDaimahouのように『使える状態』なのかをはっきりさせた方が無難ですよ。

それと、今回の質問についてですが、ざっと読んだ感じでは、「フラグを折る作業」がありません。

StartメソッドでDaimahouboolをfalseに → Updateメソッドの最初でDaimahouboolをtrueに
→ (Daimahouboolが立っているかどうか) ...

のようになっているようですが、「フラグを折る作業」がStartメソッドの一回限りなので、常にtrueの状態です。

ロジックを追ってみるとわかるはずです。

1. 最初にStartメソッドで初期化。つまりDaimahoubool = falseに。
2. Updateメソッドで「ゲージが溜まったら」系で条件分岐
   2.1. (溜まっていたら)Daimahouboolをtrueに
3. Daimahouboolがtrueなら
   3.1. (= 溜まっていたら) ごにょごにょする
4. (2)に戻って処理する
...

となっています。

ではいつ、Daimahouboolがfalseになるのでしょうか。

文脈的に Daimahoubool は 「現在、大魔法が使える(状態)」をtrueと見なすタイプだと仮定して考えると、多分、「大魔法を放った(後)」になるかなと。

なので、

    // Update is called once per frame
    void Update()
    {
        if (DaimahouGage.fillAmount >= 1)
        {
            Daimahoubool = true;
        }

        if(Daimahoubool == true)
        {
            Book.GetComponent<bookaction>().enabled = false;
            director.Play();
            Touroku.SetActive(true);
            MovieBook.SetActive(true);
            DaimahouUI.SetActive(false);
            
            // <- この辺にDaimahouBoolをfalseにする処理を入れる
        }
    }

とかになるのではないかなと。

Comments

[mikichom_1229]

回答ありがとうございます！
Daimahouboolはバトル内で一度しか使用しないので、boolの名前はこのままでいこうと判断しました。
今後は名前をそのように設定していこうと思います。アドバイスありがとうございます！

Answer 2

すべての動作を一度に行うのではなく、動作を分けることでうまく動作をすることができました。
回答してくださった方々誠にありがとうございました。

public class DaimahouScript : MonoBehaviour
{
    private bool Daimahoubool = false;
    private bool ZukanTourokubool = false;

    public PlayableDirector director;
    public Image DaimahouGage;

    public GameObject DaimahouUI;

    public GameObject Book;

    public GameObject MovieBook;
    public GameObject Touroku;

    public GameObject Daimahoutext;

    public GameObject DeguSpawn;
    public GameObject GageUI;

    // Start is called before the first frame update
    void Start()
    {
        DaimahouGage = DaimahouGage.GetComponent<Image>();
    }

    void Update()
    {
        if (DaimahouGage.fillAmount == 1)
        { 
            if (!Daimahoubool)
            {
                Invoke("Daimahoutextmethod",0.5f);
                Book.GetComponent<bookaction>().enabled = false;
            }
        }

        if (Daimahoutext.activeSelf == true)
        {
            if (!ZukanTourokubool)
            {
                if (Input.GetKeyDown("joystick button 6"))
                {
                    DaimahouUI.SetActive(false);
                    director.Play();
                    Touroku.SetActive(true);
                    MovieBook.SetActive(true);
                }
            }
        }
    }

    void Daimahoutextmethod()
    {
        Daimahoutext.SetActive(true);
    }
   
}

Answer 3

Daimahouboolがfalseの時だけ実行するとしておけば良いかと思いました。

if (!Daimahoubool)
{
    if (DaimahouGage.fillAmount >= 1)
    {
        Daimahoubool = true;
    }
}

が、似たようなコードで実行してうまくいっていないようですね。

実行は一度だけになったのですが、DaimahouUIが非アクティブになってくれません。
下記のその他の部分は実行されているのでしょうか。

Book.GetComponent<bookaction>().enabled = false;
director.Play();
Touroku.SetActive(true);
MovieBook.SetActive(true);

また、

スクリプトを以下のように変更すると非アクティブ化はするのですが点滅しながら非アクティブになってしまいます。
とのことですが、記載のコードと点滅との関連がわかりません。activeと非activeを繰り返しているわけではないですよね。

ImageがDaimahouUIの中にあるからでしょうか？
可能性をつぶすためにもImageを別オブジェクトに対応させてみてはいかがでしょうか。

Comments

[mikichom_1229]

回答ありがとうございます！
アクティブと非アクティブを繰り返し、点滅していました。
DaimahouUIの子オブジェクトにImageを置いている状況です。

また、DaimahouUIの非アクティブ化以外はすべて実行されています。

[YOshim]

>アクティブと非アクティブを繰り返し、点滅していました。
DaimahouUIがアクティブと非アクティブを繰り返しているということでしょうか。
コードを見る限りDaimahouUIをアクティブにするコードはないように見受けられます。
そうであれば、他のコードでDaimahouUIを操作しており、それが影響を与えているのではないでしょうか。

[mikichom_1229]

Book.GetComponent<bookaction>().enabled = false;

この部分で、アクティブ化する部分のスクリプトを非アクティブにしているのですが、それがうまく作用していないのでしょうか。

[YOshim]

Book.GetComponent<bookaction>()とDaimahouUIの関係がわかっていません。

>>アクティブと非アクティブを繰り返し、点滅していました。
>DaimahouUIがアクティブと非アクティブを繰り返しているということでしょうか。
アクティブ・非アクティブを繰り返しているのは何のオブジェクトでしょうか。

[mikichom_1229]

何度もご回答誠にありがとうございます。
大変参考になりました。
無事に解決いたしました！

Answer 4

DaimahouGage.fillAmountが1なら条件が正となるので
どうやってこのゲージが増えているのか分かりませんが、値を1以下にすればよいかと思います。

        if (DaimahouGage.fillAmount >= 1)
        {
            Daimahoubool = true;
            DaimahouGage.fillAmount = 0;
        }

Comments

[mikichom_1229]

回答ありがとうございます。
その方法ですと、やはり何度も実行されてしまいます。