如何正确看待Linq的DistinctBy扩展和ForEach扩展
发布网友
发布时间：2022-04-11 17:58
共1个回答
热心网友
时间：2022-04-11 19:27
class Person{public int Age { get; set; }
public string Name { get; set; }
public override string ToString(){return string.Format("Age:{0} Name:{1}", Age, Name);}}[STAThread]
static void Main(){var persons = new[] {
new Person{Age = 10,Name="a"},
new Person{Age = 10,Name="b"},
new Person{Age = 10,Name="c"},
new Person{Age = 20,Name="d"}};foreach (var p in persons.GroupBy(o=o.Age).Select(g=g.First())){Console.WriteLine(p);}}运行结果从结果看，代码成功完成了根据键值Age进行Distinct的操作。
明眼人可能一眼就看出来了，不错，就是使用GroupBy、Select和First这三个子操作组合出DistinctBy的功能！只不过实现一个DistinctBy功能需要写这么做代码实在是不好，于是可以封装一下，把GroupBy、Select和First这三个子操作的组合封装成一个DistinctBy扩展方法，就可以方便的使用了。封装的代码 too simple，就不贴上来了！
关于ForEach扩展
1.Linq中已经有Select扩展了，因此不必在实现一个ForEach，因为在大多数情况下可以用Select来实现ForEach的功能（当然我不建议这样做，因为Select是延迟操作，如果仅仅实现ForEach功能，有些时候代码不能够按照设计意图正确运行）；
2.ForEach破坏了Linq的编程模式，就是说破坏了链式编程模式，所谓链式编程，大概的意思就是将多个操作通过点号.链接在一起，相信写过Linq的同志很清楚这句话的含义吧！
我看到网上有人这么来设计ForEach扩展：
public static void ForEach<T(this IEnumerable<T source, Action<T foreachAction){foreach (var t in source){foreachAction(t);}}我不得不说这是简直是对Linq的“亵渎”啊，试想，如果别人使用了你的这个ForEach之后，后续操作岂不是成天方夜谭了吗？所以说这样设计ForEach是万万不可的！即使勉强要设计，也要设计成这样才对啊！
public static IEnumerable<T ForEach<T(this IEnumerable<T source, Action<T foreachAction){foreach (var t in source){foreachAction(t);}return source;}而且在我看来，这样添加ForEach扩展并不妥当，原因在于：这样设计的ForEach扩展，破坏了Linq的延迟操作特性！
所以说，我认为微软不在Linq中包含ForEach扩展算是一个正确的选择，如果要用到ForEach功能，还是老老实实用foreach语句来写吧！其实也并不复杂！
