Question

我的代码有一个奇怪的问题。

下面我用以下代码测试Chunk类的代码：

List<Chunk> chunks = new ArrayList<Chunk>();
chunks.add(new Chunk(1,1,1));
System.out.println(chunks.indexOf(new Vector3i(1, 1, 1)));

这是Chunk类＆＃39;等于方法：

public boolean equals(Object object) {
    System.out.println("Test _1_");
    if (object != null && object instanceof Vector3i) {
        System.out.println("Test _2_");
        if((this.x == ((Vector3i) object).x)&&(this.y == ((Vector3i) object).y)&&(this.z == ((Vector3i) object).z)) {
            System.out.println("Test _3_");
            return true;
        }
    }
    System.out.println("Test _4_");

    return false;
}

Vector3i：

public class Vector3i {
    public int x;
    public int y;
    public int z;


    public Vector3i(int x, int y, int z) {
        this.x = x;
        this.y = y;
        this.z = z;
    }

}

当我运行它时，它只返回-1。来自equals方法的测试打印不打印，这意味着它甚至没有开始执行。那是为什么？

Answer 1

如果您检查ArrayList.indexOf实施，您会看到在您的情况下调用了Vector3i.equals。实际上它甚至在JavaDoc中为List指定：

更正式地说，如果没有这样的索引，则返回最低索引i，如(o==null ? get(i)==null : o.equals(get(i)))或-1。

一般来说，equals操作必须是对称的：{{1}}。所以你必须实现a.equals(b) == b.equals(a)。

另请注意，您当前的Vector3i.equals实施缺乏反身性等其他属性。另外，请考虑在实施equals时实施hashCode。

Answer 2

chunks.indexOf(new Vector3i(1, 1, 1)

在Vector3i类上调用equals（）方法，但不要在Chunk类上调用...

Java ArrayList indexOf返回-1

2 个答案: