HashSet不会删除具有重写相等性的dupes

时间:2013-06-04 01:52:40

标签: c# .net

我有一个类Pair,它需要有一个重写的相等运算符,因为它存储的两个值是可互换的,代码是这个

Pair.cs

public class Pair
    {
        protected bool Equals(Pair other)
        {
            return (Equals(A, other.A) && Equals(B, other.B)) || (Equals(A, other.B) && Equals(B, other.A));
        }

        public override bool Equals(object obj)
        {
            if (ReferenceEquals(null, obj)) return false;
            if (ReferenceEquals(this, obj)) return true;
            return obj.GetType() == this.GetType() && Equals((Pair) obj);
        }

        public override int GetHashCode()
        {
            unchecked
            {
                return ((A != null ? A.GetHashCode() : 0)*397) ^ (B != null ? B.GetHashCode() : 0);
            }
        }

        public readonly Collision A, B;

        public Pair(Collision a, Collision b)
        {
            A = a;
            B = b;
        }

        public static bool operator ==(Pair a, Pair b)
        {
            if (ReferenceEquals(a, b))
                return true;

            if ((object)a == null || (object)b == null)
                return false;

            return (a.A == b.A && a.B == b.B) || (a.A == b.B && a.B == b.A);
        }

        public static bool operator !=(Pair a, Pair b)
        {
            return !(a == b);
        }
    }

我让ReSharper添加了EqualsGetHashCode方法,我知道Equals没问题,但是GetHashCode输出了正确的值吗?

我有HashSet<Pair>用于存储对,我需要确保此列表中没有重复,但是当我将对添加到HashSet时,它不会删除重复。

只是为了澄清,这将是重复的:

Pair a = new Pair(objecta, objectb);
Pair b = new Pair(objectb, objecta);
HashSet<Pair> pairs = new HashSet<Pair>();
pairs.Add(a);
pairs.Add(b);
return pairs.Count(); //Count() returns 2 when it should be 1!

2 个答案:

答案 0 :(得分:3)

您的GetHashCode实施不会为(A,B)和(B,A)返回相同的值。 HashSet检查它是否在插入时已包含给定的哈希值。如果没有,则该对象将被视为新对象。

如果您更正了GetHashCode,那么在插入第二个Pair时,HashSet将看到哈希码已经存在,并验证与共享相同的其他对象的相等性哈希码。

答案 1 :(得分:2)

请删除此内容:* 397

如果它们被认为是重复项,则它们必须从GetHashCode()返回相同的int。 (但是,如果它们不被视为重复,则仍然允许它们返回相同的int,这只会影响性能)。