use else instead of return