使用C# Linq的確給我們帶來了很多的方便，但是如果不合理使用，會造成一些隱藏的bug，而且很難被發現。

今天我就分享一個工作中遇到的問題。


需求：對list進行遍歷，把滿足某一條件的item Remove掉。

    List<ClassA> list = new List<ClassA>();
for (int i = 0; i < 1000; i++)
{
    list.Add(new ClassA());
}
  

    il.ForEach(x => il.Remove(x));
  

?
為了簡化代碼，在此不加條件語句。

以上代碼貌似是把list中所有的item都Remove掉，但其實不然。




可以看到在執行完ForEach之后list中還有500項。

想必高手們應該之后了吧？在對list進行刪除的時候，list整個集合的index已經發生了變化。

Remove一次，原來集合的index就會整體向前移動一個。

?

原index：

3,4,5,6

Remove()后

index：

2,3,4,5
?

其實以上代碼等價于：

    List<IA> list = new List<IA>();
for (int i = 0; i < 1000; i++)
{
    list.Add(new ClassA());
}
for (int i = 0; i < 500; i++)
{
    list.Remove(list[i]);
}
  

?


所以大家應該知道什么原因了吧？

所以應對以上bug，并且使用簡介的Linq，正確的辦法是：

    for (int i = 0; i <list.Count; i++)
{
??? if (list[i].a==0)
??? {
??????? list.RemoveAt(i);
??????? i--;
??? }
}
  

?

    list.ToList().ForEach(x => {
    if (x.a==0)
    {
        list.Remove(x);
    }
});
  

ToList()會new 一個list，然后對新的list進行遍歷，刪除舊list中與之對應index的值，至少這樣的寫法是對的。但是最優的辦法是使用List<T>中的RemoveAll(Predicate<T> match)方法，該方法還會return 被刪除的items的個數。

    list.RemoveAll(x => x.a == 0);
  

?

C# Linq ForEach 使用請注意