游戏变形机制多if判断失效问题及状态机替代方案咨询

核心问题 #

你的代码有三个关键问题导致else if分支无法触发：

1. 逻辑运算符优先级错误：||的优先级低于&&，第一个if的条件会被解析为normalMode == true || ((normalMode == false) && speedMode == true && heavyMode == false)，只要normalMode为true就会进入第一个分支，直接跳过后续判断。
2. 变量名错误：第二个else if里的speedForm == false应该是speedMode == false（你定义的状态变量是speedMode）。
3. 状态互斥性未保证：三个bool变量没有严格互斥，可能出现多个状态同时为true的情况，导致逻辑判断混乱。

修复后的基础版本 #

先修正逻辑和变量错误，保证每个分支的条件准确：

if (Input.GetKeyDown(KeyCode.Space))
{
    // 当前是正常模式 或者 速度模式 → 切换到重型模式
    if (normalMode || (speedMode && !heavyMode))
    {
        normalMode = false;
        speedMode = false;
        heavyMode = true;
        sr.sprite = heavyForm;
        speed = 7f;
        rb.mass = 2f;
    }
    // 当前是重型模式 → 切换到速度模式
    else if (heavyMode && !speedMode && !normalMode)
    {
        normalMode = false;
        speedMode = true;
        heavyMode = false;
        sr.sprite = speedForm;
        speed = 15f;
        rb.mass = 0.5f;
    }
    // 当前是速度模式 → 切回正常模式
    else
    {
        normalMode = true;
        speedMode = false;
        heavyMode = false;
        sr.sprite = normalForm;
        speed = 10f;
        rb.mass = 1f;
    }
}

更简洁的优化方案（不用状态机） #

用枚举代替三个bool变量，彻底避免状态互斥问题，代码可读性和维护性更高：

// 先定义枚举类型（可放在类外部或内部）
public enum PlayerForm { Normal, Heavy, Speed }

// 类内声明状态变量
private PlayerForm currentForm;

// 初始化默认状态
void Start()
{
    currentForm = PlayerForm.Normal;
    sr.sprite = normalForm;
    speed = 10f;
    rb.mass = 1f;
}

// 输入处理逻辑
void Update()
{
    if (Input.GetKeyDown(KeyCode.Space))
    {
        // 循环切换状态：Normal → Heavy → Speed → Normal...
        switch(currentForm)
        {
            case PlayerForm.Normal:
                currentForm = PlayerForm.Heavy;
                sr.sprite = heavyForm;
                speed = 7f;
                rb.mass = 2f;
                break;
            case PlayerForm.Heavy:
                currentForm = PlayerForm.Speed;
                sr.sprite = speedForm;
                speed = 15f;
                rb.mass = 0.5f;
                break;
            case PlayerForm.Speed:
                currentForm = PlayerForm.Normal;
                sr.sprite = normalForm;
                speed = 10f;
                rb.mass = 1f;
                break;
        }
    }
}

额外建议 #

• 枚举方案完全杜绝了多状态同时生效的问题，后续扩展新形态时只需新增枚举值和对应的switch分支即可。
• 如果坚持使用原有的bool变量，务必在切换状态时强制将另外两个状态设为false，确保状态唯一。

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