unecessary else