где я ошибаюсь с этой функцией?

Question

Итак, я должен переместить все числа палиндромов из вектора mValues ​​в вектор mPalindromes

, и единственными подсказками, которые мне дали, был псевдокод: // итерация по вектору основных значений // если значение это палиндром // добавить его в вектор палиндрома // удалить его из вектора значений

, и вот что я придумал:

for (int i = 0; i < mValues.size(); i++)
    {
        if (IsPalindromeNumber(mValues[i]) )
        {
            mPalindromes.push_back(mValues[i]);

            mValues.pop_back();

            
        }
    }

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

edit: Я понял, что проблема заключалась в том, что я использовал стирание вместо всплывающего окна.

Answer 1

Когда вы выполняете

mValues.erase(mValues.begin() + i); // with correction from comments

, вы изменяете mValues, поэтому i начинает индексирование в другом месте в mValues. Одно исправление было бы

mValues.erase(mValues.begin() + i);
i--;

, но оно не очень элегантно. Было бы лучше скопировать палиндромы в mPalindromes, а затем использовать std::remove_if или что-то в этом роде во втором проходе.