Przesłanianie równań(): czy zerowe porównanie jest zbędne przy wywołaniu base.Equals()?

Question

Kiedy przesłanianie metody Equals() The MSDN recommends this:

class Point: Object { 
    protected int x, y; 

    public Point(int X, int Y) { 
     this.x = X; 
     this.y = Y; 
    } 

    public override bool Equals(Object obj) { 

     //Check for null and compare run-time types. 
     if (obj == null || GetType() != obj.GetType()) return false; 

     Point p = (Point)obj; 

     return (x == p.x) && (y == p.y); 
    } 
} 

Ale jeśli wiemy, że podklasa dziedziczy bezpośrednio od Object, następnie jest następujący odpowiednik? Zwróć uwagę na !base.Equals() połączenia:

class Point: Object { 
    protected int x, y; 

    public Point(int X, int Y) { 
     this.x = X; 
     this.y = Y; 
    } 

    public override bool Equals(Object obj) { 

     if (!base.Equals(obj) || GetType() != obj.GetType()) return false; 

     Point p = (Point)obj; 

     return (x == p.x) && (y == p.y); 
    } 
} 

Answer 1

W przypadku this odniesienia jest null masz rację, że kontrola może być (ale nie wydają się być gwarantowane) zbędne, jak to widać w RuntimeHelpers.Equals realizacji cyt this answer.

Jednak kontrola !base.Equals(obj) złamie twój Equals. Gdy odwołania nie są null - !base.Equals, będą również dawać true dla dowolnych różnych wartości 10 nie tylko dla wartości null.

Scenariusz gdy idzie źle to na przykład:

Point x = new Point(1,2); 
Point y = new Point(1,2); 

Console.WriteLine(x.Equals(y)); // will print 'False' 

Nawet x i y są równe pod względem logiki biznesowej, są one różne obiekty tak base.Equal zwraca false.