Why is this an issue?
Redundant Boolean literals should be removed from expressions to improve readability.
Noncompliant code example
if (booleanMethod() || false) { /* ... */ }
doSomething(!false)
booleanVariable = if (booleanMethod()) true else false
booleanVariable = if (booleanMethod()) true else exp
booleanVariable = if (booleanMethod()) false else exp
booleanVariable = if (booleanMethod()) exp else true
booleanVariable = if (booleanMethod()) exp else false
Compliant solution
if (booleanMethod()) { /* ... */ }
doSomething(true)
booleanVariable = booleanMethod()
booleanVariable = booleanMethod() || exp
booleanVariable = !booleanMethod() && exp
booleanVariable = !booleanMethod() || exp
booleanVariable = booleanMethod() && exp
