使用IEqualityComparer <T>的推荐最佳做法是什么？

我正在寻找现实世界的最佳实践,其他人如何实现复杂域的解决方案.

每当你考虑使用a时IEqualityComparer,请暂停思考是否可以让类实现IEquatable.如果Product应始终按ID比较a ,只需将其定义为等同,以便您可以使用默认比较器.

也就是说,您可能还需要一些自定义比较器的原因:

如果有多种方式可以认为类的实例是相等的.最好的例子是字符串,框架为其提供了六种不同的比较器StringComparer.

如果以不能将其定义为的方式定义类IEquatable.这将包括由其他人定义的类和由编译器生成的类(特别是匿名类型,默认情况下使用属性方式比较).

如果您确定需要比较器,您当然可以使用通用比较器(请参阅DMenT的答案),但如果您需要重用该逻辑,则应将其封装在专用类中.您甚至可以通过继承通用基础来声明它:

class ProductByIdComparer : GenericEqualityComparer
{
    public ProductByIdComparer()
        : base((x, y) => x.ProductId == y.ProductId, z => z.ProductId)
    { }
}

就使用而言,您应尽可能利用比较器.例如,ToLower()您应该声明字典使用不区分大小写,而不是调用用作字典键的每个字符串(逻辑将在您的应用程序中散布)StringComparer.接受比较器的LINQ运算符也是如此.但同样,要始终考虑应该是类的固有行为而不是外部定义的等同行为.

我做了以下,我不确定它是否是真实世界的最佳实践,但它对我来说很好.:)

public class GenericEqualityComparer : IEqualityComparer
{
    private Func _comparer;
    private Func _hashCodeEvaluator;
    public GenericEqualityComparer(Func comparer)
    {
        _comparer = comparer;
    }

    public GenericEqualityComparer(Func comparer, Func hashCodeEvaluator)
    {
        _comparer = comparer;
        _hashCodeEvaluator = hashCodeEvaluator;
    }

    #region IEqualityComparer Members

    public bool Equals(T x, T y)
    {
        return _comparer(x, y);
    }

    public int GetHashCode(T obj)
    {
        if(obj == null) {
            throw new ArgumentNullException("obj");
        }
        if(_hashCodeEvaluator == null) {
            return 0;
        } 
        return _hashCodeEvaluator(obj);
    }

    #endregion
}

然后你可以在你的收藏中使用它.

var comparer = new GenericEqualityComparer((x, y) => x.ProductId == y.ProductId);
var current = SelectAll().Where(p => p.ShopByGroup == group).ToList();
var toDelete = current.Except(products, comparer);
var toAdd = products.Except(current, comparer);

如果需要支持自定义GetHashCode()功能,请使用备用构造函数提供lambda来执行备用计算:

var comparer = new GenericEqualityComparer(
       (x, y) => { return x.ProductId == y.ProductId; }, 
       (x)    => { return x.Product.GetHashCode()}
);

我希望这有帮助.=)

有关(更好)备选方案,请参阅此文章:在IEqualityComparer中包装委托

向下滚动到KeyEqualityComparer上的部分,尤其是关于GetHashCode重要性的部分.有一个完整的讨论为什么obj.GetHashCode();(正如DMenT的帖子所建议的那样)是错误的,而应该只返回0.