Why is this an issue?
If a local variable is declared but not used, it is dead code and should be removed. Doing so will improve maintainability because developers will
not wonder what the variable is used for.
Noncompliant code example
public int numberOfMinutes(int hours) {
int seconds = 0; // seconds is never used
return hours * 60;
}
Compliant solution
public int numberOfMinutes(int hours) {
return hours * 60;
}