C# · 12月 26, 2021

c# – 为什么不应该使用反射来实现Equals和GetHashCode?

我有一些对象有一堆字段,我发现自己必须实现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