Почему моя функция обрезки не работает?

Question

void trim(string &str) 
{ 
    string::iterator it = str.begin(); 
    string::iterator end = str.end() - 1; 

    // trim at the starting 
    for(; it != str.end() && isspace(*it); it++) 
     ; 
    str.replace(str.begin(), it, ""); 

    // trim at the end 
    for(; end >= str.begin() && isspace(*end); end--) 
     ; 
    str.replace(str.end(), end, ""); // i get the out_of_range exception here 
} 

Я хочу обрезать строку пробелов. Сначала я запускаю пробелы от начала, и он отлично работает, а затем я нахожу положения пробелов с конца и пытаюсь удалить его, и это вызывает исключение.

Почему?

Answer 1

Изменение строки аннулирует итераторы в строку. Один из способов исправить это - изменить строку только один раз. Кстати, это также может быть быстрее:

void trim(std::string &str) 
{ 
    std::string::size_type begin=0; 
    while(begin<str.size() && isspace(str[begin])) 
     ++begin; 
    std::string::size_type end=str.size()-1; 
    while(end>begin && isspace(str[end])) 
     --end; 
    str = str.substr(begin, end - begin + 1) 
} 

Answer 2

Итераторы действительны только до тех пор, пока вы не измените строку. Как только вы меняете строку, конечный итератор наверняка становится недействительным.

Вам необходимо восстановить итераторы после каждого действия, которое могло бы изменить строку.

Answer 3

str.end() не относится ни к какой позиции со строкой; следовательно, если вы попытаетесь разыменовать его, вы получите исключение.

Answer 4

Я предложил бы просто использовать boost::trim

Answer 5

Проблема заключается в том, что вы храните итератор для последнего символа в строке, и использовать его только после того, как вы (потенциально) удалены символы из строки, так что хранимая итератор теперь недействителен (больше не указывая на последний символ).

Это поможет объявить переменные как можно ближе к их первому использованию. В частности, итераторы, как правило, становятся недействительными настолько легко, что вы, как правило, избегаете удерживать их дольше, чем это необходимо.

Итак, инициализируйте конечный итератор после того, как вы удалили ведущие пробелы. (Можно было бы также рассмотреть вопрос о снятии завершающих пробелов первыми.)

---

Могу ли я также предлагаю использовать функцию erase члена для удаления символов из строки.

Могут возникнуть другие ошибки в стирании пробелов, но вы можете понять, когда вы туда попадете.