C＃为什么List.Remove（）打破'for循环'？

Question

我真的不知道为什么for循环只循环一次。我认为逻辑是正确的，下面是代码。

// the list is named 'dataList' 

for(int i = 0 ; i < dataList.Count; i ++) 
{ 
    string[] data = dataList[i].Split('+'); 
    string[] wsno = data[0].Split(':'); 
    if(wsno[1].Equals(tbWorkSheet.Text)) 
    { 
     dataList.Remove(dataList[i]) <<<< remove string that has the same number 
     //data removed 
     //for loop ends up here idk why.. 
    } 
} 

dataList.Count将是列表的大小。

这种情况是这样的.. 我想删除存储在列表中的多个字符串，每个字符串都有它的组号。 因此，for循环将循环直到最后一个。但是当它找到一个与所需数字相同的字符串时，它将执行将其删除的if语句。

Answer 1

这是非常糟糕的主意。通过从列表中删除值，您将跳过列表中的下一个项目。

例如您的列表中有3项在索引0，1，2

第一次迭代中：i = 0 您在索引中删除0项列表现在有索引0项，1（拳头项目删除）。

第二次迭代：i = 1. 注意位于索引1处的项目是如何移动到0的。而我们的索引计数器从0变为1.所以我们将“跳过”这个项目。

如果您的列表中只有2个项目，他将在第一次删除后中断。这是因为您的指数增加时计数减少。

通过逆向排序，您将消除此问题。

List<int> list = new List<int> { 2, 1 }; 
for (int i = list.Count - 1; i >= 0; i--) 
{ 
    list.RemoveAt(i); 
} 

编辑：Rawling提到，您可以前进，如果你减少您柜台当您删除的项目。

for (int i = 0; i < list.Count; i++) 
{ 
    list.RemoveAt(i--); 
} 

Answer 2

您不能在for循环中更改列表。 您可以将要删除的数据保存在另一个列表中，然后在退出循环后将其删除。

Answer 3

删除项目后，您忘记了递减i。这会跳过下一个项目，因为所有后续项目将被重新编号减1。

Answer 4

循环向后通过列表：

for(int i = dataList.Count; i >= 0 ; i--) 
{ 
    string[] data = dataList[i].Split('+'); 
    string[] wsno = data[0].Split(':'); 
    if(wsno[1].Equals(tbWorkSheet.Text)) 
    { 
     dataList.RemoveAt(dataList[i]) <<<< remove string that has the same number 
     //data removed 
     //for loop ends up here idk why.. 
    } 
} 

Answer 5

您可以使用List.RemoveAll泛型方法。方法从列表中删除与谓词匹配的所有项目。 方法签名是以下：

INT List.RemoveAll（谓词匹配）

它返回被删除的元素的数量。

请参阅MSDN：http://msdn.microsoft.com/en-US/library/wdka673a(v=vs.110).aspx

Answer 6

我宁愿做一个新的列表，而不是修改原始集合。

能整齐地使用LINQ来完成：

dataList = 
    dataList 
     .Select(x => new{ 
      dataItem = x, 
      secondWsno = x.Split('+').First().Split(':').Skip(1).First() 
     }) 
     .Where(x => !x.secondWsno.Equals(tbWorkSheet.Text)) 
     .ToList();