在Java中为继承编写equals()方法

时间:2017-12-10 23:53:42

标签: java inheritance boolean

我正在做一个继承问题,除了布尔部分我完成了所有事情。我的代码将编译,但在比较名称,生日和ssn时,它只会显示为false。

例如,我输入了:

  • 吉姆
  • 6 30 2001
  • 123456789
  • 吉姆
  • 6 30 2001
  • 123456789

输出将为false。

public class Person
{
    private String name;
    private Date birthday;
    private int ssn;


    public Person(String name, Date birthday, int ssn)
    {
        this.name = name;
        this.birthday = birthday;
        this.ssn = ssn;
    }

    public String getName()
    {
        return name;
    }

    public Date getBirthday()
    {
        return birthday;
    }


    public int getSSN()
    {
        return ssn;
    }
     /* I already called a dates toString() method for birthday */
    public String toString() 
    {
        return name + " has birthday " +  birthday  + " and SSN " + ssn;
    }


    public boolean equals(Object otherObj)
    {
        if(otherObj == null)
            return false;
        else if(otherObj.getClass() != this.getClass())
            return false;
        else
        {
            Person otherP = (Person)otherObj;
            if(otherP.name.equals(this.name) &&
            otherP.birthday == this.birthday &&
            otherP.ssn == this.ssn)
                return true;
            else
                return false;
        }
    }
}

2 个答案:

答案 0 :(得分:1)

这里没有继承权。实际上,在从超类继承时编写equals方法会引发围绕类设计的完全不同的讨论(更喜欢组合而不是继承)。

在这种情况下,我认为您需要遵循经验法则,即始终使用equals方法对所有字段进行逻辑比较。 ==检查用于身份比较,很少能用于逻辑相等(例如枚举)

需要注意的另一点是,您的课程会因使用散列的数据结构行为不当而发生错误。

Joshua Bloch的Effective Java第3章详细介绍了这个主题

答案 1 :(得分:1)

我认为问题在于:

if(otherP.name.equals(this.name) &&
        otherP.birthday == this.birthday &&
        otherP.ssn == this.ssn)

上面的代码是检查两个Person对象是否相等的主要代码。您说当您提供两个具有完全相同的生日,名称和SSN的对象时,就会出现问题。我不确定您的输入代码是如何创建对象的,但我可以告诉它可能会为每个生日创建一个新日期。

==运算符和equals方法之间存在重要差异。任何类型的变量都包含一个值。对于原始数据类型(例如int),变量的值是它实际存储的值。但是,对于对象变量(例如StringDate),该值实际上是对对象的引用。如果两个对象变量引用堆上的同一个对象,它们将具有相同的值;如果它们引用堆上的不同对象,它们将具有不同的值。

==运算符只检查变量的值,没有别的。当它与基元一起使用时,它会检查存储在变量中的值是否相同,这是正确的,因为这些值是直接存储的。与对象变量一起使用时,它会检查两个变量是否引用堆上的同一对象。由于变量的值实际上是引用,因此当您检查两个值或引用是否相同时,这是结果。

另一方面,equals方法实际上(应该)进一步检查两个对象是否相等,因为它们的状态是相同的。例如,它可以检查两个Date对象是否代表相同的日期。 Java无法神奇地判断两个不同的对象是否相等 - 该类必须实现equals方法本身。 (否则,它会转到默认实现,只是检查引用。)

在上面的代码示例中,您正在原始==上使用ssn运算符(因为它是一个原语),而equals方法正确地使用了对象{{ 1}}(因为每个name中的引用可能不同,但表示相同的字符串)。但是,您在对象变量Person上使用==运算符,它只检查被比较的两个birthday对象是否具有相同的Person对象。可能,因为您的调用代码只是在每个输入上创建一个新的Date对象,所以检查失败,因为两个Date对象有两个不同的Person对象。只需将其更改为birthday方法调用,您的代码就可以正常工作。