这种算法实现有什么问题[Erathosthene筛]

Question

我正在尝试用C++ 实现Eratosthene筛选.但是经过多次尝试后,我总是遇到运行时错误.我认为这与使用的迭代器的状态在某处被破坏有关.我不能把手指放在它上面.这是我的代码:

    //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);

ETA:删除了擦除()使其他迭代器失效的东西(在这种情况下看起来很安全) - 只需将elements_iter设置为erase()的返回值,你就应该好了.