我正在做一个继承问题,除了布尔部分我完成了所有事情。我的代码将编译,但在比较名称,生日和ssn时,它只会显示为false。
例如,我输入了:
输出将为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;
}
}
}
答案 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
),变量的值是它实际存储的值。但是,对于对象变量(例如String
,Date
),该值实际上是对对象的引用。如果两个对象变量引用堆上的同一个对象,它们将具有相同的值;如果它们引用堆上的不同对象,它们将具有不同的值。
==
运算符只检查变量的值,没有别的。当它与基元一起使用时,它会检查存储在变量中的值是否相同,这是正确的,因为这些值是直接存储的。与对象变量一起使用时,它会检查两个变量是否引用堆上的同一对象。由于变量的值实际上是引用,因此当您检查两个值或引用是否相同时,这是结果。
另一方面,equals
方法实际上(应该)进一步检查两个对象是否相等,因为它们的状态是相同的。例如,它可以检查两个Date
对象是否代表相同的日期。 Java无法神奇地判断两个不同的对象是否相等 - 该类必须实现equals
方法本身。 (否则,它会转到默认实现,只是检查引用。)
在上面的代码示例中,您正在原始==
上使用ssn
运算符(因为它是一个原语),而equals
方法正确地使用了对象{{ 1}}(因为每个name
中的引用可能不同,但表示相同的字符串)。但是,您在对象变量Person
上使用==
运算符,它只检查被比较的两个birthday
对象是否具有相同的Person
对象。可能,因为您的调用代码只是在每个输入上创建一个新的Date
对象,所以检查失败,因为两个Date
对象有两个不同的Person
对象。只需将其更改为birthday
方法调用,您的代码就可以正常工作。