fixed unnecessary else