重写equals方法问题

Question

我正在尝试创建一个有理数类并覆盖等于和哈希代码方法。但是我的等号在明显不正确的情况下又回来了，即分子和分母不同。知道是什么原因造成的吗？

public boolean equals(Object rhs) {
    if (this == rhs){
        return true;
    }
    if (rhs == null){
        return false;
    }
    if (!(rhs instanceof Rational)){
        return false;
    }
    Rational other = (Rational) rhs;
    if (denom == other.denom){
        if (num == other.num);{
            return true;
        }
    }
    return false;
}

Answer 1

删除；后if（num==other.num ); {，将其更改为if（num==other.num）{

把它留在那里，它在if之后基本上什么也不做，然后进入块：

{
    return true;
}

所以在那一点上它总是会返回真的。

Answer 2

删除此行上的分号，它充当if语句的主体。

if (num == other.num);{

使用分号，如果分母相等，则返回true；分子的检查被有效忽略。

Answer 3

这是问题（如果不是打字错误）：

if (num == other.num);{

分号意味着if语句是一个空语句，因此它的求值并没有真正参与等于验证过程。只需去掉分号：

if (num == other.num){