表达式可以简化。 Checkstyle的

Question

我该如何简化？变量fuel是来自名为boolean的父类的Vehicle。 useTax()方法是来自同一Vehicle类的抽象方法。

当我开始运行checkstyle时，它会出现字符串“Expression可以简化”。并突出显示if (fuel == true)。

public double useTax() {
    double tax;
    if (fuel == true) { // <-- why?
        tax = value * ALTERNATIVE_FUEL_TAX_RATE;
    }
    else {
        tax = value * TAX_RATE;
    }
    if (value > LUXURY_THRESHOLD) {
        tax += value * LUXURY_TAX_RATE;
    }
    return tax;
}

Answer 1

我认为它告诉你

if (fuel == true)

可以更简洁地写成

if (fuel)

Answer 2

if (fuel == true)

此处您的代码有不必要的boolean冗余。

你可以使用

if (fuel)   

Answer 3

我认为这意味着代替＆＃34; if（fuel == true）＆＃34;，你也可以简化它＆＃34; if（fuel）＆＃34;