C++ “向量迭代器不可递减”？

Question

我正在尝试解决 这个 UVa 问题。

我正在尝试使用 Vector 来解决这个问题。我需要模拟循环链表之类的东西，所以我使用迭代器来访问元素。但经过尝试，我发现 Vector 迭代器在递增和递减方面存在一些问题，并且我无法通过使用反向迭代器作为参数来删除元素。我现在很困惑。我的代码是否有任何问题，因为我错过了一些重要的细节，或者我应该以其他方式解决这个问题？

提前致谢。

这是我的代码

#include <iostream>
#include <vector>
#include <iomanip>

using namespace std;

vector<int> people;

int main()
{
    int n, k, m;            // k -> counter clockwise, m -> cloclwise
    while (cin >> n >> k >> m)
    {
        if (n == 0 && k == 0 && m == 0)
            return 0;
        for (int i = 1; i <= n; i++)
            people.push_back(i);
        vector<int>::iterator k_pos = people.begin();
        vector<int>::reverse_iterator m_pos = people.rbegin();

        //cout << n << " " << k << " " << m << endl;

        while (!people.empty())
        {
            int k_choose, m_choose;
            for (int i = 1; i < k; i++)
            {
                k_pos++;
                if (k_pos == people.end()) // if reach the end, go to begin
                    k_pos = people.begin();
            }

            k_choose = *k_pos;
            cout << k_choose << endl;

            for (int i = 1; i < m; i++)
            {
                m_pos++;
                if (m_pos == people.rend())
                    m_pos = people.rbegin();
            }

            m_choose = *m_pos;


            if (k_choose == m_choose)
            {
                cout << setw(3) << k_choose << ",";
                people.erase(k_pos);                 // erase the element
            }

            else
            {
                cout << setw(3) << k_choose << setw(3) << m_choose << ",";
                k_pos = people.erase(k_pos);         // erase the element
                //vector<int>::iterator temp;
                //for (temp = people.begin(); *temp != *m_pos; temp++)
                //{
                //}
                //cout << "ok" << endl;
                people.erase(--m_pos.base());*****problem

            }
            vector<int>::iterator temp;
            for (temp = people.begin(); temp != people.end(); temp++)
                cout << *temp << endl;

            k_pos++;                              *****problem
            if (k_pos == people.end())          // point to next
                k_pos = people.begin();

            m_pos++;                              *****problem
            if (m_pos == people.rend())         // point to next
                m_pos = people.rbegin();
        }
    }
    return 0;
}

Answer 1

擦除或推入向量后，它的所有迭代器可能会变得无效（如果向量被重新分配）。这就是为什么在 else 中执行擦除后 m_pos 可能会变得无效。我的建议是使用索引（至少这是我在竞争性编程中所做的）。

Answer 2

如果m_pos等于people.rbegin()怎么办？
因此 --m_pos 不是有效的迭代器，这可能是问题的根源。

您也有可能删除了前一行中由 --m_pos 指向的元素：

k_pos = people.erase(k_pos);         // erase the element

您可以在代码中改进的另一件事是设置 k_pos 和 m_pos 迭代器以更有效的方式。相反：

for (int i = 1; i < k; i++)
{
    k_pos++;
    if (k_pos == people.end()) // if reach the end, go to begin
    k_pos = people.begin();
}

你可以写：

#include <iterator>
std::advance(people.begin(), k % people.size());