4

我最初是为了测试我想在这里提出的一个基于理论的最佳实践问题,但在这个过程中,我在 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");
        }
    }
4

1 回答 1

5

这是因为比较是在提供的对象而不是集合中的元素上完成的。从HashSet#contains(Object)

如果此集合包含指定的元素,则返回 true。更正式地说,当且仅当这个集合包含一个元素 e 使得(o==null ? e==null : o.equals(e))时返回 true 。

因此,在您的示例中,您将进行比较,例如 integer.equals(person). 因此,如果您的集合包含Person对象,if(obj instanceof Integer)则永远不会检查条件,但如果您的集合包含 Integer对象,则将满足该条件并因此进行检查。

于 2013-06-03T01:16:03.400 回答