Unity玩家移动脚本偶发飞窜问题排查求助

飞窜问题的原因与修复 #

问题根源 #

你在每个方向按键的逻辑中，重复将rigidbody.velocity.y添加到目标速度的y分量上。当同时按下多个方向键时，y分量会被多次累加（比如同时按W+A，y速度会变成原速度的2倍），最终赋值给刚体后导致玩家突然飞窜。

另外，斜向移动时多个方向向量叠加后，总速度会超过设定的speed（比如W+D时速度为speed*√2），这也会导致移动速度异常。

修复代码 #

using UnityEngine;

public class PlayerMovement : MonoBehaviour
{
    public float speed = 5f;
    [SerializeField] private float jumpForce = 5f;
    private Rigidbody playerRb;
    public CameraController cameraController;

    // 标记是否接触地面
    private bool isGrounded;

    void Start()
    {
        // 自动获取刚体组件，避免手动赋值遗漏
        playerRb = GetComponent<Rigidbody>();
    }

    // 刚体操作放在FixedUpdate，物理计算更稳定
    void FixedUpdate()
    {
        HandleMovement();
        HandleJump();
    }

    void HandleMovement()
    {
        // 使用Input.GetAxis获取平滑的WASD输入
        float horizontal = Input.GetAxis("Horizontal");
        float vertical = Input.GetAxis("Vertical");

        if (horizontal == 0 && vertical == 0)
        {
            // 没有输入时，保持垂直速度，XZ速度归0
            playerRb.velocity = new Vector3(0, playerRb.velocity.y, 0);
            return;
        }

        // 根据相机Y轴旋转计算移动方向
        float yRotationRad = cameraController.yRotation * Mathf.Deg2Rad;
        Vector3 forward = new Vector3(Mathf.Sin(yRotationRad), 0, Mathf.Cos(yRotationRad));
        Vector3 right = new Vector3(Mathf.Sin(yRotationRad + Mathf.PI/2), 0, Mathf.Cos(yRotationRad + Mathf.PI/2));

        // 计算目标移动向量
        Vector3 moveDir = forward * vertical + right * horizontal;
        // 归一化避免斜向速度超标
        moveDir = moveDir.normalized * speed;

        // 保留原有垂直速度，只替换XZ平面速度
        playerRb.velocity = new Vector3(moveDir.x, playerRb.velocity.y, moveDir.z);
    }

    void HandleJump()
    {
        // 用地面检测替代velocity.y == 0的判断
        if (Input.GetKeyDown(KeyCode.Space) && isGrounded)
        {
            playerRb.velocity = new Vector3(playerRb.velocity.x, jumpForce, playerRb.velocity.z);
            isGrounded = false;
        }
    }

    // 通过碰撞检测标记地面状态
    private void OnCollisionStay(Collision collision)
    {
        // 可以给地面标签，这里简化为判断碰撞体是否为静态
        if (collision.gameObject.isStatic)
        {
            isGrounded = true;
        }
    }

    private void OnCollisionExit(Collision collision)
    {
        if (collision.gameObject.isStatic)
        {
            isGrounded = false;
        }
    }
}

其他潜在问题与优化点 #

• Update处理刚体逻辑：原代码在Update中修改刚体速度，Update的执行间隔随帧率变化，会导致物理行为不稳定。改用FixedUpdate（固定时间间隔执行）更符合Unity物理系统的设计。
• 跳跃条件不可靠：rigidbody.velocity.y == 0无法准确判断是否落地，因为刚体碰撞后可能残留微小垂直速度。改用碰撞检测标记地面状态（如上述代码的isGrounded）更可靠。
• 输入处理繁琐：原代码用多个Input.GetKey判断方向，不如Input.GetAxis简洁，还能自带输入平滑效果，提升操作手感。
• 变量命名冲突：原代码将公共变量命名为rigidbody，这是Unity内置的组件属性名，容易引发混淆和潜在Bug，建议改为playerRb这类自定义名称。
• 硬编码参数：jumpHeight直接硬编码为5，改为[SerializeField]或public后，可在Inspector面板实时调整参数，无需修改代码。
• 未初始化组件：原代码依赖手动给rigidbody赋值，添加GetComponent<Rigidbody>()自动获取后，避免遗漏赋值导致的空引用错误。
• 冗余调试日志：Debug.Log(rigidbody.velocity)每帧执行，会严重拖慢运行效率，调试完成后应删除。
• 直接修改velocity的弊端：直接赋值rigidbody.velocity会绕过部分物理约束，复杂场景下可能出现穿透、碰撞异常等问题。如果追求更自然的物理效果，可改用playerRb.AddForce(moveDir, ForceMode.VelocityChange)替代直接赋值。

内容的提问来源于stack exchange，提问作者Jasper Jackson