C# Linq ForEach 施用请注意

2012-09-07

C# Linq ForEach 使用请注意使用C# Linq的确给我们带来了很多的方便，但是如果不合理使用，会造成一些隐藏的

C# Linq ForEach 使用请注意

使用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掉，但其实不然。

C# Linq ForEach 施用请注意

可以看到在执行完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]);}

?
C# Linq ForEach 施用请注意

所以大家应该知道什么原因了吧？

所以应对以上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);

热点排行

.NET

C# Linq ForEach 施用请注意