アイテムを持っている時にアイテムを取れないようにしているのにもう一回アイテムをとると二回使えてしまう

Question

こんにちは、レースゲームを作っております。
コードです

アイテムフラグを上げるオブジェクト

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

public class Questionblock : MonoBehaviour
{
    private void OnTriggerEnter(Collider other)
    {
        if (other.gameObject.CompareTag("Cart"))
        {
            other.GetComponentInParent<ItemCatch>().getItem = true;
        }
    }
}

カートについているコード

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

public class ItemCatch : MonoBehaviour
{
    public bool getItem;                        // アイテムをゲットしたかのフラグ
    private bool haveItem;                      // アイテムを持っているかのフラグ   
    private bool useItem;                       // アイテムを使ったかのフラグ
    private int selectItem;                     // アイテムを選んだものを入れる
    private GameObject item;
    private new Rigidbody rigidbody;

    [SerializeField] private int itemCount;     // アイテム数
    [SerializeField] private float power;       // キノコパワー

    void Start()
    {
        rigidbody = GetComponent<Rigidbody>();
    }
    void Update()
    {
        ChooseItem();
        UseItems();
    }
    /// <summary>
    /// アイテムを選ぶ
    /// </summary>
    private void ChooseItem()
    {
        if (!getItem) return;
        if (haveItem) return;

        print(getItem);
        print("アイテムセレクト");
        var itemNumber = (int)Random.Range(0, itemCount);
        selectItem = itemNumber;  
        getItem = false;
        useItem = false;
        haveItem = true;
       
    }
    /// <summary>
    /// アイテムを使う
    /// </summary>
    private void UseItems()
    {
        if (!haveItem) return;
        print("アイテム持っている");
        switch (selectItem)
        {
            case 0: SuperMushroom(); break;
        }
    }
    /// <summary>
    /// スーパーキノコ
    /// </summary>
    private void SuperMushroom()
    {
        if (useItem) return;
        if (Input.GetKeyDown(KeyCode.J) && haveItem){
            rigidbody.AddForce(transform.forward * power);
            haveItem = false;
            useItem = true; ;
            print("使った");
        }
    }
}

やった事
上にも書いたのですが、アイテムを持っているときに　アイテムをとると　二回使えてしまうというバグがあります。
getItem でアイテムを選び
haveItem でアイテムを持っているときにはreturn で入らないようにし
それでも足りなかったので　新しいbool を作って、useItem を入れて return で返すようにしているのですが、何故か二回アイテムを使えてしまいます。
アイテム数はもう少し増やす予定なのですが、こういう時は配列やListに入れたほうが楽なのでしょうか？
どうかよろしくお願いします。解説やコメントなどくださるとうれしいです。
フラグなどが苦手なのでの考え方などもご教示お願いします。

Answer 1

回答依頼、ありがとうございます。

見たところ、**「アイテムを持っていたら…」「アイテム用のトリガーに触れたら…」**などの処理をboolでのフラグで頑張って管理しているように見えます。
もっとシンプルに考えるべきかと。

（コード、確認はしましたが手打ちなので誤字脱字あったらすみません）

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

public class Questionblock : MonoBehaviour
{
    private void OnTriggerEnter(Collider other)
    {
        if (other.gameObject.CompareTag("Cart"))
        {
            // other.GetComponentInParent<ItemCatch>().getItem = true;
            // ItemCatchのgetItemフラグ操作ではなく、専用の関数を実行する
            other.GetComponentInParent<ItemCatch>().GetItem();
        }
    }
}

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

public class ItemCatch : MonoBehaviour
{
    // public bool getItem;                        // アイテムをゲットしたかのフラグ
    private bool haveItem;                      // アイテムを持っているかのフラグ   
    // private bool useItem;                       // アイテムを使ったかのフラグ
    private int selectItem;                     // アイテムを選んだものを入れる
    private GameObject item;
    private new Rigidbody rigidbody;

    [SerializeField] private int itemCount;     // アイテム数
    [SerializeField] private float power;       // キノコパワー

    void Start()
    {
        rigidbody = GetComponent<Rigidbody>();
    }
    void Update()
    {
        // Update()で呼び続ける必要はない
        /* ChooseItem();
        UseItems(); */

        // キーの入力とアイテム所持のフラグを常に監視
        if (Input.GetKeyDown(KeyCode.J) && haveItem) {
            UseItem();
        }
    }

    /// <summary>
    /// アイテムをゲットしたとき
    /// 外部から呼ばれるため、アクセス修飾子はpublic
    /// </summary>
    public void GetItem()
    {
        print("アイテムゲット");

        print("アイテムセレクト");
        // めちゃくちゃシンプルな書き方ですが、関数の実行結果を変数に格納しています。
        // 詳しくは「unity 関数 返り値」などで調べて見てください。
        selectItem = ChooseItem();
        
        // アイテム所持フラグをtrueに
        haveItem = true;
        print("アイテム所持: " + haveItem.ToString());
    }

    /// <summary>
    /// アイテムを選ぶ
    /// </summary>
    private int ChooseItem()
    {
        /* if (!getItem) return;
        if (haveItem) return;

        print(getItem);
        print("アイテムセレクト");
        var itemNumber = (int)Random.Range(0, itemCount);
        selectItem = itemNumber;  
        getItem = false;
        useItem = false;
        haveItem = true; */

        // ChooseItem()が呼ばれるのはGetItem()からだけなので、フラグなどで管理しておく必要はない。
        // Update()で管理しようとしていた（常にChooseItem()が呼ばれている）ため挙動も不安定だったのかも？

        var itemNumber = (int)Random.Range(0, itemCount);
        return itemNumber;
        /* 本来は、return (int)Random.Range(0, itemCount); と
            記述できますが分かりやすいようにいったん変数に入れています。 */
    }

    /// <summary>
    /// アイテムを使う
    /// 関数名から s を抜きました
    /// </summary>
    private void UseItem()
    {
        /* if (!haveItem) return;
        print("アイテム持っている");
        switch (selectItem)
        {
            case 0: SuperMushroom(); break;
        }*/

        /* UseItem()も、ifを使っての分岐方法の位置を考えれば
           呼ばれるのはキーを押した瞬間の一度だけなのでフラグ管理は要らない。 */

        print("アイテムを使う");
        switch (selectItem) {
            case 0: SuperMushroom(); break;
        }

        // アイテムを使用したので、所持フラグをfalseに
        haveItem = false;
    }

    /// <summary>
    /// スーパーキノコ
    /// </summary>
    private void SuperMushroom()
    {
        /*if (useItem) return;
        if (Input.GetKeyDown(KeyCode.J) && haveItem){
            rigidbody.AddForce(transform.forward * power);
            haveItem = false;
            useItem = true; ;
            print("使った");
        }*/

        print("スーパーキノコ");
        rigidbody.AddForce(transform.forward * power);
    }
}

Comments

[Um_kok]

お返事遅れてすみません。回答ありがとうございます。
確かにUpdateで回すのは効率が良くないでした。なのでバグが起きていたんでしょうね。
なるほどです。すごくわかりにくいコードを書いていました。原因がわかってすっきりしました。
関数の返り値などあまり使ったことがないので、すごく勉強になりました。
凄くコードが短くなってわかりやすくなりました。ありがとうございます。
困った事があったらまたお願いします。