Unity 2D中SetActive随机激活对象失效问题求助

代码存在的核心问题 #

• 随机索引范围错误：Random.Range(0, _steps.Count - 1) 是左闭右开区间，会导致最后一个Step永远不会被选中，应该改为Random.Range(0, _steps.Count)。
• 索引越界风险：循环变量i遍历的是_steps的长度，却用它去访问_steps[ranI].objects[i]，如果某个Step的objects数组长度小于i，直接触发索引越界异常。
• 逻辑不符合需求：你要的是随机激活1个对象，但当前循环会遍历所有_steps元素，每次都随机选一个Step去激活对应i索引的对象，完全偏离了“随机激活1个对象”的目标。
• 未重置对象状态：没有先将所有对象设为非激活状态，之前激活的对象会保持显示，导致新激活的对象叠加显示。

修复后的代码 #

调整nextStep方法，实现“随机选中一个Step，再从该Step的objects里随机选一个对象激活，同时关闭其他所有对象”的逻辑：

public void AddCount()
{
    Count++;
    if (Count == 2)
    {
        nextStep();
        Count = 0;
    }
}

public void nextStep()
{
    // 先关闭所有Step下的所有对象
    foreach (var step in _steps)
    {
        foreach (var obj in step.objects)
        {
            obj.SetActive(false);
        }
    }

    // 随机选中一个Step
    int randomStepIndex = Random.Range(0, _steps.Count);
    Step targetStep = _steps[randomStepIndex];

    // 从选中的Step里随机选一个对象激活
    if (targetStep.objects.Length > 0)
    {
        int randomObjIndex = Random.Range(0, targetStep.objects.Length);
        targetStep.objects[randomObjIndex].SetActive(true);
    }
}

额外优化建议 #

• 给_steps和Count添加空值/边界判断，比如if (_steps == null || _steps.Count == 0) return;，避免空引用异常。
• 如果需求是“从所有Step的所有objects里随机选一个激活”，可以改成先把所有对象存入一个列表，再随机选：

public void nextStep()
{
    // 收集所有对象
    List<GameObject> allObjects = new List<GameObject>();
    foreach (var step in _steps)
    {
        if (step.objects != null)
        {
            allObjects.AddRange(step.objects);
        }
    }

    if (allObjects.Count == 0) return;

    // 关闭所有对象
    foreach (var obj in allObjects)
    {
        obj.SetActive(false);
    }

    // 随机激活一个
    int randomIndex = Random.Range(0, allObjects.Count);
    allObjects[randomIndex].SetActive(true);
}

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