.Contains() 方法不调用 Overridden equals 方法

Question

我有一个问题，我创建了一个 Foo 对象的 ArrayList，我覆盖了 equals 方法，但我无法让 contains 方法调用 equals 方法。我试过同时覆盖equals和hashcode，但它仍然不起作用。我敢肯定有一个合乎逻辑的解释为什么会这样，但我现在无法自己弄明白，哈哈。我只是想要一种方法来查看列表是否包含指定的 id。

这里有一些代码：

import java.util.ArrayList;
import java.util.List;

public class Foo {

    private String id;


    public static void main(String... args){
        Foo a = new Foo("ID1");
        Foo b = new Foo("ID2");
        Foo c = new Foo("ID3");
        List<Foo> fooList = new ArrayList<Foo>();
        fooList.add(a);
        fooList.add(b);
        fooList.add(c);
        System.out.println(fooList.contains("ID1"));
        System.out.println(fooList.contains("ID2"));
        System.out.println(fooList.contains("ID5"));
    }   

    public Foo(String id){
        this.id = id;
    }

    @Override
    public boolean equals(Object o){
        if(o instanceof String){
            String toCompare = (String) o;
            return id.equals(toCompare);
        }
        return false;
    }



    @Override
    public int hashCode(){
        return 1;
    }
}

输出： 错误的 错误的 假的

Answer 1

这是因为您的equals() 不是对称：

new Foo("ID1").equals("ID1");

但是

"ID1".equals(new Foo("ID1"));

不正确。这违反了equals() 合约：

equals 方法在非空对象引用上实现等价关系：

• [...]

• 它是对称的：对于任何非空引用值x 和y，当且仅当y.equals(x) 返回true 时x.equals(y) 应该返回true。

它也不是反身：

• 它是自反的：对于任何非空引用值x，x.equals(x) 应该返回 true。

Foo foo = new Foo("ID1");
foo.equals(foo)  //false!

@mbockus 提供了equals() 的正确实现：

public boolean equals(Object o){
  if(o instanceof Foo){
    Foo toCompare = (Foo) o;
    return this.id.equals(toCompare.id);
  }
  return false;
}

但现在你必须将Foo 的实例传递给contains()：

System.out.println(fooList.contains(new Foo("ID1")));
System.out.println(fooList.contains(new Foo("ID2")));
System.out.println(fooList.contains(new Foo("ID5")));

最后你应该实现hashCode() 以提供一致的结果（如果两个对象相等，它们必须有相等的hashCode()）：

@Override
public int hashCode() {
    return id.hashCode();
}

Answer 2

您的 equals 方法需要随着覆盖 hashCode() 函数而改变。目前，当您需要检查 Foo 对象时，您正在检查您要比较的对象是否为 instanceof String。

public boolean equals(Object o){
    if(o instanceof Foo){
        Foo toCompare = (Foo) o;
        return this.id.equals(toCompare.id);
    }
    return false;
}

如果您使用的是 Eclipse，我建议让 Eclipse 为您生成 hashCode 和 equals，方法是转到 Source -> Generate hashcode() and equals()...

Answer 3

你应该实现 hashCode

@Override
public int hashCode() {
    return id.hashCode();
}

即使 contains 在没有它的情况下也适用于 ArrayList。您的大问题是您的 equals 需要字符串，而不是 Foo 对象，并且您要求包含字符串。如果实现询问列表中的每个弹出是否等于您发送的字符串，那么您的代码可以工作，但实现询问字符串是否等于您的 Foo 对象，当然不是。

使用等于

@Override
public boolean equals(Object o){
    if(o instanceof Foo){
        String toCompare = ((Foo) o).id;
        return id.equals(toCompare);
    }
    return false;
}

然后检查包含

System.out.println(fooList.contains(new Foo("ID1")));