Unity碰撞检测触发协程的正确启动位置及无法结束问题

核心问题定位 #

协程无法正常结束是3个逻辑错误共同导致的，和协程调用位置直接相关：

1. 协程宿主被提前销毁
   你调用StartCoroutine()的主体是挂载碰撞检测脚本的物体（通常是技能投射物、范围触发物），而非被施加Debuff的玩家对象。Unity协程的生命周期完全绑定在调用StartCoroutine的MonoBehaviour所属物体上，你启动协程后立刻执行Destroy(gameObject)销毁当前物体，协程会随宿主被强制终止，根本不会执行到5秒后的属性恢复逻辑。
   注意：哪怕传入的IEnumerator是PlayerController类的方法，只要StartCoroutine在其他物体的脚本上调用，协程就属于调用方物体，和方法所属对象无关
2. 碰撞检测无触发限制，重复启动协程
   你把碰撞检测放在Update中每帧执行，只要碰撞体重叠就会反复启动新的减速协程，既会造成性能浪费，也会导致多个协程反复修改玩家速度值，最终数值逻辑完全混乱，表现为协程永远无法正常结束。
3. 空值校验缺失
   你调用c.TryGetComponent后没有判断返回值，如果碰撞到的物体未挂载PlayerController组件，后续直接访问player会触发空引用错误，直接中断逻辑执行。

修复步骤 #

1. 修正协程调用主体，把协程绑定到玩家对象上，保证投射物销毁后协程能继续执行完
2. 增加一次性命中标记，避免重复触发碰撞检测、重复启动协程
3. 补全空值判断，避免空引用报错
4. （可选优化）简化协程逻辑，去掉无意义的每帧重复赋值

修复后代码 #

首先给碰撞检测脚本增加命中标记：

// 新增：标记是否已经触发过碰撞，避免重复执行
private bool hasHit = false;
private void Update()
{
    if(!hasHit)
    {
        CheckForCollision();
    }
}

修改碰撞检测逻辑：

private void CheckForCollision()
{
    Collider[] colliders = Physics.OverlapSphere(position, scale, layerMask);

    foreach (Collider c in colliders)
    {
        // 新增：校验组件获取结果，空引用直接跳过
        if (c.TryGetComponent<PlayerController>(out PlayerController player))
        {
            // 修复：在player对象上启动协程，生命周期绑定玩家而非当前投射物
            player.StartCoroutine(player.ApplyDebuff(player.speed, 30, 5, 
                debuffValue => player.speed = debuffValue, 
                originalValue => player.speed = originalValue));
            player.TakeDamage(50);
        }
    }

    // 注：如果是单体命中，处理完第一个目标就加break跳出循环；如果是AOE范围伤害，遍历完所有目标再走下面逻辑
    hasHit = true;
    Destroy(gameObject);
}

优化后的协程逻辑（固定值减速不需要每帧重复赋值，减少无意义计算）：

public IEnumerator ApplyDebuff(float stat, float percentage, float duration, 
System.Action<float> OnDebuffStart, System.Action<float> OnDebuffEnd)
{
    float originalValue = stat;
    float debuffValue = originalValue - (percentage / 100 * originalValue);
    OnDebuffStart(debuffValue); // 进入协程时施加一次减速即可
    
    yield return new WaitForSeconds(duration); // 直接等待对应时长，不需要每帧循环判断
    
    OnDebuffEnd(originalValue);
}

后续扩展建议 #

如果需要实现多Debuff叠加效果，不要用协程直接修改属性值，建议给PlayerController增加Debuff管理列表：每次施加Debuff时把效果参数存入列表，实时计算最终属性值；Debuff时间到后从列表移除对应效果再重算属性，避免多个协程同时改值导致的数值错乱。

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