Should this be outside the if?