java - Java将此类与该类进行相等性比较

我的任务是覆盖方法equals（）。使用Stack12<E> that = (Stack12<E>)o;和o instanceof Stack12有一些问题。我想知道它们是不好的做法，尤其是我在for循环中使用that感觉不适合我吗。

还有另一种方法可以将此类与其他对象进行比较吗？还是我的比较方法足够强大？

  public boolean equals(java.lang.Object o){
  if(o == this) return true;
  if(o == null || !(o instanceof Stack12)){
     return false;
  }

  Stack12<E> that = (Stack12<E>)o;
  if(this.size != that.size || this.capacity != that.capacity ){
     return false;
  }
  for(int i = 0; i < this.size; i++){
     if( that.stack[i] != this.stack[i] ){
        return false;
     }
  }
  return true;
 }

最佳答案

我要补充的一个警告是，每当您覆盖equals(...)时，您也将要覆盖hashCode()。我同意看到instanceof被过度使用使我担心代码的味道，但是我认为您别无选择，只能在这种情况下使用instanceof。至于强制转换为泛型类型，我可能是错的，但是在运行时，泛型确实不存在，因此可能没有意义。

我看到的一个潜在的主要问题是您在for循环中使用了==。如果您的堆栈数组使用对象，则应在循环内使用equals(...)。您的类是泛型的事实表明堆栈数组确实包含对象，但是我不确定，因为我们看不到这一点。