2014-11-13 2 views
2

Как я могу это упростить? Переменная является boolean из родительского класса с именем Vehicle. Метод useTax() является абстрактным методом из того же класса Vehicle.Выражение может быть упрощено. Checkstyle

Когда я иду на запуск контрольного стиля, он появляется со строкой «Выражение может быть упрощено». и подчеркивает 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; 
} 
+2

удалить '== true'. При проверке логических значений вы можете использовать 'if (bool)', чтобы проверить, является ли его true, и 'if (! Bool)', чтобы проверить, является ли его false –

ответ

3

Я думаю, что это говорит вам, что

if (fuel == true) 

можно записать более кратко, как

if (fuel) 
1

Я думаю, что это означает, а не «если (топливо == верно)» , вы можете упростить его также «если (топливо)»

2
if (fuel == true) 

Здесь у вашего кода нет необходимости boolean избыточности.

Вы можете просто использовать

if (fuel)