Redundant Boolean literals should be removed from expressions to improve readability.
if (booleanMethod() == true) { /* ... */ } if (booleanMethod() == false) { /* ... */ } if (booleanMethod() || false) { /* ... */ } doSomething(!false); doSomething(booleanMethod() == true); booleanVariable = booleanMethod() ? true : false; booleanVariable = booleanMethod() ? true : exp; booleanVariable = booleanMethod() ? false : exp; booleanVariable = booleanMethod() ? exp : true; booleanVariable = booleanMethod() ? exp : false;
if (booleanMethod()) { /* ... */ } if (!booleanMethod()) { /* ... */ } if (booleanMethod()) { /* ... */ } doSomething(true); doSomething(booleanMethod()); booleanVariable = booleanMethod(); booleanVariable = booleanMethod() || exp; booleanVariable = !booleanMethod() && exp; booleanVariable = !booleanMethod() || exp; booleanVariable = booleanMethod() && exp;