我最初是为了测试我想在这里提出的一个基于理论的最佳实践问题,但在这个过程中,我在 java.Set 类中发现了一些有趣的行为。最初,我想知道这种方法的任何潜在缺陷,但现在我发现它根本不起作用,我想知道为什么。
我有一些对象是我的应用程序的数据库对象的容器。这些对象都有唯一integer id
的,并且hashCode()
由equals()
整数 id 定义(用于存储在哈希集中)。
好吧,我希望能够检查一个哈希集是否包含只给定 id 的对象。当然,我可以创建一个对象的新实例并以这种方式进行检查。但是,只是为了好玩,我想看看我是否可以完成它。当然,这对于一个hashmap来说也是微不足道的,所以这真的不是一个重要的问题,只是为了好玩和知识。
因此,我创建了一个类,并尝试调用contains()
,integer
而不是对象的实例。当然,Netbeans 对此给出了一个有趣的警告
Suspicious call to java.util.Collection.contains:
Given object cannot contain instances of int (expected Person)
忽略错误并运行代码,我震惊地发现Java甚至没有调用equals方法。我在我的 equals 方法中放置了 debugSystem.out.println()
来验证,是的,它甚至没有被调用。
在下面发布的代码中,预期的输出应该是(如果我的理论是正确的):
Here Yes Here Yes
或(如果我的理论不正确):
Here Yes Here No
但是,输出是:
Here Yes No
注意,在“No”之前没有“Here”,证明 equals 方法甚至没有被调用。
任何人都可以阐明吗?我总是被告知要添加这个以equals()
提高效率:
if (!(obj instanceof Person))
return false;
但是,如果equals()
在这种情况下甚至没有调用,那将毫无意义。
这是SSCCE:
谢谢你的时间。
import java.util.LinkedHashSet;
import java.util.Set;
/**
*
* @author Ryan
*/
public class Test7 {
public static void main(String[] args) {
class Person {
public final int id;
public final String name;
public Person(int id, String name) {
this.id = id;
this.name = name;
}
@Override
public boolean equals(Object obj) {
System.out.println("Here");
if (this == obj)
return true;
if (obj instanceof Person)
return id == ((Person)obj).id;
else if(obj instanceof Integer)
return id == (Integer)obj;
else {
System.out.println("Returning False");
return false;
}
}
@Override
public int hashCode() {
return id;
}
}
Set<Person> set = new LinkedHashSet<Person>();
set.add(new Person(1, "Bob"));
set.add(new Person(2, "George"));
set.add(new Person(3, "Sam"));
if(set.contains(new Person(1, "Bob")))
System.out.println("Yes");
else
System.out.println("No");
if(set.contains(1))
System.out.println("Yes");
else
System.out.println("No");
}
}