UseEnumSwitch
Prefer using a switch instead of a chained if-else for enums

The problem

Consider using switch instead of if/else for enums. That is, prefer this:

switch (foo.getBar()) { case BAZ: doSomething(); break; default: doSomethingElse(); } 

instead of this:

if (foo.getBar().equals(Bar.BAZ)) { doSomething(); } else { doSomethingElse(); } 

Switches on enums have a few small advantages worth considering:

• It sidesteps the equals vs. == debate.
• You get to call BAZ by its simple name without a static import. That’s good because you might not want to see it appear unqualified in other parts of the file where the context would not make its meaning so clear.
• You have the option of [protection against missing cases][MissingCasesInEnumSwitch] if you want it.

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("UseEnumSwitch") to the enclosing element.