从列表中删除所有行,其中每行不包含其他列表中的任何项目

时间:2013-10-14 16:50:21

标签: c# linq collections lambda iterator

为了它的价值,我花了一些时间查看下面相关的帖子,除了它在同一个列表中使用多个属性而不是两个独立列表,它也不涉及包含比较的文本而不是项目匹配。

How to remove all objects from List<object> where object.variable exists at least once in any other object.variable2?

我有一个充满水果的字符串列表,名为'fruits'

Apple
Orange
Banana

我还有一个名为products的字符串列表,其中包含一些水果(以及其他misc信息)和其他一些产品。

ShoeFromNike
ApplePie
OrangeDrink

我需要从第二个列表中删除所有项目,其中每个单独的行不包含水果列表中列出的任何项目。

最终结果将是仅包含以下内容的产品列表:

ApplePie
OrangeDrink

我最好的迭代方法:

//this fails becaucse as I remove items the indexes change and I remove the wrong items (I do realize I could reverse this logic and if it meets the criteria ADD it to a new list and i'm sure there's a better way.)
 for (int i = 0; i < products.Count(); i++)
        {
            bool foundMatch = false;
            foreach (string fruit in fruits)
                if (products[i].Contains(fruit))
                    foundMatch = true;

            if (foundMatch == false)
                products.Remove(products[i]);
        }

我最好的lambda方法:

        products.RemoveAll(p => !p.Contains(fruits.Select(f=> f)));

3 个答案:

答案 0 :(得分:2)

这是我提出的,可能有更好的方法。

products.RemoveAll(p => fruits.Where(f=>p.Contains(f)).Count() == 0);

在英文中,它会读取,删除产品所含水果名称为零的所有产品。

(老实说,循环可能并不是一个糟糕的选择,因为它将来可能更具可读性。)

答案 1 :(得分:2)

我个人喜欢使用.Any(),这似乎更适合我;

    products.RemoveAll(p => !fruits.Any(f => f.IndexOf(p, StringComparison.CurrentCultureIgnoreCase) >= 0));

答案 2 :(得分:1)

如果你想保持循环,你也可以做同样的事情,但要颠倒循环的顺序......

for (int i = products.Count()- 1; i >= 0; i--)
{
    bool foundMatch = false;
    foreach (string fruit in fruits)
        if (products[i].Contains(fruit))
            foundMatch = true;

    if (foundMatch == false)
        products.Remove(products[i]);
}

这可以避免在索引循环之前从列表中删除。