C# - List.Remove() 始终删除列表中的第一个对象

Question

在 Visual Studio 2008 (C#) 中工作... 我使用 List 集合来存储自定义类 (Shift) 的实例。

我想使用“删除”方法从列表中删除某个班次。

但 List.Remove() 总是删除它找到的第一个项目。

我已经为我的 Shift 实现了 IComparable 接口，我认为这已经足够了，然后我添加了 IEqualityComparer 的实现，但它仍然没有效果。

这是我的实现的摘录：

地区I可比会员

 public int CompareTo(object obj)
    {
        移位 s1 = 这个；
        Shift s2 = (Shift)obj;
        if (s1.开始时间！= s2.开始时间)
            返回 s1.start.CompareTo(s2.start);
        别的
            返回 s1.end.CompareTo(s2.end);
    }

结束区域

地区 IEqualityComparer 成员

 public bool 等于(Shift x, Shift y)
    {
        
        if ((x.opening) != (y.opening)) 返回 false;
        if ((x.ending) != (y.ending)) 返回 false;
        if (!x.opening) if (x._start != y._start) return false;
        if (!x.close) if (x._end != y._end) return false;
        if (x.when != y.when) 返回 false;
        if (x.day!= y.day) 返回 false;
        if (x.EmployeeID != y.EmployeeID) 返回 false;
        返回真；
    }

    公共 int GetHashCode(Shift obj)
    {
        返回 obj.ToString().ToLower().GetHashCode();
    }

结束区域

然而，仍然 - 当列表包含两个班次时，说“8:00 - 15:00”； “12:00 - 16:00”，调用Remove(“12:00-16:00”)会导致“8:00 - 15:00”被删除，而后一个仍保留在集合中！

这是怎么回事？谢谢

Answer 1

您可以覆盖 object.GetHashCode 和 object.Equals：

public override bool Equals(object obj)
{
    if(obj == null)
    {
        return false;
    }
    return Equals(this, obj as Shift);
}

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

您还应该在 Equals(x, y) 中执行 null 检查。

Answer 2

IComparable 通常不用于比较相等性（它用于排序），因此 List.Remove() 会忽略它。

出于平等目的，IEqualityComparer 并不等同于 IComparable。它应该由一个比较器对象实现 - 即比较其他对象是否相等的对象。如果您希望相等比较是您的类所固有的，那么您需要实现 IEquatable。或者只是在类上重写 Object.Equals() 和 Object.GetHashCode()，而不实现任何接口。

Answer 3

删除使用 EqualityComparer.Default 来确定相等性并选择要删除的对象，如果在您的对象上实现，则将使用 IEquatable，否则，它将使用引用相等。

您有两种选择来获得所需的行为：

1) 让 Shift 实现 IEquatable（不仅仅是重写 Object.Equals 或创建方法，而是创建 Shift - Shift : IEquatable）

2) 使用 List.RemoveAt

Answer 4

根据您提供的示例，您正在调用：

List<Shift>.Remove("12:00 - 16:00");

"12:00 - 16:00" 在本例中，是一个 String 值，而不是实际的 Shift对象。确保您正在编写的 CompareTo 方法中正确地将 String 值转换为 Shift 对象。否则，当比较开始时间时……事情可能会变得混乱。