why isn't this an else?