Question

让我说我有一个班级

public class Data{
    public int k;
    public int l;
    public Data(int k, int l){
      this.k = k; 
      this.l = l;
    }
    public boolean equals(Date m){
      if(this.k == m.k && this.l = m.l)
           return true;
      return false;
    }
}

我向ArrayList添加了一些Data对象：

ArrayList<Data> holder = new ArrayList<Data>;
Data one = new Data(0,0);
Data two = new Data(0,4);
Data three = new Data(0,5);

为什么indexOf找不到这个？：

holder.indexOf(new Data(0,4)); //returns -1

indexOf是否比自己遍历整个数组列表更好？或者我错过了什么。

Answer 1

indexOf()方法遍历整个列表。以下是Java 7源代码的摘录：

public int indexOf(Object o) {
    if (o == null) {
        for (int i = 0; i < size; i++)
            if (elementData[i]==null)
                return i;
    } else {
        for (int i = 0; i < size; i++)
            if (o.equals(elementData[i]))
                return i;
    }
    return -1;
}

让Java通过它比自己编写更好。只需确保您的equals方法足以找到所需的对象。您还想要覆盖hashCode()。

我不会写出你的equals方法，但我建议你至少：

检查为空
测试您要比较的实例是否相同
您不需要if(boolean_expr) { return true; };只返回布尔表达式。
确保您实际上覆盖了equals方法 - 签名需要Object参数，而不是Date。

Answer 2

equals方法的签名是错误的。您没有覆盖equals中的Object，只是重载它。

要覆盖equals中Object方法的行为，您的签名必须与Object中的签名完全匹配。试试这个：

public boolean equals(Object o) {
    if(!(o instanceof Data)) return false;
    Data other = (Data) o;
    return (this.k == other.k && this.l == other.l);
}

此外，正如其他人所建议的那样，最好覆盖hashCode方法，以使对象在基于地图的集合中正常工作。

Answer 3

Makoto的答案是对的。我会说同样的。但是你的代码中有一些错误。

你写了＃34;公共布尔等于（日期m）{＆＃34;。我想，你的意思是数据而不是日期。
您写了＆＃34; if（this.k == m.k＆amp;＆amp; this.l = m.l）＆＃34;。 if查询中的第二个条件必须是＆＃34; ==＆＃34;。

问题： Makoto的答案是一个解决方案。我的解决方案是使用eclipse的帮助来自动生成哈希码和等于方法。像这样：

public class Data {

    // your class code here

    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + k;
        result = prime * result + l;
        return result;
    }

    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (obj == null) {
            return false;
        }
        if (!(obj instanceof Data)) {
            return false;
        }
        Data other = (Data) obj;
        if (k != other.k) {
            return false;
        }
        if (l != other.l) {
            return false;
        }
        return true;
    }
}

Answer 4

按照惯例，您希望在覆盖等于

时覆盖哈希码

您很可能会发现indexOf使用hashcode方法匹配对象而不是equals

如果您使用eclise编辑代码 - eclipse将从“源”菜单为您生成一个好的equals和hashcode方法。

Java ArrayList IndexOf - 查找对象索引

4 个答案: