Mirror网络框架中集合修改异常的分析与解决方案

问题背景

在使用Mirror网络框架的NetworkRoomManager组件时，开发者在场景切换过程中可能会遇到一个常见的C#异常："InvalidOperationException: Collection was modified; enumeration operation may not execute."。这个异常通常发生在集合被枚举(遍历)的同时又被修改的情况下。

异常原因深度分析

该异常的核心问题在于Mirror框架的NetworkRoomManager.cs文件中，OnServerSceneChanged方法使用了foreach循环遍历pendingPlayers列表，而在循环内部调用的SceneLoadedForPlayer方法中，某些条件下会向同一个pendingPlayers列表添加新元素。

具体来说，当满足以下条件时会出现问题：

1. 当前场景是房间场景(RoomScene)
2. 有新的玩家连接请求处理
3. 系统正在遍历pendingPlayers列表

这种在遍历过程中修改集合的操作违反了C#集合操作的基本原则，导致运行时抛出异常。

技术细节

在C#中，List等集合类型不是线程安全的，也不允许在枚举(如foreach循环)过程中被修改。这是因为：

1. foreach循环使用枚举器(IEnumerator)来遍历集合
2. 枚举器会维护集合的状态快照
3. 如果在枚举过程中集合被修改，枚举器的状态就会不一致
4. 为了防止潜在的数据损坏，C#会主动抛出异常

解决方案

针对这个问题，开发者可以采取以下几种解决方案：

方案一：使用索引循环替代foreach

for (int index = pendingPlayers.Count - 1; index >= 0; index--)
{
    PendingPlayer pending = pendingPlayers[index];
    SceneLoadedForPlayer(pending.conn, pending.roomPlayer);
}

这种反向遍历的方式即使在循环中修改了集合也不会影响遍历过程。

方案二：使用临时集合

var tempList = new List<PendingPlayer>(pendingPlayers);
foreach (var pending in tempList)
{
    SceneLoadedForPlayer(pending.conn, pending.roomPlayer);
}

通过创建集合的副本，可以安全地遍历而不受原集合修改的影响。

方案三：使用线程安全集合

如SafeCollections等专门设计的线程安全集合类型，可以在修改时提供更好的控制和处理机制。

最佳实践建议

1. 在Mirror框架的网络编程中，应特别注意集合操作的线程安全性
2. 避免在foreach循环中修改正在遍历的集合
3. 对于可能被多线程访问的集合，考虑使用锁机制或线程安全集合
4. 在场景切换等关键操作中，添加适当的同步控制

总结

集合修改异常是网络编程中常见的问题，特别是在Mirror这样的实时网络框架中。理解集合操作的原理和限制，选择合适的遍历和修改方式，可以有效避免这类问题的发生。开发者应根据具体场景选择最适合的解决方案，确保网络游戏的稳定运行。