9

次のクラスがあります。

public class SupplierCategory : IEquatable<SupplierCategory>
{
    public string Name { get; set; }
    public string Parent { get; set; }

    #region IEquatable<SupplierCategory> Members

    public bool Equals(SupplierCategory other)
    {
        return this.Name == other.Name && this.Parent == other.Parent;
    }

    #endregion
}

public class CategoryPathComparer : IEqualityComparer<List<SupplierCategory>>
{
    #region IEqualityComparer<List<SupplierCategory>> Members

    public bool Equals(List<SupplierCategory> x, List<SupplierCategory> y)
    {
        return x.SequenceEqual(y);
    }

    public int GetHashCode(List<SupplierCategory> obj)
    {
        return obj.GetHashCode();
    }

    #endregion
}

そして、私は次のlinqクエリを使用しています:

CategoryPathComparer comparer = new CategoryPathComparer();
List<List<SupplierCategory>> categoryPaths = (from i in infoList
                                                          select
                                                            new List<SupplierCategory>() { 
                                                             new SupplierCategory() { Name = i[3] },
                                                             new SupplierCategory() { Name = i[4], Parent = i[3] },
                                                             new SupplierCategory() { Name = i[5], Parent = i[4] }}).Distinct(comparer).ToList();

しかし、次のコードが示すように、distinct は私がやりたいことをしません。

comp.Equals(categoryPaths[0], categoryPaths[1]); //returns True

これを間違った方法で使用していますか？意図したとおりに比較されないのはなぜですか。

編集:比較子が機能することを示すために、次のように true を返します。

List<SupplierCategory> list1 = new List<SupplierCategory>() {
    new SupplierCategory() { Name = "Cat1" },
    new SupplierCategory() { Name = "Cat2", Parent = "Cat1" },
    new SupplierCategory() { Name = "Cat3", Parent = "Cat2" }
};
List<SupplierCategory> list1 = new List<SupplierCategory>() {
    new SupplierCategory() { Name = "Cat1" },
    new SupplierCategory() { Name = "Cat2", Parent = "Cat1" },
    new SupplierCategory() { Name = "Cat3", Parent = "Cat2" }
};
CategoryPathComparer comp = new CategoryPathComparer();
Console.WriteLine(comp.Equals(list1, list2).ToString());
4

2 に答える 2

11

あなたの問題は、IEqualityComparer正しく実装していないことです。

を実装するときは、2 つの等しいオブジェクトが同じハッシュコードを持つように実装IEqualityComparer<T>する必要があります。GetHashCode

そうしないと、ここに表示されているように、正しくない動作が発生します。

次のように GetHashCode を実装する必要があります: (この回答の礼儀)

public int GetHashCode(List<SupplierCategory> obj) {
    int hash = 17;

    foreach(var value in obj)
        hash = hash * 23 + obj.GetHashCode();

    return hash;
}

また、一貫性を保つためにオーバーライドする必要がありますGetHashCode。SupplierCategory例えば：

public override int GetHashCode() {
    int hash = 17;
    hash = hash * 23 + Name.GetHashCode();
    hash = hash * 23 + Parent.GetHashCode();
    return hash;
}

最後に、必要はありませんが、 in をオーバーライドEqualsして、に実装したメソッドをSupplierCategory呼び出すようにする必要があります。EqualsIEquatable

于 2009-10-25T13:53:29.400 に答える
4

実際、この問題はドキュメントでカバーされています：http: //msdn.microsoft.com/en-us/library/bb338049.aspx。

于 2009-11-03T00:51:39.893 に答える