我有一个类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添加了Equals
和GetHashCode
方法,我知道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!
答案 0 :(得分:3)
您的GetHashCode
实施不会为(A,B)和(B,A)返回相同的值。 HashSet
检查它是否在插入时已包含给定的哈希值。如果没有,则该对象将被视为新对象。
如果您更正了GetHashCode
,那么在插入第二个Pair
时,HashSet
将看到哈希码已经存在,并验证与共享相同的其他对象的相等性哈希码。
答案 1 :(得分:2)
请删除此内容:* 397
如果它们被认为是重复项,则它们必须从GetHashCode()返回相同的int。 (但是,如果它们不被视为重复,则仍然允许它们返回相同的int,这只会影响性能)。