G-4250: Avoid using identical conditions in different branches of the same IF or CASE statement.
Blocker
Maintainability, Reliability, Testability
Reason
Conditions are evaluated top to bottom in branches of a case
statement or chain of if
/elsif
statements. The first condition to evaluate as true leads to that branch being executed, the rest will never execute. Having an identical duplicated condition in another branch will never be reached and will be dead code.
Example (bad)
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 |
|
Example (good)
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 |
|