Не удаляет int в векторе через итератор, как исправить?

Question

[Loligan]

Не удаляет =(

vector friends;

bool function::del_friend(human *hmn, int id_user)
{
cout<<"START "<friends.size()<friends.size()==0)
{
return false;
}
for(vector::iterator it=hmn->friends.begin(); it != hmn->friends.end(); ++it)
{
if(id_user==*it)
{
hmn->friends.erase(it);
it--;
}
}
cout<<"DELETE OK "<friends.size()<

Answer 1

[Дмитрий]

Ну прежде всего erase возвращает итератор на елемент, следующий за удаленным
it = hmn->friends.erase(it);
Вариант с --it не валиден.

Во вторых лаконичнее будет воспользоваться erase-remove идиомой https://en.wikipedia.org/wiki/Erase–remove_idiom
там есть пример на ваш случай

Comments

[Loligan]

Немного не понял конечно как ей пользоваться. Я попытался сделать так но он неправильный

hmn->friends(remove(hmn->friends.begin(),hmn->friends.end(),id_user),hmn->friends.end());

[Loligan]

Сделал и не удаляет
hmn->friends.erase(remove(hmn->friends.begin(),hmn->friends.end(),id_user), hmn->friends.end());

Answer 2

[Dvvarreyn]

{
return false;
}

Это что?
Или из-за оформления там if побился?
При текущем оформление вообще не очевидно, что до цикла функция может дойти.

Comments

[Loligan]

Да, if ушел. Там была проверка на размер вектора. Если равен нулю то возвратить false

Answer 3

[Алексей Рытиков]

Оформите нормально код и поставьте вывод внутри условия с удалением

if(id_user==*it)
{
    cout << "erase";
    hmn->friends.erase(it);
    it--;
}

Не удаляет или не находит?

Comments

[Loligan]

Ненаходит