换句话说,假设我有这个类：

class EqualsTestClass
{
    public string MyDescription { get; set; }
    public int MyId { get; set; }

    public override bool Equals(object obj)
    {
        EqualsTestClass eq = obj as EqualsTestClass;
        if(eq == null) {
            return false;
        } else {
            return MyId.Equals(eq.MyId);
        }
    }

    public override int GetHashCode()
    {
        int hashcode = 23;
        return (hashcode * 17) + MyId.GetHashCode();
    }
}

如果它们具有相同的MyId,我认为两个对象相等.因此,如果Id相等但描述不同,它们仍然被认为是相等的.

我只是想知道这种方法的缺陷是什么？当然,像这样的结构将按预期运行：

List<EqualsTestClass> test = new List<EqualsTestClass>();

        EqualsTestClass eq1 = new EqualsTestClass();
        eq1.MyId = 1;
        eq1.MyDescription = "Des1";

        EqualsTestClass eq2 = new EqualsTestClass();
        eq2.MyId = 1;
        eq2.MyDescription = "Des2";

        test.Add(eq1);
        if (!test.Contains(eq2))
        {
            // Will not be executed,as test.Contains is true
            test.Add(eq2);
        }

由于eq2的值等于eq1,因此不会添加.但这是我控制的代码,但我想知道框架中是否存在可能导致意外问题的代码？

那么,我应该总是在我的Equals()比较中添加所有公共字段,或者是什么原则可以避免令人讨厌的惊喜,因为一些完全出乎意料的糟糕的Framework-Mojo？