Redundant statements can be removed RVV-A0010
Anti-pattern
Major
a day ago
—
a year old
Omit unnecessary break at the end of case clause
442
443 switch b.Status {
444 case "created", "running":
445 break446 case "complete":
447 return nil
448 case "failed":
Description
Redundant statements, like a break at the end of a case block, can be removed for improving the code's readability.
Bad practice
loop:
for {
switch a {
case 1:
a++
println("one")
break // unnecessary
case 2:
println("two")
break loop
default:
println("default")
}
}
Recommended
loop:
for {
switch a {
case 1:
a++
println("one")
case 2:
println("two")
break loop
default:
println("default")
}
}