我有一些对象有一堆字段,我发现自己必须实现GetHashCode和Equals.手动执行每个字段是很痛苦的,所以我写道：
public override int GetHashCode()
{
int hash = 17;
foreach (PropertyInfo p in GetType().GetProperties())
{
hash = hash * 23 + p.GetValue(this,null).GetHashCode();
}
return hash;
}
public override bool Equals(object obj)
{
foreach (PropertyInfo p in GetType().GetProperties())
{
if (p.GetValue(obj,null) != p.GetValue(this,null))
return false;
}
return true;
}
除了速度考虑,为什么我不应该这样实现？
解决方法
这是我避免这条路线的几个原因
比较字段而不是属性更可靠
>你的代码做出错误的假设,如果它们是相同的引用(你使用==),两个对象被认为是相等的.这不是这样,因为许多类型通过.Equals实现值相等.两个不同的参考文献被认为是相等的,并且会击败你的测试是非常有可能和合法的.
>如果这种平等的形式以广泛的方式通过代码库使用,那么当对象图形具有循环时,它将很容易导致无限递归.
> GetHashCode方法忽略一个属性可以为null
下面是一个类型的具体示例,这将在应用程序中引起无限递归
class C1 {
public object Prop1 { get; set; }
};
var local = new C1();
local.Prop1 = local;
var x = local.GetHashCode(); // Infinite recursion
