System.Collections.IEnumerator 对象的意外行为

Question

我想删除集合中的所有元素，因为我没有看到 .RemoveAll 或类似的东西，所以我使用下面的代码。

int number = addressBook.Items.Count;
System.Collections.IEnumerator enumerator = ...
while (enumerator.MoveNext())
{
  target = enumerator.Current as Outlook.ContactItem;
  target.Delete();
}

但是，我注意到集合中剩余元素的数量大约是程序每次运行的一半。我的结论是.Delete() 自己跳到下一个元素，这意味着循环条件中的.MoveNext() 跳到下一个元素之后。

所以我尝试如下重置枚举器。

int number = addressBook.Items.Count;
System.Collections.IEnumerator enumerator = ...
while (enumerator.MoveNext())
{
  target = enumerator.Current as Outlook.ContactItem;
  target.Delete();
  enumerator.Reset();
}

但是，当我检查.Count 时，我看到在删除最后一个元素并重置枚举器之后，剩余的元素数仍然为 1。当然，我遇到了一个例外。

我在这张照片中遗漏了什么？我知道这不是错误，因为 会在很久以前报告并解决...

Answer 1

IEnumerator 只能在基础集合保持不变的情况下使用。一旦它发生变化（例如通过从中删除元素），枚举器就会不可恢复地失效。

您不能在修改集合时使用一个枚举器来浏览集合，而 Reset 也无济于事。您只需要每次都创建一个新的枚举器。例如。类似的东西可能会起作用：

while (addressBook.Items.Count > 0)
{
    foreach(Outlook.ContactItem target in ...)
    {
        target.Delete();
        break; // <-- !
    }
}

Answer 2

两件事。

1. 只要在删除每个项目后重新实例化它，就应该能够使用枚举器删除所有项目。使用 LINQ，它应该如下所示：

   while (addressBook.Items.Count > 0)
   {
       addressBook.Items
           .OfType<Outlook.ContactItem>()
           .Last()
           .Delete();
   }
   

2. 根据 MSDN，您实际上必须删除 从最后一项开始的项目：http://msdn.microsoft.com/en-us/library/microsoft.office.interop.outlook._contactitem.delete.aspx

Answer 3

见Marc Gravell's Answer。

重置是多余的；如此之多，以至于它是在 迭代器块的语言规范以在重置时引发异常。这 正确的做法是简单地处理和释放旧的迭代器， 并再次调用 GetEnumerator。或者更好：避免阅读 两次，因为并非所有数据都是可重复的。

参考以下链接：

Can't add/remove items from a collection while foreach is iterating over it
When IEnumerator.Reset() method is called?

using IEnumerator to removing items

希望对您有所帮助..

Answer 4

我希望使用Enumerable.Reverse 扩展会是对while...Last().Delete() 解决方案的可喜改进。这可能是这样的：

foreach(var target in addressBook.Items
    .OfType<Outlook.ContactItem>()
    .Reverse())
{
    target.Delete();
}

我不经常使用Reverse；事实上，我不记得曾经在生产代码中使用过它，但我也不必与 Outlook 进行互操作。这里的好处是您不会像while...Last().Delete() 方法那样在每次删除项目后重复遍历整个列表剩余部分。当项目列表很小时，性能可能没有太大（任何？）差异，但是如果遍历很昂贵（我想这是因为这可能归结为引擎盖下的 COM iterop 调用）或者如果列表是大，那么性能差异可能很大。唯一确定的方法是测试/配置文件。