if you return you dont need an else