RevitAPI:不要在遍历(Iterate)过滤出来的收集器(FilteredElementCollector)时修改文档(Document)

有客户报了一个问题，说用API加载族（Familly）的时候，如果这个文件是一个新的文件，那不会出现问题，但是如果是升级文件，Revit就会崩溃。

他的代码大概是这样的：

UIApplication rvtApp = commandData.Application;
UIDocument rvtDoc = rvtApp.ActiveUIDocument;

FilteredElementCollector collector =
    new FilteredElementCollector(rvtDoc.Document)
    .OfClass(typeof(Family));
FilteredElementIterator itr =
    collector.GetElementIterator();
while (itr.MoveNext())
{
    Element elem = (Element)itr.Current;
    ReloadFamily(rvtApp, rvtDoc, elem);
}

首先过滤出来所有的族Family，然后去本地路径里面搜索对应于这个族的.rfa文件，然后调用Document.LoadFamily来Load这个.rfa文件。

我试着在每次调用ReloadFamily之前，输出对应的elem的Id和名字，

while (itr.MoveNext())
{
    Element elem = (Element)itr.Current;
    WriteLog(elem.Id + ":" + elem.Name);
    ReloadFamily(rvtApp, rvtDoc, elem);
}

输出之后发现，有的Id会出现两次，而且出现两次之后，Revit就崩溃了。

那么一定是这个“两次”造成的问题。

思考是不是collector里面的元素有重复呢？重写代码把所有的elem的id都打印出来，发现并没有重复。

又试了另外一种办法，不是用Iterator，而是使用foreach，

foreach (var elem in collector.ToElements())
{
    ReloadFamily(rvtApp, rvtDoc, elem);
}

发现Revit不会崩溃，程序可以正常跑通了。

聪明的读者一定知道foreach一定不会对结果产生影响，而是这里的ToElements()方法，该方法创建了一个新的集合。

原因已经浮现，那就是我们在遍历collector的同时，对Document进行了修改，正是这个修改造成了Revit的崩溃。

举个简单的例子，看下面的代码：

List<int> ids = new List<int>() { 1, 2, 3, 4 };
foreach (int id in ids)
{
    ids.Add(5); //Exception!!!
}

当我们遍历ids的时候，对ids本身进行添加或者删除元素的操作会抛异常：

System.InvalidOperationException: 集合已修改；可能无法执行枚举操作。

同样的道理，对Revit文档进行遍历操作的时候，增加或删除Revit的元素，也会造成相同的问题。

所以，以后遍历文档的时候，大家需要注意哦。