Si se trata de asignar/quitar clases con métodos de jQuery, está bien así, aunque puedes usar el método
toggleClass, con el cual asignas la clase al elemento si no la tiene y se la quitas, si la tiene. De esta manera, evitarías el uso de condiciones.
Saludos