use string instead of char