if you are returning then you don't need the else