测试平等的充分方法

Question

我想为我所拥有的类实现一个自定义的equals()方法， Board 。 该方法比较每个板的数组，定义为private int[] board ，如果数组相等则返回true，否则返回false。 我知道在测试相等性方面有一些“陷阱”，所以我想知道以下代码是否是最佳且足以真正测试相等性的：

public boolean equals(Object y) {
    if (this.getClass() != y.getClass()) return false; //must be same class -- duh
    Board that = (Board) y; //y cast as Board
    int[] thisBoardCopy = this.getBoard(); //copy of current board
    int[] thatBoardCopy = that.getBoard(); //copy of y's board
    return Arrays.equals(thisBoardCopy, thatBoardCopy);
}

Answer 1

在java中编写.equals方法的常用习惯是：

public boolean equals(Object y) {
    if(y == this) return true;
    if(!(y instanceof Board.class)) return false;
    final Board that = (Board) y; //y cast as Board
    return Arrays.equals(getBoard(), that.getBoard());
}

第一个测试只是加速，如果它是同一个Board ，第二个测试有两个功能：

1. 如果y为null ，则返回false - 这会减少代码量
2. 它做你的支票， y是正确的类。

编辑

我不确定你的意见中的“复制”是什么意思，我认为你的意思是“参考”。 如果你在将它们传递给equals之前复制这些数组，我建议你不要这样做，因为如果这个对象进入Map或Set ，这个方法可以被调用很多次。

Answer 2

你最好做

if (!this.getClass().equals (y.getClass())) return false;

如果y为null ，则会出现NullPointerException。

不，这仍然会引发NPE。 应该：

if (y == null || !this.getClass().equals (y.getClass())) return false;