Warum sollte ich Equals und GetHashCode nicht mithilfe von Reflektion implementieren?

Question

Ich habe einige Objekte mit einer Reihe von Feldern und ich finde, dass ich GetHashCode und Equals implementieren muss. Es ist schmerzhaft zu gehen, obwohl jedes Feld manuell so dass ich sie wie folgt geschrieben:

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; 
} 

Andere als die Geschwindigkeit Überlegungen, warum soll ich sie nicht umsetzen, wie das?

Answer 1

Hier sind ein paar Gründe, warum ich diesen Weg

die falsche Annahme, dass zwei Objekte gelten als gleich vermeiden würde, wenn sie

• Es ist sehr viel zuverlässiger Felder vergleichen anstelle von Eigenschaften
• Ihr Code macht die gleiche Referenz (Sie verwenden ==). Dies ist nicht der Fall, da viele Typen die Wertgleichheit über .Equals implementieren. Es ist sehr möglich und legal, dass zwei verschiedene Referenzen als Equals betrachtet werden und Ihren Test übertreffen würden.
• Wenn diese Form der Gleichheit weit verbreitet durch Ihre Codebasis verwendet wird, wird es sehr leicht zu unendlicher Rekursion führen, wenn der Objektgraph Zyklen hat.
• Die GetHashCode Methode ignoriert, dass eine Eigenschaft null könnte

Im Folgenden ein konkretes Beispiel für eine Art ist, die unendliche Rekursion in Ihrer Anwendung

class C1 { 
    public object Prop1 { get; set; } 
}; 

var local = new C1(); 
local.Prop1 = local; 
var x = local.GetHashCode(); // Infinite recursion 

Answer 2

Alle Werttypeigenschaften werden durch die Aufrufe GetValue eingerahmt, was bedeutet, dass sie niemals als gleichwertig angesehen werden, auch wenn sie denselben Wert haben.

Sie können dies vermeiden, indem Sie die statische Equals(x,y) Methode aufrufen - die dann mit der virtuellen x.Equals(y) Methode verschieben, wenn erforderlich - statt dem nicht-virtuellen == Operator, der in diesem Fall immer Referenz Gleichheit testen.

if (!object.Equals(p.GetValue(obj, null), p.GetValue(this, null))) 
    return false; 

Answer 3

1. Es kann eine schlecht konditionierten Hash geben (nicht alle Eigenschaften gleich sind in Objektidentität zu bestimmen.)

2. Wie derzeit implementiert, die Hash-Berechnung überlaufen kann.

Answer 4

Wenn Ihr Objekt gleich nur, wenn alle Eigenschaften gleich sind, dann gehen Sie vor. Aber ich bezweifle es. ZB ein Mitarbeiter ist einzigartig durch seine Mitarbeiter-ID. Wenn Sie dies tun, können Sie Änderungen an Mitarbeiterdaten nicht vergleichen.