比較演算子を設計するときのStackOverflow

Question

this articleによると、Equal（Thing）を以下のように伴いました。

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

public override bool Equals(object input) 
{ 
    Thing comparee = input as Thing; 
    return comparee != null && comparee.Id == Id; 
} 

public static bool operator ==(Thing self, Thing other) 
{ 
    return self != null && other != null && self.Id == other.Id; 
} 

public static bool operator !=(Thing self, Thing other) 
{ 
    return self == null || other == null || self.Id != other.Id; 
} 

問題は、それは私がオペレータの再定義を追加する前に働いていたが、今私はStackOverflowExceptionがを得ることです。私は何が欠けていますか？

Answer 1

が、しかしそこので、ここで、私はこの場合にはしたいと思い、別のポイントであり、私はそれを実装する方法を次のとおりです。等しい基準を使用して

public static bool operator ==(Thing self, Thing other) 
{ 
    return !ReferenceEquals(self, null) && 
      !ReferenceEquals(other, null) && 
      self.Id == other.Id; 
} 

public static bool operator !=(Thing self, Thing other) 
{ 
    return !(self == other); 
} 

は、スタックオーバーフローが発生することはありませんオペレーター==/!=オペレーターを使用せず、オペレーターの!を単に返すように!=を実装すると、同等性テストが変更される場合に備えて保守が省かれます。これはDRY原則の実装です。

Answer 2

Thingを比較する==演算子を定義すると、someThing == nullと言うときに使用されます。同様に!=と同じです。

self == otherと言うと、operator ==(self, other)と呼ばれることになります。あなたの基準の中でoperator !=(self, null)を呼び出すself != nullがあります。self == nullをチェックし、operator ==(self, null)を呼び出すと、スタックスペースが足りなくなるまで行くことになります。

リファレンス比較のために、資料をobjectにキャストすることでこの問題を回避できることは間違いありません。または、Object.ReferenceEquals(self, null)などと言うことができます。これは==に依存しないため、再帰を取得しません。

私は完全にチャオの答えに同意