Nested switch
structures are difficult to understand because you can easily confuse the cases of an inner switch
as
belonging to an outer statement. Therefore nested switch
statements should be avoided.
Specifically, you should structure your code to avoid the need for nested switch
statements, but if you cannot, then consider moving
the inner switch
to another function.
Noncompliant code example
func foo(x,y int) {
switch x {
case 0:
switch y { // Noncompliant; nested switch
// ...
}
case 1:
// ...
default:
// ...
}
}
Compliant solution
func foo(x,y int) {
switch x {
case 0:
bar(y)
case 1:
// ...
default:
// ...
}
}
func bar(y int) {
switch y {
// ...
}
}