Unityで複数のPlayerを同じスクリプトで制御したい

Question

Unityで複数のPlayerを同じスクリプトで制御したい

マウスのみで遊べる2Dゲームを制作しているのですが
①カーソルが重なっていると、どの位置から弾が発射されるかのSpriteを描画
②クリックされると拡大され且つ弾発射準備
③クリックが離されると弾を発射
④弾発射後再度撃てるようになるまでのクールタイム

という処理をするオブジェクト(Player)をシーン上に
4つ置いているのですが、1つのオブジェクトが弾発射後クールタイムに入っている時に
残りの3つのオブジェクトにカーソルが触れると弾を発射していないのに
クールタイムに入ってしまいます。
弾を発射したオブジェクトにのみクールタイムに入ってほしいのです
クールタイムのフラグを共有しているから
上記のようになってしまうと思うのですが
どのようにしたら共有しないようにできるでしょうか
4つのオブジェクトにPlayerMouseSelectをアタッチしています。

よろしくお願いします。

該当のソースコード

public class PlayerMouseSelect : MonoBehaviour
{
    [SerializeField]
    private GameObject clickEffect, shotEffect;

    [SerializeField]
    public GameObject mouseObj;
    private PlayerSelectManager psMg;
    private GameObject target;
    private GameObject child;
    private int enterTimer;
    private int clickTimer;

    private Vector3 scale;
    private Vector3 normalScale;                                //元の大きさを保存

    [SerializeField]
    private int coolTime;                                       //弾発射のクールタイム
    private float coolTimer;

    private AudioSource audioSource;
    public List<AudioClip> audioClip = new List<AudioClip>();   //SE

    // Start is called before the first frame update
    void Start()
    {
        audioSource = gameObject.AddComponent<AudioSource>();

        psMg = mouseObj.GetComponent<PlayerSelectManager>();
        enterTimer = 0;
        clickTimer = 0;

        scale = transform.localScale;
        normalScale = scale;

        coolTimer = 0;
    }

    // Update is called once per frame
    void Update()
    {
        target = psMg.GetTargetObj();

        if (!target) return;

        MouseEnter();
        MouseDown();
        MouseUp();
        MouseExit();
        CoolTime();
    }

    //カーソルが重なった時
    void MouseEnter()
    {
        if (psMg.GetCoolFlg()) return;
        if (psMg.GetPlayerHit() && enterTimer == 0)
        {
            enterTimer = 1;

            //拡大
            ExpansionScale(0.15f, 0.15f);

            //SE
            audioSource.PlayOneShot(audioClip[0]);

            //十字のレーザーポインタ描画
            psMg.LaserPointer(true, false);
        }

    }

    //クリック時
    void MouseDown()
    {
        if (psMg.GetCoolFlg()) return;
        if (psMg.GetPlayerClick() && clickTimer == 0)
        {
            clickTimer = 1;

            //クリック時のエフェクト生成
            Instantiate(clickEffect, target.transform.position, target.transform.rotation);
            audioSource.PlayOneShot(audioClip[1]);

            //拡大
            ExpansionScale(0.5f, 0.5f);

            //色を水色
            psMg.ColorChange(0.0f, 0.9f, 1.0f, 1.0f);
            //十字のコライダーをtrue
            psMg.LaserPointer(true, true);
        }
    }

    //クリック離した時
    void MouseUp()
    {
        if (psMg.GetCoolFlg()) return;

        if (psMg.GetPlayerShot())
        {
            clickTimer = 0;

            psMg.SetPlayerClick(false);
            psMg.SetCoolFlg(true);
            //ショット時のエフェクト生成
            Instantiate(shotEffect, target.transform.position, target.transform.rotation);
            audioSource.PlayOneShot(audioClip[2]);

            //元の色に戻す(白)
            psMg.ColorChange(1.0f, 1.0f, 1.0f, 1.0f);

            //十字のレーザーポインタ描画しない
            //十字のコライダーをfalse
            psMg.LaserPointer(false, false);

            //拡大していたのを元の大きさに戻す
            NormalScale();
        }
    }

    //カーソルが離れた時
    void MouseExit()
    {
        if (psMg.GetCoolFlg()) return;
        if (!psMg.GetPlayerHit())
        {
            enterTimer = 0;
            //十字のレーザーポインタ描画しない
            //十字のコライダーをfalse
            psMg.LaserPointer(false, false);

            //拡大していたのを元の大きさに戻す
            NormalScale();
        }
    }

    //クールタイム
    void CoolTime()
    {
        if (!psMg.GetCoolFlg())
        {
            coolTimer = 0;
            return;
        }
        psMg.SetPlayeShot(false);
        if (psMg.GetCoolFlg())
        {
            psMg.ColorChange(1.0f, 1.0f, 1.0f, 0.5f);
            //NormalScale();
            psMg.LaserPointer(false, false);
            coolTimer += Time.deltaTime;
            if (coolTimer >= coolTime)
            {
                psMg.ColorChange(1.0f, 1.0f, 1.0f, 1.0f);
                psMg.SetCoolFlg(false);
            }
        }

    }

    //targetを拡大
    void ExpansionScale(float addX, float addY)
    {
        scale.x += addX;
        scale.y += addY;
        target.transform.localScale = scale;
    }

    //元の大きさ
    void NormalScale()
    {
        scale.x = normalScale.x;
        scale.y = normalScale.y;
        target.transform.localScale = scale;
    }

}

public class PlayerSelectManager : MonoBehaviour
{
    private float maxDistance;            //Rayの長さ
    private int layerMask;                //1~7番目のレイヤーを無視
    private bool playerHit;               //Playerに触れている
    private bool beforShot;               //クリック時
    private bool shot;                    //クリック離すと弾を発射
    private bool coolFlg;                 //弾発射のクールタイム

    //選択されているPlayer
    private GameObject targetObj;
    public GameObject GetTargetObj() { return targetObj; }
    public void SetTargetObj(GameObject obj) { targetObj = obj; }

    //マウスカーソルが当たっているかどうか
    public bool GetPlayerHit() { return playerHit; }
    public void SetPlayerHit(bool hit) { playerHit = hit; }

    //左クリック押下
    public bool GetPlayerClick() { return beforShot; }
    public void SetPlayerClick(bool click) { beforShot = click; }

    //左クリック離した
    public bool GetPlayerShot() { return shot; }
    public void SetPlayeShot(bool shotF) { shot = shotF; }

    //クールタイム
    public bool GetCoolFlg() { return coolFlg; }
    public void SetCoolFlg(bool flg) { coolFlg = flg; }

    //targetObjの色を変更
    public void ColorChange(float r, float g, float b, float a)
    { targetObj.GetComponent<SpriteRenderer>().color = new Color(r, g, b, a); }

    //レーザーポインターの描画、コライダー
    public void LaserPointer(bool spriteFlg, bool coliderFlg)
    {
        //targetObjの子供
        foreach (Transform childObj in targetObj.transform)
        {
            childObj.GetComponent<SpriteRenderer>().enabled = spriteFlg;

            //targetObjの孫
            foreach (Transform grandChildObj in childObj)
            {
                grandChildObj.GetComponent<BoxCollider2D>().enabled = coliderFlg;
            }
        }
    }

    // Start is called before the first frame update
    void Start()
    {
        maxDistance = 1.0f;
        layerMask = 1 << 8;
        playerHit = false;
        beforShot = false;
        shot = false;
        coolFlg = false;
    }

    // Update is called once per frame
    void Update()
    {
        //メインカメラ上のマウスカーソルのある位置からRayを飛ばす
        Ray ray = Camera.main.ScreenPointToRay(Input.mousePosition);

        RaycastHit2D hitObj = Physics2D.Raycast((Vector2)ray.origin, (Vector2)ray.direction, maxDistance, layerMask);

        //if (GetCoolFlg()) return;
        //当たったもの(hitObj)がPlayeなら
        // hitObjを少し拡大、レーザーポインター描画(hitObjの子供)
        if (hitObj.collider != null)
        {
            if (hitObj.collider.gameObject.layer == LayerMask.NameToLayer("Player"))
            {
                SetTargetObj(hitObj.collider.gameObject);
                SetPlayerHit(true);
            }
        }
        else
        {
            //当たっていなかったら元の大きさに、レーザーポインターfalse
            SetPlayerHit(false);
        }

        //クリックされたらhitObjさらに拡大、色を水色、レーザーポインターの当たり判定true
        if (GetPlayerHit())
        {
            if (Input.GetMouseButton(0))
            {
                SetPlayerClick(true);
            }
        }
        else if (Input.GetMouseButton(0))
        {
            if (GetPlayerHit())
            {
                SetPlayerClick(true);
            }
        }

        //クリック離すと弾が発射、hitObj元の大きさに、レーザーポインターfalse
        if (GetPlayerClick())
        {
            if (Input.GetMouseButtonUp(0))
            {
                SetPlayeShot(true);
            }
        }

    }

}

試したこと

Playerごとにスクリプトを作成すれば動くが
同じスクリプトで制御したい。

OnMouseEnter()やOnMouseDown()等を使用したが
判定が反応しなくなったりする時があったので使用をやめた。

Comments

[退会済みユーザー]

あまりC#な書き方でない気がします…読みにくいです。

PlayerMouseSelectのUpdate内に追加、
if (!target) return;
次の行に
if (!target != this.gameObject) return;
こうするとどうなりますか？

[KeyCase]

コメントありがとうございます。

if (!target != this.gameObject) return;
を追加すると、PlayerMouseSelect内の
各関数が呼ばれなくなしました。

Answer 1

フラグ関連をPlayerMouseSelectで
管理することにより解決しました。

Answer 2

質問から気になっている部分通り、coolFlgをpsMgが持っているため、
MouseUp()から一度psMg.SetCoolFlag(true)されてしまうと
Enter、Upなどのメソッドで
if(psMg.GetCoolFlg()) return;となっていますので、以降の処理は打ち切ってしまいます。
原因はこの処理で間違いないかと思いますが他の問題も結構ありそうです。

コードを試した訳ではないので憶測ですが、
例えば、PlayerMouseSelectのUpdate内でif(!target) return;となっていますのでtargetを取得出来ていなければcoolTimeも走りません。

MouseEnter,Down,Up,Exit,CoolTimeと実行しますが、psMgの状態targetObjはrayのHitで変更される。という部分を見るに
Enter→Down→ここで別オブジェクトにドラッグ→Upなどでのバグも発生しそうな気がします。

個人的にcoolTimeが何を指しているか？がちょっと分かりませんでした。
Playerの操作を一定以上受け付けないようになのか、
選択されたオブジェクトは一定時間後にならないと弾を発射出来ないという意味なのか？
感覚的には下記ですが、コードを見ると上記を作りたいように見えます、、、

共有しない方法となってますが、psMgは一つ、PlayerMouseSelectが各オブジェクトごと、となっている様ですので個々の情報はPlayerMouseSelect側で管理するか。psMgに配列など用意して個々を判断するか？でしょうか。

正直、psMgの処理はEventSystemsを利用した方が楽な気がしますが、使う使わないは個人の自由だと思いますので。