Warum ist string :: substr schlecht?

Question

diesen Code Gegeben:

auto str = "a,b,c"s; 
vector<string> tokens; 
string::size_type start = -1; 
string::size_type finish = str.find(','); 

do { 
    tokens.push_back(str.substr(start + 1, finish)); 
    start = finish; 
    finish = str.find(',', start + 1); 
} while (start != string::npos); 

Ich würde tokens erwarten enthalten:

1. "a"
2. "b"
3. "c"

Wenn ich laufen der Code jedoch bekomme ich:

1. "a"
2. "b, c"
3. "c"

Ich habe in diesem anschaulichen Beispiel Debug-Informationen ausgedruckt: http://ideone.com/fx3uC9

Ist das ein Fehler? Ich sehe es in gcc und Visual Studio, also scheint es, als hätte ich etwas falsch gemacht.

Answer 1

Wenn Sie die Dokumentation für substr() lesen, die Signatur:

basic_string substr(size_type pos = 0, 
        size_type count = npos) const; 

Das zweite Argument nicht die Position des Endes ist (wie jede andere Funktion in <algorithm>, die eine Reihe nimmt), dann ist es die Länge der Teilstring. Also wo Sie finish als das Ende haben, brauchen Sie stattdessen finish - (start + 1) als die Länge.