Passing string by value is inefficient, should be ```const string&```.