Quelle est la meilleure stratégie pour d'égal à Égal et GetHashCode?

En supposant que les instances sont égaux parce que les codes de hachage sont l'égalité est fausse.

Je suppose que votre mise en œuvre de GetHashCode est OK, mais j'ai l'habitude d'utiliser les choses de similaire à ceci:

public override int GetHashCode() {
    return object1.GetHashCode ^ intValue1 ^ (intValue2 << 16);
}

Aucune des réponses ici vraiment frappé la tache pour moi. Puisque vous avez déjà dit que vous ne pouvez pas utiliser Id pour l'égalité, et vous avez besoin d'utiliser un faisceau de propriétés, voici une meilleure façon de le faire. Note: je ne considère pas cet être la meilleure façon de mettre en œuvre Equals et GetHashCode. C'est une meilleure version de l'OP du code.

public override bool Equals(object obj) {
   var myClass = obj as MyClass;

   if (myClass != null) {
      //Order these by the most different first.
      //That is, whatever value is most selective, and the fewest
      //instances have the same value, put that first.
      return this.Id == myClass.Id
         && this.Name == myClass.Name
         && this.Quantity == myClass.Quantity
         && this.Color == myClass.Color;
   } else {
      //This may not make sense unless GetHashCode refers to `base` as well!
      return base.Equals(obj);
   }
}

public override int GetHashCode() {
   int hash = 19;
   unchecked { //allow "wrap around" in the int
      hash = hash * 31 + this.Id; //assuming integer
      hash = hash * 31 + this.Name.GetHashCode();
      hash = hash * 31 + this.Quantity; //again assuming integer
      hash = hash * 31 + this.Color.GetHashCode();
   }
   return hash;
}

Voir cette réponse de Jon Skeet pour une partie du raisonnement derrière cela. À l'aide de xor n'est pas bon parce que les divers ensembles de données peuvent résultant dans le même hash. Cette synthèse autour de la méthode avec les nombres premiers (les valeurs de départ de 19 et 31 ci-dessus, ou d'autres valeurs que vous choisissez) fait un meilleur travail de la segmentation dans des "compartiments" qui ont peu de collisions chaque.

Si l'un de vos valeurs peuvent être nulles, je vous encourage à réfléchir attentivement à la manière dont ils devraient comparer. Vous pouvez utiliser le court-circuit null évaluation et le nul de la coalescence de l'opérateur, peut-être. Mais assurez-vous que si les valeurs null doit comparer que l'égalité que vous attribuer des codes de hachage pour les différents nullable propriétés lorsqu'ils sont nuls.

Aussi, je ne suis pas convaincu que votre Equals mise en œuvre de sens. Lorsque deux objets sont comparés pour l'égalité, le premier leur GetHashCode valeurs sont comparées. Seulement si ceux-ci sont différents, c'est le Equals méthode run (de sorte que si deux objets de hachage à la même valeur sont différents, cela sera détecté). Depuis votre GetHashCode mise en œuvre ne fait pas référence à la base, il peut n'avoir aucun sens pour votre Equals méthode pour le faire. Plus précisément, vous aurez un sérieux problème d'attente de casser des choses si Equals peut retourner true pour deux objets dont les codes de hachage sont différents.

Je suis tombé sur cette vieille question et, à mon humble avis, je n'ai pas trouvé de réponse claire et simple énoncé de la question initiale formulée par @tucaz.

Je suis d'accord avec beaucoup de considérations partagé ci-dessus (ou en dessous :D), mais «la Question du point» a été raté (je pense).

À condition que:

• L'égalité est nécessaire pour les entités
• Entité-les Objets peuvent être considérés comme égaux si ils ont une carte de la même entité, id est, ils se réfèrent à la même Entité Clé»
• L'exemple illustré par @tucaz juste de parler d'une «Id» (voir la mise en œuvre de GetHashCode())... pour ne pas mentionner le buggy est Égal à(...)

Je peux deviner qu'une simple mise en œuvre pourrait être:

public class MyEntity: IEquatable<MyEntity> {
    int Id;

    public MyEntity(int id){
        Id = id;
    }

    public override bool Equals(object obj) => Equals(obj as MyEntity);
    public bool Equals(MyEntity obj) => obj != null && Id == obj.Id;
    public override int GetHashCode() => Id;
}

C'est tout!