ЗВ
И "111/1"
Size: a a a
2020 August 25
IM
Захар Виноградов
Expression: string subscript out of range
t может быть меньше нуля
ЗВ
Это я уже исправил, не помогло.
IM
Захар Виноградов
Это я уже исправил, не помогло.
D
Вообще всю функцию можно заменить на:
size_t last_slash_index = file.find_last_of('/');
std::string newFile = file.substr(0, last_slash_index);
return newFile;ПК
Захар Виноградов
Попробовал "1111"
Ну это невалидная строка, с ней будет падать программа. Слэш всегда должен быть
IM
Вообще всю функцию можно заменить на:
size_t last_slash_index = file.find_last_of('/');
std::string newFile = file.substr(0, last_slash_index);
return newFile;Проверку last slash index надо
D
Проверку last slash index надо
Ну я просто отрефакторил, но не исправил ошибки :)
ПК
Проверку last slash index надо
да вроде не надо
D
да вроде не надо
Кстати да
IM
да вроде не надо
Npos
D
Npos
Ну и?
ПК
Npos
substr умный
D
Оно просто всю строку тогда скопирует
IM
А точно. Извиняюсь
ЗВ
Вообще всю функцию можно заменить на:
size_t last_slash_index = file.find_last_of('/');
std::string newFile = file.substr(0, last_slash_index);
return newFile;Спасибо. Серьезно не хватает знания таких функций, пытаюсь их по-своему реализовать)
ЗВ
Все же интересно, почему код не работал.
D
Захар Виноградов
Все же интересно, почему код не работал.
Потому что ты выходил за границы строки
Тебе же написали
Тебе же написали
ЗВ
Все, понял, спасибо за помощь.
D
Захар Виноградов
Все, понял, спасибо за помощь.
Твоя функция закладывалась на знание о том, что в строке есть хотя бы один /
Если его нет в строке, то ты пытался обратиться к -1 элементу строки
Если его нет в строке, то ты пытался обратиться к -1 элементу строки