used assign to fill string instead of loop