On the fence about this. Something seems slightly odd when the if/elseif conditions are unrelated. But its not a big deal, and I guess the alternative would mean checking the condition twice.