オブジェクトをGetKeyDownで移動→停止させたい

前提

Unity初心者で2ｄのルーレットゲームを作っています。
Ｃ♯でスクリプトを書いている途中に問題が起こりました。

実現したいこと

画面中央で静止している円が
①スペースキー入力で上方向に運動
②エンターキー入力で停止

発生している問題・エラーメッセージ

エラーメッセージ自体はでていませんが、円が意図した動きにならず困っています。（エンターキーを押しても円が止まらない）

該当のソースコード

ソースコード
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class PlayerController: MonoBehaviour
{
    public Rigidbody2D rigid2D;

    
    
    // Start is called before the first frame update
    void Start()
    {
        Application.targetFrameRate=60;
        this.rigid2D=GetComponent<Rigidbody2D>();
    }

    // Update is called once per frame
    void Update()
    {
        if(Input.GetKeyDown(KeyCode.Space))
        {
            this.rigid2D.velocity=new Vector2(0,5f);
                        
            if(Input.GetKeyDown(KeyCode.Return))
            {
                this.rigid2D.velocity*=0;
            }
        }                        
          
        
    }

行動規範の内容に同意します

回答1件

ベストアンサー

結論から言えば、②の処理が「Spaceを押したとき(KeyDown)」に「Returnを押していたら(KeyDown)」となっていることが問題です。

現状の動きをコードから日本語に起こしてみます。

   void Update()
    {
        if(Spaceキーを押した瞬間)
        { // [A]
            上方向に動かす; // {1}
                        
            if(Returnキーを押した瞬間)
            {
                動きを止める; // {2}
            }
        } // [B]                  
    }

{1}については、恐らく要求に合っています。
ですが、{2}が実行される条件をよく見てください。
if文というのはその条件が正しければ、その1個あとの文を実行します。
この場合「もしSpaceキーを押した瞬間であれば」というif文で実行される文はif文の次の
{ // [A]から} // [B](これを「ブロック文」と言います)になるため、
if(スペースキーを押した瞬間)
の「もしSpaceキーを押した瞬間であれば」
という条件指定は[A]から[B]まで有効です。

つまり{2}が実行される条件は、
「Spaceキーを押した瞬間」に「Returnキーを押した瞬間」です。

では「Spaceキーを押した瞬間」と「Returnキーを押した瞬間」をどう分けたら良いかというと、単純な話で
入れ子にしなければいいだけです。

   void Update()
    {
        if(スペースキーを押した瞬間)
        {
            上方向に動かす;
        }                  
    
        if(Returnキーを押した瞬間)
        {
            動きを止める;
        }
    }

あとはこれをご自分でコードに直してください。

投稿2022/10/30 11:51