我正在尝试实现 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;
}
我做错了什么?什么导致了运行时错误?
更新:当我在测试中运行它时,我收到一条错误消息“列表迭代器不兼容”。
最佳答案
这一行:
num_list.erase(elements_iter);
这会给你带来问题,因为你在迭代列表的同时修改了它。你可以这样做来避免这个问题:
elements_iter = num_list.erase(elements_iter);
预计到达时间:删除了有关 erase() 使其他迭代器无效的内容(看起来它们在这种情况下是安全的)- 只需将 elements_iter 设置为 erase() 的返回值,您就可以开始了。
关于c++ - 这个算法实现有什么问题 [Sieve of Erathosthene],我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/1127617/