Удалить все '/' из всех std::string содержащихся в std::vector

@lukinegor · Регистрация: 31.01.2020

Author24 — интернет-сервис помощи студентам

Нужно из всех слов содержащихся в векторе убрать все передние слеши.

C++

void clear_words(std::vector<std::string> &words)
{
    for(int i = 0; i < words.size(); i++)
    {
        for(int t = 0; t < (words[i]).size(); t++)
        {
            if( (words[i])[t] == '/')
            {
                (words[i]).erase(t);
            }
        }
    }
}

Простая функция, но почему-то те слова, в которых есть '/', становятся пустыми, работает не так, как надо.

Не по теме:

upd:
там запятая пропущена в заголовке темы

Алексей1153 · 21.06.2020, 21:56

lukinegor, смещение итератора нужно тоже учитывать

C++

...
        for(auto it=words[i].begin(); it!=words[i].end(); /*it++*/)
        {
            if( *it == '/')
            {
                it=words[i].erase(it);
            }
            else
            {
               it++;
            }
        }
...

@DrOffset · 21.06.2020, 21:58

lukinegor,

C++

void clear_words(std::vector<std::string> &words)
{
    for(size_t i = 0; i < words.size(); i++)
    {
        std::string & current = words[i];
        
        current.erase(std::remove_if(current.begin(), current.end(), [](char c) { return c == '/'; }));
    }
}

@nonedark2008 · 21.06.2020, 22:01

Сообщение от lukinegor

(words[i]).erase(t);

(words[i]).erase(t, 1);
Если был удален символ, то не нужно делать t++.

@lukinegor · 21.06.2020, 22:06 **[ТС]**

Алексей1153, nonedark2008, спасибо большое! Совсем забыл про это

Добавлено через 1 минуту
DrOffset, зачем нужен current, если можно обращаться напрямую к элементу вектора?
upd: для читабельности?

Алексей1153 · 21.06.2020, 22:11

lukinegor, так через ссылку и будет прямое обращение. Кстати, вариант с std::remove_if лучше, чем "вручную" итераторы перебирать

@DrOffset · 21.06.2020, 22:16

Сообщение от lukinegor

зачем нужен current, если можно обращаться напрямую к элементу вектора?
upd: для читабельности?

Чтобы не делать три раза words[i].

Добавлено через 25 секунд

Сообщение от lukinegor

можно обращаться напрямую к элементу вектора?

Это и так и так прямое обращение.

Удалить все '/' из всех std::string содержащихся в std::vector

Решение

Решение

Решение

@nonedark2008 1394 / 1023 / 325 Регистрация: 28.07.2012 Сообщений: 2,813
	21.06.2020, 22:01	4
	Сообщение было отмечено lukinegor как решение Решение Сообщение от lukinegor (words[i]).erase(t); `(words[i]).erase(t, 1);` Если был удален символ, то не нужно делать `t++`. 0

@lukinegor 1 / 1 / 0 Регистрация: 31.01.2020 Сообщений: 148
	21.06.2020, 22:06 [ТС]	5
	Алексей1153, nonedark2008, спасибо большое! Совсем забыл про это Добавлено через 1 минуту DrOffset, зачем нужен current, если можно обращаться напрямую к элементу вектора? upd: для читабельности? 0

Алексей1153 фрилансер 5497 / 5093 / 1047 Регистрация: 11.10.2019 Сообщений: 13,321
	21.06.2020, 22:11	6
	lukinegor, так через ссылку и будет прямое обращение. Кстати, вариант с std::remove_if лучше, чем "вручную" итераторы перебирать 0

@DrOffset 18829 / 9832 / 2403 Регистрация: 30.01.2014 Сообщений: 17,267
	21.06.2020, 22:16	7
	Сообщение от lukinegor зачем нужен current, если можно обращаться напрямую к элементу вектора? upd: для читабельности? Чтобы не делать три раза `words[i]`. Добавлено через 25 секунд Сообщение от lukinegor можно обращаться напрямую к элементу вектора? Это и так и так прямое обращение. 0

Опции темы