Unity c#中嵌套循环中的StackOverflowException

Question

在 Unity c# 中，我从 json 获取数据，并使用嵌套循环（如下代码）查找/比较游戏对象以在本地游戏对象中设置该数据。但是在达到一定数量的对象（> 275）后，我得到了 StackOverflowException 错误。基本上 80 次循环运行 10 次，运行 300 次，请参见下面的代码。（注意：此代码在 j

public void SetDataInObject()
    {
        if (j < ObjList.objects.Count)  //count is approx 300
        {
            for (int k = 0; k < allObjs.Length; k++)  // Length is 10
            {                
                stCalc = uiMana.floorStats[k].GetComponent<StatesCalc>();

                for (int m = 0; m < allObjs[k].floorObjs.Length; m++)  //Length is 80
                {
                    string serverObj = ObjList.objects[j].name;
                    string localObj = allObjs[k].floorObjs[m].gameObject.name;

                    if (localObj == serverObj)
                    {
                        ObjManager curObjManger = allObjs[k].floorObjs[m].GetComponent<ObjManager>();
                        //Logic to set data in local objects
                    }
                }                
            }
            j++;
            SetDataInObject();
        }
    }

Answer 1

您的内部块运行大约 300 * 10 * 80 = 240000 次递归，多次存储一些值，并且还需要一些更长的解析才能了解那里做了什么。

实际上，我看不到您可以在方法中更改 ObjList.objects 的任何地方.. 所以反对简单地使用没有任何递归和“魔术”索引字段的闭环：

public void SetDataInObjects()
{
    for (var j = 0; j < ObjList.objects.Count; j++)
    {
        // This actually stays the same for all inner iterations until j is changed
        // so keep it around as long as possible to save resources
        var serverObj = ObjList.objects[j].name;

        for (int k = 0; k < allObjs.Length; k++)  // Length is 10
        {                
            stCalc = uiMana.floorStats[k].GetComponent<StatesCalc>();
            // Also this stays the same until k is changed
            // so keep the reference around to save access calls
            var currentObj = allObjs[k];

            for (int m = 0; m < currentObj.floorObjs.Length; m++)  //Length is 80
            {
                var localObj = currentObj.floorObjs[m].gameObject.name;

                if (localObj == serverObj)
                {
                    ObjManager curObjManger = allObjs[k].floorObjs[m].GetComponent<ObjManager>();
                    //Logic to set data in local objects
                }
            }                
        }
    }
}

您存储的变量是简单的类型，例如 int、string 和 Component 引用 .. GC 不应该关心太复杂的事情。

实际上，我认为完全不使用 j 和 m 的索引会更好地解决这个问题

public void SetDataInObjects()
{
    foreach(var serverObj in ObjList.objects)
    {
        var serverName = serverObj.name;

        for (int k = 0; k < allObjs.Length; k++)  // Length is 10
        {                
            stCalc = uiMana.floorStats[k].GetComponent<StatesCalc>();
            // Also this stays the same until k is changed
            // so keep the reference around to save access calls
            var currentObj = allObjs[k];

            foreach (var floorObj in currentObj.floorObjs)  //Length is 80
            {
                if (localObjName.Equals(serverName))
                {
                    var curObjManger = floorObj.GetComponent<ObjManager>();
                    //Logic to set data in local objects
                }
            }                
        }
    }
}

这显然仍将运行内部块 300 * 10 * 80 = 240000 次，但使用在每次相应迭代后清除的局部变量，因此它们不会增加分配内存。

请注意，仍然重复调用GetComponent 非常昂贵，如果可能的话，您应该事先存储引用。

例如通过将floorStats 的类型直接设为StatesCalc[] 并将floorObjs 的类型设为ObjectManager[]，假设您通过Inspector 引用它们，它将自动存储相应的类型引用，因此运行时不需要GetComponent。