C# - List<T>.Remove()总是删除列表中的第一个对象

Question

C# - List<T>.Remove()总是删除列表中的第一个对象

c#.netequalitygeneric-listiequalitycomparer

5

在Visual Studio 2008（C#）中工作时，我使用List集合来存储我的自定义类(Shift)的实例。我想通过使用Remove方法从列表中删除某个班次。但是，List.Remove()总是删除它找到的第一项。为此，我已经实现了Shift的IComparable接口，我认为这应该足够了，然后我添加了IEqualityComparer的实现，但仍然没有效果。以下是我的实现摘录：

region IComparable Members

    public int CompareTo(object obj)
    {
        Shift s1 = this;
        Shift s2 = (Shift)obj;
        if (s1.start.time != s2.start.time)
            return s1.start.CompareTo(s2.start);
        else
            return s1.end.CompareTo(s2.end);
    }

endregion

region IEqualityComparer Members

    public bool Equals(Shift x, Shift y)
    {
        
        if ((x.opening) != (y.opening)) return false;
        if ((x.closing) != (y.closing)) return false;
        if (!x.opening) if (x._start != y._start) return false;
        if (!x.closing) if (x._end != y._end) return false;
        if (x.when != y.when) return false;
        if (x.day != y.day) return false;
        if (x.EmployeeID != y.EmployeeID) return false;
        return true;
    }

    public int GetHashCode(Shift obj)
    {
        return obj.ToString().ToLower().GetHashCode();
    }

endregion

然而，当列表包含两个班次时，比如“8:00-15:00”和“12:00-16:00”，调用Remove("12:00-16:00")会导致“8:00-15:00”被删除，而后面的班次仍留在集合中!

这里出了什么问题？谢谢。

- Konrad Morawski

4个回答

4

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

IEqualityComparer并不是用于相等性目的的IComparable的等价物。它应该由一个比较器对象实现 - 也就是说，一个比较其他对象相等的对象。如果你想让相等比较固有于你的类中，那么你需要实现IEquatable<T>。或者只需在你的类上覆盖Object.Equals()和Object.GetHashCode()，而不实现任何接口。

- Pavel Minaev

其实 Object.Equals() 很好用！这似乎是最简单的选项。谢谢。 - Konrad Morawski

1

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

你有两个选项可以获得想要的行为:

1) 使Shift实现IEquatable<T>（不仅覆盖Object.Equals或制作该方法，而是使Shift-Shift：IEquatable <Shift>）

2) 使用List<T>.RemoveAt

- Reed Copsey

0

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

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

在这种情况下，"12:00 - 16:00" 是一个String值，而不是一个实际的Shift对象。请确保在您的CompareTo方法中，您的代码正确将String值转换为Shift对象。否则，当比较开始时间时...事情可能会出错。

- Justin Niessner

事实上，Justin，我并没有真正传递一个字符串参数，这只是我自己的“伪代码”，只是为了展示这个概念。如果有歧义，对不起。 - Konrad Morawski

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Rex M · Accepted Answer

你可以覆写 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)中，你也应该进行空值检查。