shouldn't this be else if?