small fix move out of the loop string constructor