Adding/Removing components from custom Inspector

Hi

I have a custom inspector that generates sphere colliders, adding them to the same game object where the inspector is attached. In the OnInspectorGUI, in case the colliders data changed, I destroy them all, and create new ones (as sometimes the number of collider changes, etc.).

The cool part is that it looks like it is working perfectly.

The main problem is that Unity complains with an error message (see the full stack bellow). I understand why this happens: unity is cycling through the inspectors, drawing them, and I am changing that list, making it smaller or larger. Thus, indexes out of range occur.

One option is to way for Unity to fix this (but, should this be fixed?)
Another option, and that is the one I am looking into, is to understand how I should mimic this behavior, without hitting this "bug".

Suggestions?

basic code structure here: https://gist.github.com/ambs/8ceefcc0d24321d10c11270dfeac2944

stack----


IndexOutOfRangeException: Array index is out of range.

UnityEditor.InspectorWindow.EditorHasLargeHeader (Int32 editorIndex) (at /home/builduser/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:1267)

UnityEditor.InspectorWindow.DrawEditor (UnityEditor.Editor editor, Int32 editorIndex, Boolean rebuildOptimizedGUIBlock, System.Boolean& showImportedObjectBarNext, UnityEngine.Rect& importedObjectBarRect) (at /home/builduser/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:1074)

UnityEditor.InspectorWindow.DrawEditors (UnityEditor.Editor[] editors) (at /home/builduser/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:1010)

UnityEditor.InspectorWindow.OnGUI () (at /home/builduser/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:361)
System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) (at /Users/builduser/buildslave/mono/build/mcs/class/corlib/System.Reflection/MonoMethod.cs:222)

 

If it is complaining about the size changing could you switch to a list instead and get around the error?

-Alexander

 

The problem is that the code failing is not mine, but from Unity.
Nevertheless, I think I know how to solve this issue. In fact, not, changing completely the approach, so, not really getting around the exception.