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

C++ "vector iterator not decrementable"?

本文关键字：迭代器 向量 C++      更新时间：2023-10-16

我正试图解决这个UVa问题。

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

提前谢谢。

这是我的代码

#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;
}

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

如果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());
相关文章：