since you're returning, no need for the rest of this to be in an else block