C＃Equals扩展程序无法检查相等性

Question

I extended the equals method and hashcode to check for equality of two identical objects with boolean properties. when I mutate the object making one of the boolean properties false instead of true it fails to recognize the difference and asserts they are equal;. Any Ideas why?

   public override bool Equals(object value)
    {
        if (!(value is LocationPropertyOptions options))
            return false;

        return Equals(options, value);
    }

    public bool Equals(LocationPropertyOptions options)
    {
        return options.GetHashCode() == GetHashCode();
    }

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

    public override string ToString()
    {
        return $"{Identifier}{AccountEntityKey}{Address}{Comments}{Contact}" +
               $"{Coordinate}{Description}{FaxNumber}{LastOrderDate}{PhoneNumber}" +
               $"{ServiceAreaOverride}{ServiceRadiusOverride}{StandardInstructions}" +
               $"{WorldTimeZone_TimeZone}{ZoneField}{CommentsOptions}";
    }

Answer 1

You cast options from value , then call Equals with options vs value . That means you compare value with value , it returns always true for you

public override bool Equals(object value)
{
    if (!(value is LocationPropertyOptions options))
       return false;    
    return Equals(options, value);
}

Try comparing this with value , like

return Equals(this, value);

Answer 2

This doesn´t really answer you question. However you should consider a few things when implementing equality to avoid this kind of error.

First you have two completely different implementations to indicate equality. Your override bool Equals(object value) redirects to the static method Object.Equals(object, object) , which just performs a ReferenceEquals . Your public bool Equals(LocationPropertyOptions) (probably an implementation for IEquatable<LocationPropertyOptions> ) on the other hand simply uses your strange GetHashCode -implementation, which is

Point two: you should not use a mutable member within your hashcode-implementation, in particular when your objects are stored within a dictionary or a hashmap, which heavily depends on a good implementaion for a hashcode. See MSDN on GetHashCode

You can override GetHashCode() for immutable reference types. In general, for mutable reference types, you should override GetHashCode() only if:

• You can compute the hash code from fields that are not mutable; or

• You can ensure that the hash code of a mutable object does not change while the object is contained in a collection that relies on its hash code.

Third and last: you shouldn´t use GetHashCode in your check for equality:

Do not test for equality of hash codes to determine whether two objects are equal

Whilst equal objects are assumed to have identical hashcodes, different objects may have the exact same hashcode anyway. An equal hashcode therefor is just an indicator that two instances may be equal.

Two objects that are equal return hash codes that are equal. However, the reverse is not true: equal hash codes do not imply object equality, because different (unequal) objects can have identical hash codes [...]
You should not assume that equal hash codes imply object equality.