ジャンプの高さがランダムになってしまう

Question

前提・実現したいこと

spaceキーを押す長さでジャンプの高さを変えようとしています。
しかし、spaceキーを長く押した時でも、
高くジャンプする時と、低くジャンプするする時があったりして、
ジャンプの高さがランダムになってしまいます。

該当のソースコード

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

public class player2d : MonoBehaviour
{
    public float speed;
    public float normalSpeed = 10;
    public float shiftSpeed = 15;
    public float normalJump = 450;
    public float shiftJump = 540;
    private bool jumpUpEnd = false;
    [SerializeField]public float jumpTime;
    private Rigidbody2D rb;
    public float jumpForce;
    public float jumpHeight;
    public float jumpLimitTime;
    bool jump = false;
    Animator animator;
    private ContactFilter2D contact;
    

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

    // Update is called once per frame
    private void Update()
    {
        bool isTouch = rb.IsTouching(contact);
        if (gameObject.transform.position.y < -20)
        {
            int sceneIndex = SceneManager.GetActiveScene().buildIndex;
            SceneManager.LoadScene(sceneIndex);
        }

        if(Input.GetKey(KeyCode.LeftShift) || Input.GetKey(KeyCode.RightShift))
        {
            speed = shiftSpeed;
            jumpForce = shiftJump;
        }
        else
        {
            speed = normalSpeed;
            jumpForce = normalJump;
        }

        if(isTouch)
        {
            jumpTime = 0.0f;
        }
            

        if (Input.GetButtonDown("Jump") && !jumpUpEnd && jumpTime < 0.3f)
        {

            jumpTime += Time.deltaTime;
            rb.velocity = new Vector2(rb.velocity.x, jumpForce * Time.deltaTime);
            jump = true;
        }

        if (Input.GetButtonUp("Jump") && !jumpUpEnd)
        {
            //二回ジャンプできなくする
            jumpUpEnd = true;
        }
    }

    private void OnCollisionEnter2D(Collision2D other)
    {
        if (other.gameObject.CompareTag("Ground"))
        {
            jumpUpEnd = false;
            jump = false;
            animator.SetInteger("CharaState", 0);
        }

        if (other.gameObject.CompareTag("moveground"))
        {
            jumpUpEnd = false;
            jump = false;
            animator.SetInteger("CharaState", 0);
        }

        if (other.gameObject.tag == "moveground")
        {
            transform.SetParent(other.transform);
        }

    }


    private void OnCollisionExit2D(Collision2D other)
    {
        if (other.gameObject.CompareTag("Ground"))
        {
            jump = true;
        }

        if (other.gameObject.CompareTag("moveground"))
        {     
            jump = true;
        }

        if (other.gameObject.tag == "moveground")
        {
            transform.SetParent(null);
        }
    }

    void FixedUpdate()
    {
        float horizontalKey = Input.GetAxis("Horizontal");

        if (Input.GetKey(KeyCode.LeftShift) || Input.GetKey(KeyCode.RightShift))
        {
            speed = shiftSpeed;
            jumpForce = shiftJump;
        }
        else
        {
            speed = normalSpeed;
            jumpForce = normalJump;
        }

        if ((horizontalKey <= 0.1 && horizontalKey >= -0.1 && jump == false) || Input.GetKeyUp(KeyCode.DownArrow) && jump == false || Input.GetKeyUp(KeyCode.UpArrow) && jump == false || Input.GetKeyUp(KeyCode.RightArrow) && jump == false || Input.GetKeyUp(KeyCode.LeftArrow) && jump == false)
        {
            rb.velocity = Vector2.zero;
        }

        if(Input.GetKey(KeyCode.RightArrow) && Input.GetKey(KeyCode.LeftArrow) && jump == false)
        {
            rb.velocity = Vector2.zero;
        }

            if (jump == true)
            {
                animator.SetInteger("CharaState", 3);
            }

            if (horizontalKey != 0 && jump == true)
            {
                animator.SetInteger("CharaState", 3);
            }
            else if (horizontalKey != 0)
            {


                Vector2 liscale = gameObject.transform.localScale;

                if ((liscale.x > 0 && horizontalKey < 0) || (liscale.x < 0 && horizontalKey > 0))
                {
                    liscale.x *= -1;
                    gameObject.transform.localScale = liscale;
                }
                animator.SetInteger("CharaState", 1);
            }
            else if (jump == true)
            {
                animator.SetInteger("CharaState", 3);
            }
            else
            {
                animator.SetInteger("CharaState", 0);
            }

        rb.velocity = new Vector2(horizontalKey * speed, rb.velocity.y);

        if(jump == true)
        {
            rb.velocity = new Vector2(horizontalKey * speed, rb.velocity.y);

        }
        if(rb.velocity.x > 6)
        {
            rb.velocity = new Vector2(6, rb.velocity.y);
        }

        if (rb.velocity.x < -6)
        {
            rb.velocity = new Vector2(-6, rb.velocity.y);
        }

        //Debug.Log(horizontalKey);
    }

    
}

試したこと

インスペクターでjumpTimeを見たところ、
ジャンプをした時に作動する時としない時がありました。

Comments

[m.ts10806]

C
は本件とは無関係と思います

[pokkuru]

すいません。直しておきました

[Y0241-N]

Update内とFixedUpdate内で

if(Input.GetKey(KeyCode.LeftShift) || Input.GetKey(KeyCode.RightShift))
{
speed = shiftSpeed;
jumpForce = shiftJump;
}
else
{
speed = normalSpeed;
jumpForce = normalJump;
}
が重複しているのでまずこれはどちらか一方にしましょう、入力値を取る場合はUpdate内で記述した方が良いのでFixedUpdate内の記述を消した方が良いです。

また、「spaceキーを押す長さでジャンプの高さを変えようとしています」とありますが、
見たところjumpForceが変更されるタイミングがShiftKeyが押されているかいないかの時しか変化がないので、これでは長押し云々でジャンプ力が変わることはないように思います。

Answer 1

GetButtonDown関数はボタンを押した瞬間のプレームの間だけtrueを返します。
1フレームは人間の感覚ではものすごい短い間に過ぎてしまいます。
そのため、その間で押している時間を加算しても欲しい値にならないのでしょう。
ボタンを押し続けていることを確認するには GetButton関数を用います。
これがどのくらいの間trueになっているかを見てジャンプの大きさを決めるというのがこの問題を解決する簡単な方法かと思います。