リザルト画面でボタンを押すと、攻撃SEが再生される

Question

ゴール時にリザルト画面が表示され、「Retry?」「Next Stage」のボタンを押すと、何故かプレイヤーに設定しているattackSEが再生されます。

元のコード(Playerスクリプト)

void FixedUpdate()
    {
        if (!isDown)
        {
            //接地判定を得る
            isGround = ground.IsGround();
            isHead = head.IsGround();

            //各種座標軸の速度を求める
            float xSpeed = GetXSpeed();
            float ySpeed = GetYSpeed();

            //アニメーションを適用
            SetAnimation();

            //移動速度を設定
            Vector2 addVelocity = Vector2.zero;
            if (moveObj != null)
            {
                addVelocity = moveObj.GetVelocity();
            }
            rb.velocity = new Vector2(xSpeed, ySpeed) + addVelocity;
        }
        else
        {
            rb.velocity = new Vector2(0, -gravity);
        }

        if (Input.GetButtonDown("Fire1"))
        {
            anim.SetBool("attack", true);
            anim.Play("player_attack");
            GManager.instance.PlaySE(attackSE);
        }
        else
        {
            anim.SetBool("attack", false);
        }
    }

この状態では、操作上の不具合は無いのですが、先に述べた通り、リザルト画面でボタンを押した時にattackSEが再生されます。

弾による攻撃に関するコードがFixedUpdateの中にあるのが原因ではと考え、取り出してみました。

public void Attack()
    {
        if (Input.GetButtonDown("Fire1"))
        {
            anim.SetBool("attack", true);
            anim.Play("player_attack");
            GManager.instance.PlaySE(attackSE);
        }
        else
        {
            anim.SetBool("attack", false);
        }
    }

この状態では、リザルト画面でボタンを押すと設定してあるSEが再生されるようになりましたが、今度はplayer_attackとattackSEが再生されなくなりました。

どう修正すれば良いのか、教えて頂きたいです。

Answer 1

if (Input.GetButtonDown("Fire1"))がクリックのことだと思うので、それがリザルト画面でのクリック＝攻撃と判定して鳴っているんだと思います。

かんたんなのはクリアしたフラグをとっておいて、それがONのときは処理を回さないようにするとかですかね？
下記例だと、クリアしたらisClearをTrueにしてもらえばUpdate内は通らなくなるので攻撃音もならなくなると思います。

bool isClear;

void FixedUpdate()
    {
        if (isClear) return;

        if (!isDown)
        {
            //接地判定を得る
            isGround = ground.IsGround();
            isHead = head.IsGround();

            //各種座標軸の速度を求める
            float xSpeed = GetXSpeed();
            float ySpeed = GetYSpeed();

            //アニメーションを適用
            SetAnimation();

            //移動速度を設定
            Vector2 addVelocity = Vector2.zero;
            if (moveObj != null)
            {
                addVelocity = moveObj.GetVelocity();
            }
            rb.velocity = new Vector2(xSpeed, ySpeed) + addVelocity;
        }
        else
        {
            rb.velocity = new Vector2(0, -gravity);
        }

        if (Input.GetButtonDown("Fire1"))
        {
            anim.SetBool("attack", true);
            anim.Play("player_attack");
            GManager.instance.PlaySE(attackSE);
        }
        else
        {
            anim.SetBool("attack", false);
        }
    }

Comments

[unity_user_a]

ご回答、ありがとうございます！
弾発射のFire1は、スペースキーに設定してあり、リザルト画面でのボタン操作はクリックなのですが、その様に判定されているんですかね？
クリアしたフラグに関しては、教えて頂いた例のisClearなどをPlayerのコードに追加すれば良いのでしょうか？他のスクリプトなどをいじる必要はないですか？

[hogefugapiyo]

Input.GetButtonDown("Fire1") はデフォルトだと、左Ctrlキーと左マウスクリックじゃなかったでしたっけ？ とりあえずフラグを立ててデバッグしてもらうのが良いかもしれません。
もしくはInput.GetKeyDown(KeyCode.Space) に書き換えて、スペースキーからの入力だけ取るというやり方も考えられます（ただ、どちらにしろリザルト画面でスペース押したら鳴る問題は残りそうですが）

[unity_user_a]

project settings でFire1をspaceに設定してあります。
教えて頂いたコードを追加しただけでは、変化がありませんでした。
しかし、スペースキーからの入力だけを取るやり方にした所、リザルト画面でボタンをクリックしてもattackSEが再生されなくなったため、解決したことに致しました。
確かに、リザルト画面が表示されている状態でも、スペースキーを押せばattackSEが鳴りますが、ゴールしてリザルト画面とボタンが出ている状態で、攻撃ボタンを押すことはあまり無く、大丈夫かと思います。