IEqualityComparerおよびContainsメソッド

I have this simple class with those 2 enum fields, I'm trying to find one item of this object in a collection (List) but the Contains methods doesn't works correctly

public class Calculator : IEqualityComparer
{
    public DashboardsComputationMode ComputationMode { get; set; }
    public Modes Mode { get; set; }

    public Calculator(DashboardsComputationMode dashboardsComputationMode, Modes mode)
    {
        ComputationMode = dashboardsComputationMode;
        Mode = mode;
    }

    public bool Equals(Calculator x, Calculator y)
    {
        return (x.ComputationMode.Equals(y.ComputationMode) && x.Mode.Equals(y.Mode));
    }

    public int GetHashCode(Calculator obj)
    {
        return obj.ComputationMode.GetHashCode() ^ obj.Mode.GetHashCode();
    }
}

public enum DashboardsComputationMode
{
    Weighted = 0,
    Aggregated = 1,
    PR = 2,
    CurrentValue = 3,
    EquivalentHours = 4,
    AggregatedCorrected = 5,
    PRCorrected = 6
}

public enum Modes
{
    InstantaneousMode = 0,
    DailyMode = 1,
    MonthlyMode = 2,
    YearlyMode = 5,
    Undefined = 4,
}

なぜこのテストはうまくいかないのでしょうか?

[TestMethod]
public void TestMethod1()
{
  var list = new List()
  {
    new Calculator(DashboardsComputationMode.PR, Modes.DailyMode),
    new Calculator(DashboardsComputationMode.CurrentValue, Modes.YearlyMode),
    new Calculator(DashboardsComputationMode.PRCorrected, Modes.MonthlyMode)
  };

  var item = new Calculator(DashboardsComputationMode.CurrentValue, Modes.YearlyMode);
  Assert.IsTrue(list[1].Equals(item));
  Assert.IsTrue(list.Contains(item));
}

最初のアサートは正常に動作します

Assert.IsTrue(list[1].Equals(item)) 

しかし、2番目の

Assert.IsTrue(list.Contains(item));
15
nl ru de
私はいつもlist.Containsが参照チェックをしていると仮定していたので、一致するだけでなく、同じインスタンスを持たなければなりません。 IEqualityComparerをとるオーバーロードがあります。
追加された 著者 Jon Egerton,

2 答え

List.Contains determines equality by using the default equality comparer (the one returned by the EqualityComparer.Default).

Here's the MSDN explanation on how EqualityComparer.Default works:

Defaultプロパティは、タイプTが    System.IEquatable インターフェイス。   その実装を使用する EqualityComparer 。それ以外の場合は、   上書きを使用する EqualityComparer を返します。    Object.Equals および Object.GetHashCode Tによって提供されます。

言い換えれば、 Calculator クラスでは、 System.IEquatable System.IEqualityComparer インターフェイスの インターフェイスを無効にするか、 Object.Equals および Object.GetHashCode a>メソッド。

ありがとう!!それはうまくIEquatableを実装して動作します
追加された 著者 Pablo Rodríguez,

You are not using IEqualityComparer in Equals and Contains both. EqualityComparer has a different significance. I corrected the code for you.

  public class CalculatorComparer : IEqualityComparer
{

    public bool Equals(Calculator x, Calculator y)
    {
        return (x.ComputationMode.Equals(y.ComputationMode) && x.Mode.Equals(y.Mode));
    }

    public int GetHashCode(Calculator obj)
    {
        return obj.ComputationMode.GetHashCode() ^ obj.Mode.GetHashCode();
    }
}
public class Calculator
{
    public DashboardsComputationMode ComputationMode { get; set; }
    public Modes Mode { get; set; }

    public Calculator(DashboardsComputationMode dashboardsComputationMode, Modes mode)
    {
        ComputationMode = dashboardsComputationMode;
        Mode = mode;
    }


    public override bool Equals(object obj)
    {
        Calculator y = obj as Calculator;

        return (this.ComputationMode.Equals(y.ComputationMode) && this.Mode.Equals(y.Mode));
    }
}

public enum DashboardsComputationMode
{
    Weighted = 0,
    Aggregated = 1,
    PR = 2,
    CurrentValue = 3,
    EquivalentHours = 4,
    AggregatedCorrected = 5,
    PRCorrected = 6
}

public enum Modes
{
    InstantaneousMode = 0,
    DailyMode = 1,
    MonthlyMode = 2,
    YearlyMode = 5,
    Undefined = 4,
}

今度は両方とも真を返すべきです。

5
追加された
@ロブあなたは正しいです。私はそれを逃す。私はそれを追加します。
追加された 著者 D J,
null の場合は calculator.Equals メソッドでヌルをチェックし、オブジェクトobjをキャストしようとした後に false を返す必要があります。
追加された 著者 RobH,
ありがとう、私はEqualsメソッドを書く場合、実際には、私はCalculatorComparerを作成する必要はありませんIEquatableを実装する
追加された 著者 Pablo Rodríguez,