c++ - 这个算法实现有什么问题 [Sieve of Erathosthene]

Question

我正在尝试实现 Sieve of Eratosthene在 C++ 中。但是，经过多次尝试后，我总是会遇到运行时错误。我认为这与正在使用的迭代器的状态在某处损坏有关。我不能把我的手指放在上面。这是我的代码:

    //Sieves all multiples of  the current sequence element
    bool multiple_sieve(std::list<int>& num_list)
    {
        std::list<int>::iterator list_iter(num_list.begin());
        std::list<int>::reverse_iterator last_element_iter(num_list.rbegin());

        for(std::list<int>::iterator elements_iter(++list_iter);
           elements_iter !=  num_list.end();)
        {
            if((*elements_iter % *list_iter == 0) &&
             (*elements_iter <= *last_element_iter) && (*list_iter != 1))
                num_list.erase(elements_iter);
            else ++elements_iter;
        }
        return true;
    }

    std::list<int>& prime_sieve(std::list<int>& num_list)
    {
        for(std::list<int>::iterator list_iter(num_list.begin());
          list_iter != num_list.end(); ++list_iter)
            multiple_sieve(num_list);
        return num_list;
    }

我做错了什么？什么导致了运行时错误？

更新:当我在测试中运行它时，我收到一条错误消息“列表迭代器不兼容”。

Answer 1

这一行:

num_list.erase(elements_iter);

这会给你带来问题，因为你在迭代列表的同时修改了它。你可以这样做来避免这个问题:

elements_iter = num_list.erase(elements_iter);

预计到达时间:删除了有关 erase() 使其他迭代器无效的内容(看起来它们在这种情况下是安全的)- 只需将 elements_iter 设置为 erase() 的返回值，您就可以开始了。