Как правильно переопредетиль Equals?

Question

[Валера Фиголь]

Есть моделька:

public class CoastModel:BaseEntity
    {
        public string AppointmentCoast { get; set; }
        public double CurrencyUAH { get; set; } = 0;
        public double CurrencyUSD { get; set; } = 0;
        public double CourseOfExchange { get; set; } = 0;
        public string CategoryId { get; set; }
        public string CheckUrl { get; set; } = null;
        public DateTime Date { get; set; }
        public string AutorCoastID { get; set; }
    }

Как переопределить метод Equals, что бы сравнить все эти поля?

Answer 1

[Дмитрий Еремин]

ну а в чем проблема?

public override bool Equals(System.Object obj)
    {
       // If parameter is null return false.
        if (obj == null)
        {
            return false;
        }

        // If parameter cannot be cast to CoastModel return false.
        CoastModel p = obj as CoastModel;
        if ((System.Object)p == null)
        {
            return false;
        }

        // Return true if the fields match:
        return (AppointmentCoast == p.AppointmentCoast ) && (CurrencyUAH  == p.CurrencyUAH ) && ... (AutorCoastID == p.AutorCoastID );
    }

Comments

[Толстый Лорри]

А как же базовый тип?

[Дмитрий Еремин]

Толстый Лорри: я так понял, вопрос как само сравнение реализовать,
Все неточности, надеюсь додумает

[Толстый Лорри]

Дмитрий Еремин: Ну вот само сравнение в данном конкретном случае выполняется неправильно. Там, откуда вы брали реализацию, есть более подходящий пример - про class ThreeDPoint : TwoDPoint {...}.

[Дмитрий Еремин]

Толстый Лорри: дап, согласен

Answer 2

[lam0x86]

Во-первых, не забывайте про правило Equals + GetHashCode. При переопределении Equals следует переопределять GetHashCode.

Во-вторых, переопределять Equals и GetHashCode для изменяемых типов - плохая практика. Всегда есть вероятность, что объект будет использован в HashSet'е или Dictionary, а они требуют неизменности ключа. GetHashCode в идеале должен быть стабильным, то есть всегда возвращать одинаковое значение.

В третьих (это уже моё субъективное мнение), в большинстве случаев вообще не стоит переопределять эти методы. Кроме, может быть, совсем примитивных типов вроде "строка", "комплексное число", "дата" и т.п. То есть, там, где не может быть никаких разночтений.
Объекты бизнес-логики чаще всего могут быть одновременно равны и не равны в разных контекстах сравнения. Например, где-то необходимо сравнить двух пользователей по ФИО, а в другом месте - глубокое сравнение всех полей, в т.ч. телефон и имейл. В первом случае объекты могут быть равны, а во втором различны.
В этом случае стоит написать несколько компараторов, реализовав интерфейс IEqualityComparer, и передавать нужный компаратор в нужном месте, например, в конструктор Dictionary.

Answer 3

[Ogoun Er]

Использую такой вариант, после изучения вопроса сравнения объектов:

public class Sample
{
    public Guid Id;

    public bool Equals(Sample other)
    {
        if ((object)this == (object)other)
            return true;
        if (this == null)
            throw new NullReferenceException();
        if (other == null)
            return false;
        if (ReferenceEquals(this, other))
            return true;
        if (this.GetType() != other.GetType())
            return false;
        return Id == other.Id;
    }

    public override bool Equals(object obj)
    {
        return Equals(obj as Sample);
    }

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

Comments

[Ogoun Er]

И да, если используется наследование как у вас, то общую часть стоит вынести в BaseEntity:

public bool Equals(BaseEntity other)
{
    if ((object)this == (object)other)
        return true;
    if (this == null) // и так бывает
        throw new NullReferenceException();
    if (other == null)
        return false;
    if (ReferenceEquals(this, other))
        return true;
    if (this.GetType() != other.GetType())
        return false;
    return Id == other.Id;
}

public override bool Equals(object obj)
{
    if (this == null)
        throw new NullReferenceException();

    return Equals(obj as BaseEntity);
}

public static bool operator ==(BaseEntity first, BaseEntity second) => Equals(first, second);
public static bool operator !=(BaseEntity first, BaseEntity second) => !Equals(first, second);

А в производном классе делать уже проще:

public class CoastModel : BaseEntity
{
    public string AppointmentCoast { get; set; }
    public double CurrencyUAH { get; set; } = 0;
    public double CurrencyUSD { get; set; } = 0;
    public double CourseOfExchange { get; set; } = 0;
    public string CategoryId { get; set; }
    public string CheckUrl { get; set; } = null;
    public DateTime Date { get; set; }
    public string AutorCoastID { get; set; }

    public override bool Equals(object obj)
    {
        return this.Equals(obj as CoastModel);
    }

    public bool Equals(CoastModel other)
    {
        if (other == null) return false;
        bool eq = base.Equals(other);
        if (false == eq) return false;
        eq &= this.CurrencyUAH == other.CurrencyUAH;
        eq &= this.CurrencyUSD == other.CurrencyUSD;
        eq &= this.CourseOfExchange == other.CourseOfExchange;
        eq &= String.Compare(this.AppointmentCoast, other.AppointmentCoast, StringComparison.Ordinal) == 0;
        eq &= String.Compare(this.CategoryId, other.CategoryId, StringComparison.Ordinal) == 0;
        eq &= String.Compare(this.CheckUrl, other.CheckUrl, StringComparison.Ordinal) == 0;
        eq &= String.Compare(this.AutorCoastID, other.AutorCoastID, StringComparison.Ordinal) == 0;
        eq &= DateTime.Compare(this.Date, other.Date) == 0;
        return eq;
    }
}