C# 扩展方法

Question

我目前正在尝试编写一个扩展方法，但它似乎没有按预期运行。在我们深入研究之前，先看一下我的代码： 我的

 public static void Remove<T>(this IEnumerable<T> source, Func<T, bool> predicate)
 {
     var items = source.Where(predicate);

     source = source.Where(t => !items.Contains(t));
 }

愿望是我可以在任何 IEnumerable 上调用此扩展方法，然后从集合中删除与谓词匹配的所有项目。我厌倦了迭代集合以查找匹配的项目，然后一次删除一个项目以避免在枚举集合时更改集合...

无论如何...当我单步执行代码时，一切似乎都有效。在存在该方法之前，源已删除正确数量的项目。但是，当我返回调用代码时，所有项目仍然存在于我的原始 IEnumerable 对象中。有什么建议吗？

预先感谢，
桑尼

Answer 1

无法按照您最初编写的方式执行此操作，您将获取一个引用变量（source）并使其引用一个新实例。这会修改本地引用 source，而不是传入的原始参数。

请记住，对于 C# 中的引用类型，默认参数传递方案是按值传递（其中传递的值是引用）。

假设您向此方法传递一个变量 x，该变量引用原始列表，并且该列表位于理论位置 1000，这意味着 source 是对位于位置 1000 的原始列表的新引用 现在，

当您说：

source = source.Where(....);

您正在将 source 分配给一个新列表（例如位置 2000），但这只会影响 source 指向的内容，而不影响 x 你通过了要将

其作为扩展方法修复，您确实需要返回新序列：

 public static IEnumerable<T> Remove<T>(this IEnumerable<T> source, Func<T, bool> predicate)
 {
     if (source == null) throw new ArgumentNullException("source");
     if (predicate == null) throw new ArgumentNullException("predicate");

     // you can also collapse your logic to returning the opposite result of your predicate
     return source.Where(x => !predicate(x));
 }

这一切都假设您希望使其完全通用于IEnumerable 正如您在问题中所问的那样。显然，正如其他示例中所指出的，如果您只关心 List，则有一个内置的 RemoveAll() 方法。

Answer 2

这种扩展应该通过返回一个新序列来实现。这样您就可以集成到一系列序列操作中：

public static IEnumerable<T> Remove<T>(this IEnumerable<T> source, Func<T, bool> predicate)
{
    return source.Where(t => !predicate(t));
}

var query = mySequence.Select(x => x.Y).Remove(x => x == 2).Select(x => 2*x);

现在该方法只不过是 Where() 的包装器，这显然没有帮助。您可能会考虑摆脱它。

如果您想实际更新底层集合（假设甚至存在），那么您不能这样做，因为 IEnumerable 不提供任何更改其内容的方法。您必须执行以下操作：

var myNewList = new List<int>(oldList.Remove(x => x == 2));

最后，如果您正在使用 List，则可以使用 RemoveAll() 方法来实际上从列表中删除项目：

int numberOfItemsRemoved = myList.RemoveAll(x => x == 2);

Answer 3

试试这个，有一个有用的 List.RemoveAll(Predicate match) 方法，我认为它是为此设计的： http://msdn.microsoft.com/en-us/library/wdka673a.aspx

所以只需在您拥有的列表上使用它即可。

source.RemoveAll(t => !items.Contains(t))

或者您的扩展方法返回所需的枚举，您可以使用它。

Answer 4

这是因为 IEnumerable 是不可变的
您必须从 Remove 方法返回另一个序列才能使其工作：

public static IEnumerable<T> Remove<T>(this IEnumerable<T> source, Func<T, bool> predicate)
{
   var items = source.Where(predicate);

   return source.Where(t => !items.Contains(t));
}