else block unneccessary