Robot module/components IEnum->array

[MikeJeffers]

Because every update loop updates every robot, which updates every module on every robotcomponent, the accessor the the underlying collection of modules and components is actually impactful.

Looping with IEnumerable is nice to defer the underlying query it may be built on. But if this is mostly static (for 99.999% of all update calls) then it would be wise to compute this and store it in an array.

The risk is that if the underlying collection (entity children) are manipulated not using the correct interface/public methods then the lazy-initializer (as the resetting mechanism to the collection) might become inconsistent. But this is why we have reviews and testing right? :)