Question

可能重复：
Why I’m getting StackOverflowError

我正在使用两个课程：日期和考试。 Date从三个整数设置Date对象：day，month，year;考试从一个String courseName和一个Date对象设置Exam对象。

我正在尝试运行此代码：

    public Exam(String name, Date d)
    {
        courseName=name;
        examDate=new Date(d);
    }

    //**a method that checks if two dates are equal**
    public boolean equals (Date r)
    {
        return (examDate.equals(r));
    }

    public static void main(String[] args)
    {
        Date d=new Date(11,11,2011);
        String a=new String("OOP");
        Exam b=new Exam(a,d);
        Date c=new Date(11,11,2011);
        System.out.println(b.equals(c));
    }

当我尝试运行代码时，我在线程“main”中遇到错误异常java.lang.StackOverflowError

错误说问题出现在Date类的一行上，它检查两个日期是否相等：

public boolean equals (Date d)
{
    return (this.equals(d));
}

我会很感激知道为什么会这样。

Answer 1

boolean equals(Date d) {
  this.equals(d); 
}

无论这个类是什么，equals方法都是完全错误的 - 它只是调用自身，而后者又调用自身，然后调用自身，无限递归，直到得到StackOverflowError。

首先：覆盖等于的正确签名是：

boolean equals(Object obj)

具有特定于类的覆盖只是......奇怪的。看起来似乎有人试图委托默认的equals（）方法，但这不是那样做的。

如果这是一个自定义Date类，则equals应如下所示：

boolean equals(Object obj) {
  if (!obj instanceof Date) {
    return false;
  }
  Date other = (Date) obj;
  return this.field1.equals(date.field1) && this.field2.equals(date.field2)...... ;
}

此外，在您实施equals（）时，立即开始实施hashCode。这会让你感到悲伤。

Why should I override hashCode() when I override equals() method?

Answer 2

正如詹姆斯所说，return (this.equals(d));肯定是错的。假设您的Date班级具有年，月和日的属性，您应该尝试类似

的内容

public boolean equals(Object o) {
    if (! o instanceof Date) return false;
    Date d = (Date)o;
    return this.year == d.year && this.month == d.month && this.day == d.day;
}

请注意，参数的静态类型为Object非常重要，请参阅James的答案

的StackOverflowError

2 个答案: