以下是从对象列表中查找重复对象的代码。但由于某种原因,哈希集甚至存储了相同的对象。
我当然错过了一些东西,但是当我检查hashset的大小时,它就会出现5。
import java.util.ArrayList;
import java.util.HashSet;
public class DuplicateTest {
public static void main(String args[]){
ArrayList<Dog> dogList = new ArrayList<Dog>();
ArrayList<Dog> duplicatesList = new ArrayList<Dog>();
HashSet<Dog> uniqueSet = new HashSet<Dog>();
Dog a = new Dog();
Dog b = new Dog();
Dog c = new Dog();
Dog d = new Dog();
Dog e = new Dog();
a.setSize("a");
b.setSize("b");
c.setSize("c");
d.setSize("a");
e.setSize("a");
dogList.add(a);
dogList.add(b);
dogList.add(c);
dogList.add(d);
dogList.add(e);
if(a.equals(d)){
System.out.println("two dogs are equal");
}
else System.out.println("dogs not eqal");
for(Dog dog : dogList){
uniqueSet.add(dog);
}
System.out.println("number of unique dogs="+ uniqueSet.size());
/*for(Dog dog:uniqueSet){
System.out.println("uniqueset ="+dog.getSize());
}
for(Dog dog : duplicatesList){
System.out.println("duplicate dog="+dog.getSize());
}*/
}
}
这是狗类
public class Dog implements Animal, Comparable<Dog>{
String size;
public void makeNoise(){
System.out.println("woof woof");
}
public String getSize() {
return size;
}
public void setSize(String size) {
this.size = size;
}
public int compareTo(Dog d){
return this.size.compareTo(d.size);
}
public boolean equals(Dog d){
return this.size.equals(d.size);
}
@Override
public int hashCode() {
// TODO Auto-generated method stub
return super.hashCode();
}
}
答案 0 :(得分:8)
此代码无法满足您的需求:
public boolean equals(Dog d){
return this.size.equals(d.size);
}
这不是重写Object.equals,这是HashSet使用的。你需要:
@Override
public boolean equals(Object d){
if (!(d instanceof Dog)) {
return false;
}
Dog dog = (Dog) d;
return this.size.equals(dog.size);
}
请注意,通过使用@Override
注释,您要求编译器验证您实际上正在覆盖方法。
编辑:如上所述, 还需要以与hashCode
方法兼容的方式覆盖equals
。鉴于您正在根据大小检查相等性,最简单的选项是:
@Override
public int hashCode() {
return size.hashCode();
}