Дублирующие элементы в hashset

Question

У меня проблема с хэшетами на данный момент. У меня есть классы, которые неизменяемы и содержат только один элемент, когда я добавляю два разных класса с одинаковыми данными в хэшсет, я получаю их оба в наборе. Это странно, потому что я перегрузил Equals и GetHashCode как для базового класса, так и для суперкласса.

public abstract class Contact :IEquatable<Contact> 
{ 
    public readonly BigInteger Id; 

    public Contact(BigInteger id) { this.Id = id; } 

    public abstract bool Equals(Contact other); 

    public abstract int GetHashCode(); 

    public abstract bool Equals(object obj); 
} 

И наследующий класс:

public class KeyOnlyContact :Contact, IEquatable<KeyOnlyContact> 
{ 
    public KeyOnlyContact(BigInteger id) :base(id) { } 

    public override bool Equals(object obj) 
    { 
     if (obj is KeyOnlyContact) 
      return Equals(obj as KeyOnlyContact); 
     else if (obj is Contact) 
      return Equals(obj as Contact); 
     else 
      return (this as object).Equals(obj); 
    } 

    public override bool Equals(Contact other) 
    { 
     if (other is KeyOnlyContact) 
      return Equals(other as KeyOnlyContact); 
     else 
      return (this as object).Equals(other as object); 
    } 

    public bool Equals(KeyOnlyContact other) 
    { 
     return other.Id.Equals(Id); 
    } 

    public override int GetHashCode() 
    { 
     return Id.GetHashCode(); 
    } 

Как вы можете видеть, вся настоящая работа откладывается на BigInteger, который является идентификатором. Это класс .net, и я подтвердил, что не получаю дубликат, если просто добавлю BigInteger к hashset.

Для уточнения:

BigInteger a; 
HashSet<Contact> set; 

set.add(new KeyOnlyContact(a)); 
set.add(new KeyOnlyContact(a)); 

set.Count == 2 

Answer 1

public abstract int GetHashCode(); 

Вы случайно повторно объявлены GetHashCode (метод скрытия). Удалите это объявление, и оно может начать работать. Когда ваш производный класс override GetHashCode, они предоставляют этой версии - они не являются, переопределяя object.GetHashCode, что и является обязательным.

Если вы хотите абстрактную GetHashCode, возможно:

public sealed override int GetHashCode() { return GetHashCodeImpl(); } 
protected abstract int GetHashCodeImpl(); 

Теперь производные типы должны обеспечить GetHashCodeImpl, и все они отображаются в object.GetHashCode.