I don't like if else blocks.  And I think "hide everything, show the one you want" is more readable than  "if c then hide show else show hide".  but if you want to leave it the way it is that's fine.