oops use string comparison