use void instead of !