why is it bad? one if clause instead of two