Wouldn't it make the code more readable (less fragmented) if we stopped doing the "early" return when != ACTIVATED and put the different code for ACTIVATED and DEACTIVATED to the bottom?
Wouldn't it make the code more readable (less fragmented) if we stopped doing the "early" return when != ACTIVATED and put the different code for ACTIVATED and DEACTIVATED to the bottom?