Question

我的Fraction程序运行顺利，但NetBeans IDE告诉我以下if是多余的：

public boolean equals(Object other)
{
  Fraction bool = (Fraction) other;

  if(this.numerator == bool.numerator && this.denominator == bool.denominator)
  {
   return true;
  }
  else return false;       
}

上面的代码编译/运行完美并传递了所有测试用例但是NetBeans的冗余标志真的很困扰我。我将reduceToLowestTerms()添加到我的代码中，然后标志消失，但我的构造函数中已经有reduceToLowestTerms()。这就是非冗余代码（根据NetBeans）的样子：

public boolean equals(Object other)
{
        Fraction bool = (Fraction) other;

        if(this.numerator == bool.numerator && this.denominator == bool.denominator)
        {
         bool.reduceToLowestTerms();
         this.reduceToLowestTerms();
         return true;
        }
         else return false;       
    }

任何建议都将不胜感激

Answer 1

这看起来类似于我的IDE在此声明中发出的警告：

＆＃39;如果＆＃39;声明可以简化
if(foo())
{
   return true;
}
else
{
   return false;
}
可以简化为
return foo();

它过于复杂和冗长的代码。您的简化将是：

return this.numerator == bool.numerator && this.denominator == bool.denominator;

但正如您所注意到的，您的代码已经正确无误。没有必要进行此更改，但它会使代码更简洁，更简化。

添加对另一个方法（reduceToLowestTerms()）的调用的原因会删除此＆＃34;标记＆＃34;是这样的代码不能再以这种方式简化为单个return语句。

冗余如果消息

1 个答案: