8

基本上，我想在 foreach 循环内从列表中删除一个项目。我知道这在使用 for 循环时是可能的，但对于其他目的，我想知道这是否可以使用 foreach 循环来实现。

在python中，我们可以通过执行以下操作来实现：

a = [1, 2, 3, 4, 5, 6, 7, 8, 9]

for i in a:
    print i

    if i == 1:
        a.pop(1)

这给出了以下输出

>>>1
3
4
5
6
7
8
9

但是当在 c# 中做类似的事情时，我得到一个 InvalidOperationException，我想知道是否有办法解决这个问题，而不仅仅是使用 for loop。

我在抛出异常时使用的 c# 中的代码：

static void Main(string[] args)
  {
  List<string> MyList = new List<string>(new string[] { "1", "2", "3", "4", "5", "6", "7", "8", "9"});

  foreach (string Item in MyList)
    {
    if (MyList.IndexOf(Item) == 0)
      {
      MyList.RemoveAt(1);
      }

    Console.WriteLine(Item);
    }
  }

提前致谢

4

3 回答 3

26

你不能这样做。从文档中IEnumerator<T>：

只要集合保持不变，枚举数就保持有效。如果对集合进行了更改，例如添加、修改或删除元素，则枚举器将不可恢复地失效，并且其行为未定义。

替代方案是：

  • 建立一个新的要删除的项目列表，然后将它们全部删除
  • 使用普通的“for”循环并确保小心不要重复相同的元素或遗漏任何一个元素。（你说过你不想这样做，但你试图做的只是行不通。）
  • 建立一个只包含你想要保留的元素的新集合

这些替代方案中的最后一个是类似 LINQ 的解决方案，您通常会在其中编写：

var newList = oldList.Where(x => ShouldBeRetained(x)).ToList();

（ShouldBeRetained当然，你想要的逻辑在哪里。）ToList()只有当你真的想要它在列表中时才需要调用。这导致更多的声明性代码通常更易于阅读。我不能轻易猜出你的原始循环是做什么的（目前看起来很奇怪），而如果你可以纯粹根据项目来表达逻辑，它会更清晰。

于 2009-07-14T08:53:09.813 回答
6

如果您只需要删除满足条件的所有项目，您可以使用List<T>.RemoveAll方法：

List<string> MyList = new List<string>(new string[] { "1", "2", "3", "4", "5", "6", "7", "8", "9" });
MyList.RemoveAll(item => item == "1");

请注意，这会修改初始列表。

于 2009-07-14T09:10:59.153 回答
1

在对集合使用 foreach 循环时，您绝对不能以任何方式更改集合。

您可以使用 for 循环并为自己管理索引或制作集合的副本，并在循环原始内容时，从副本中删除与原始内容相同的项目。

在这两种情况下，它都不是那么清晰或方便:)。

于 2009-07-14T08:54:16.893 回答