代码之家 › 专栏 › 技术社区 › Edward Tanguay

在删除不需要的对象之前,最简单的方法是什么?

collections list generics .net c#

Edward Tanguay · 技术社区 · 16 年前

在我的应用程序中,_collection是一个列表,我需要从中 删除所有用户对象 哪个不匹配标准 .

但是,以下代码得到 无效操作 自_集合本身更改后,其第二次迭代中出错:

foreach (User user in _collection)
{
    if (!user.IsApproved())
    {
        _collection.Remove(user);
    }
}

我可以创建另一个列表集合并来回复制它们,但是我有非克隆引用类型等问题。

有没有比将集合复制到另一个列表变量更优雅地完成上述操作的方法?

3 回复 | 直到 16 年前

Mehrdad Afshari 16 年前

_collection.RemoveAll(user => !user.IsApproved());

如果你是仍然 2:

_collection.RemoveAll(delegate(User u) { return !u.IsApproved(); });

顺便说一下,如果你不想触摸原始列表,你可以 另一个 已批准用户的列表:

_collection.FindAll(user => user.IsApproved());

Dan Tao 16 年前

始终可以从顶部索引开始,向下迭代到0:

for (int i = _collection.Count - 1; i >= 0; i--)
{
    User user = _collection[i];
    if (!user.IsApproved())
    {
        _collection.RemoveAt(i);
    }
}

不过,迈尔达德的回答看起来很优雅。

Mike J 16 年前

每当有机会在循环中修改集合时,请选择 for 循环代替。Mehrdad给出的解决方案非常可爱,值得一试!

以下是处理可修改集合时有用的代码:

for(int index=0;index < _collection.Count; index++)
{
    if (!_collection[index].IsApproved)
    {
        _collection.RemoveAt(index);
        index--;
    }
}

推荐文章

SpaceCowboy74 Mohammad Aghazadeh · 使用Fluent Validation比较数组中的值

7 月前