Unity 死亡時に動けなくさせるプログラムはこれではなぜダメなんですか？

Question

Updateメソッドの中を見てほしいのですが、ここでaliveがTrueの時はHurt,Die,Attack,Jump,Runメソッドを適応するようにしているのでここをreturn falseにしたら死亡時に動けなくなるんじゃないかと思いIf分を使用したのですがvoidだからreturnは返せないよというエラーが出ました。
どうすればいいと思いますか？

まだUnity初めて1週間しかたっていないので簡単な質問でもご了承ください。

using UnityEngine;
using System.Collections;
using UnityEngine.UI;
using System.Collections.Generic;
namespace ClearSky
{
    public class SimplePlayerController : MonoBehaviour
    {
        public float movePower = 10f;
        public float jumpPower = 15f; //Set Gravity Scale in Rigidbody2D Component to 5

        private Rigidbody2D rb;
        private Animator anim;
        Vector3 movement;
        private int direction = 1;
        bool isJumping = false;
        private bool alive = true;


        // Start is called before the first frame update
        void Start()
        {
            rb = GetComponent<Rigidbody2D>();
            anim = GetComponent<Animator>();
        }

        private void Update()
        {
            Restart();
            if (alive)
            {
                Hurt();
                Die();
                Attack();
                Jump();
                Run();
            }else{
                return false;
            }
        }
        private void OnTriggerEnter2D(Collider2D other)
        {
            anim.SetBool("isJump", false);
        }


        void Run()
        {
            Vector3 moveVelocity = Vector3.zero;
            anim.SetBool("isRun", false);


            if (Input.GetAxisRaw("Horizontal") < 0)
            {
                direction = -1;
                moveVelocity = Vector3.left;

                transform.localScale = new Vector3(direction, 1, 1);
                if (!anim.GetBool("isJump"))
                    anim.SetBool("isRun", true);

            }
            if (Input.GetAxisRaw("Horizontal") > 0)
            {
                direction = 1;
                moveVelocity = Vector3.right;

                transform.localScale = new Vector3(direction, 1, 1);
                if (!anim.GetBool("isJump"))
                    anim.SetBool("isRun", true);

            }
            transform.position += moveVelocity * movePower * Time.deltaTime;
        }
        void Jump()
        {
            if ((Input.GetButtonDown("Jump") || Input.GetAxisRaw("Vertical") > 0)
            && !anim.GetBool("isJump"))
            {
                isJumping = true;
                anim.SetBool("isJump", true);
            }
            if (!isJumping)
            {
                return;
            }

            rb.velocity = Vector2.zero;

            Vector2 jumpVelocity = new Vector2(0, jumpPower);
            rb.AddForce(jumpVelocity, ForceMode2D.Impulse);

            isJumping = false;
        }
        void Attack()
        {
            if (Input.GetKeyDown(KeyCode.Alpha1))
            {
                anim.SetTrigger("attack");
            }
        }
        void Hurt()
        {
            if (Input.GetKeyDown(KeyCode.Alpha2))
            {
                anim.SetTrigger("hurt");
                if (direction == 1)
                    rb.AddForce(new Vector2(-5f, 1f), ForceMode2D.Impulse);
                else
                    rb.AddForce(new Vector2(5f, 1f), ForceMode2D.Impulse);
            }
        }
        void Die()
        {
            if (Input.GetKeyDown(KeyCode.Alpha3))
            {
                anim.SetTrigger("die");
                alive = false;
                rb.gravityScale = 0;
                rb.velocity = Vector2.zero;
               

            }
        }
        void Restart()
        {
            if (Input.GetKeyDown(KeyCode.Alpha0))
            {
                anim.SetTrigger("idle");
                alive = true;
            }
        }
        
    }
}

Answer 1

Update関数はUnityのシステムが提供する機能で値を返すことは出来ませんし、仮にできても、(システムがそういう意図で用意していない限り) aliveに自動的に設定もしてくれません。aliveはアプリが定義した変数ですので、その管理はアプリ自身が行わなければなりません。どうするかと言うと死亡したら自身でtrueを設定するだけです。

Comments

[YAmaGNZ]

aliveは生存フラグでfalseで死亡じゃないですか？

[Serbonis]

そうですね間違いました。

[IUYWc1ixztjwmt8]

回答ありがとうございます、Dieメソッドで死んだときalive = false;と定義しているのですが死亡時動いてしまいます。

Answer 2

メソッド内で処理終わりたいならreturnとだけ書けばとりあえずそこまでで終わります。
よくあるのが条件判定を先にもってきて判断する書き方です

void Attack () {
   if(isDead) return; // 死んでたら実行しない
   if(isJump) return; // ジャンプ中は実行しない
   if(isAttack) return; // 攻撃中なら攻撃しない

   // 攻撃処理
}

今回提示コードだととりあえず下記の通りreturnだけにすれば動くと思いますが
このコードあんまり意味ないと思います。aliveでなければelseに入るんですが、そこでreturnしたところでスキップされる処理がありません。そもそもaliveがtrueのときのみ処理走ってるはずなので、aliveがfalseになれば動かなくなるんじゃないでしょうか？

        private void Update()
        {
            Restart();
            if (alive)
            {
                Hurt();
                Die();
                Attack();
                Jump();
                Run();
            }else{
                return;
            }
        }

Comments

[IUYWc1ixztjwmt8]

自分のコードのDieメソッドでaliveがfalseになっているのに死亡時動けてしまうんですよね。

[hogefugapiyo]

private bool alive = true; を public bool alive = true;にするとインスペクターでfalse/trueを確認できるので、まずは死んだときにfalseになっているかを確認してみてください

[IUYWc1ixztjwmt8]

死んだときにはfalseになっていませんでした。
boolの値をfalseにした時は動かなくなりました。

[hogefugapiyo]

① 自分のコードのDieメソッドでaliveがfalseになっているのに死亡時動けてしまうんですよね。
② falseにしたら動かなくなりましたね。

ということは、①が間違っているということになります。aliveがfalseなら動かなくなることが確認できているので、死んだときにaliveがfalseになっていない or すぐtrueに戻っている のでしょう。

[IUYWc1ixztjwmt8]

これもしかしたらなんですが、
3ボタンを押した時にしか発動されなくて、敵と接触した時に死んでも発動されないんじゃないんですか？
```cs
if (Input.GetKeyDown(KeyCode.Alpha3))
```

[IUYWc1ixztjwmt8]

3ボタンを押したときはちゃんと死んだとき何も操作できないようになってました、ありがとうございます。

[hogefugapiyo]

もちろんそうですよ。
動作確認のために自分でこう書かれているんじゃないんですか？

[IUYWc1ixztjwmt8]

いえ、これはAsset Storeのキャラクターのプログラムなので自分で書いたプログラムじゃないんです。すみませんでした。

[hogefugapiyo]

そうなんですね、解決してよかったです。

Answer 3

voidだからreturnは返せないよというエラーが出ました。

voidじゃなくするとか

Answer 4

＞voidだからreturnは返せないよというエラーが出ました
返さなければいいんじゃなですか？

Comments

[IUYWc1ixztjwmt8]

return falseを　falseだけにするってことですか？

[YAmaGNZ]

return false;
を書かなければいいのではないですか？
voidは値を返さないということなのでreturn;と値を返さないでreturnするという方法もあります。
C#の基本的な文法も学習されることをお勧めします。
また、質問する際は自分で解釈したエラー内容ではなくそのまま記載することをお勧めします。

[YAmaGNZ]

というか、ソース見たら他のところでしっかりvoidの関数利用してreturnもしているじゃないですか。
ご自身でどのような意味を持つのか理解されていないということになりますね。
なおのことC#の基本的な文法を学習されるべきかと思います。

[IUYWc1ixztjwmt8]

回答ありがとうございます、すみませんこれはUnityStoreで購入したAssetのコードなんで自分で書いてないんですよね。ですが、return;も学んだはずなのにreturn falseって書いてしまいました。勉強不足でしたすみません。