22

有人可以帮我吗？

编译这段代码：

void test()
{
  std::set<int> test;
  test.insert(42);
  test.erase(std::remove(test.begin(), test.end(), 30), test.end());  // <- Line 33
}

编译时产生以下错误：

$ make
g++ -c -Wall -pedantic-errors -Wextra -Wunused -Werror a_star.cpp
/usr/lib/gcc/i686-pc-cygwin/4.3.4/include/c++/bits/stl_algo.h: In function `_FIter std::remove(_FIter, _FIter, const _Tp&) [with _FIter = std::_Rb_tree_const_iterator<int>, _Tp = int]':
a_star.cpp:33:   instantiated from here
/usr/lib/gcc/i686-pc-cygwin/4.3.4/include/c++/bits/stl_algo.h:779: error: assignment of read-only location `__result.std::_Rb_tree_const_iterator<_Tp>::operator* [with _Tp = int]()'
make: *** [a_star.o] Error 1
4

4 回答 4

27

在std::set中，元素不可修改。因此，std::set::iterator也是不可修改的。从本教程第 27.3.2.1 节开始：

在简单的关联容器中，元素是键，元素是完全不可变的；因此嵌套类型 iterator 和 const_iterator 是相同的。

因此，erase-remove不能按原样应用该成语。您必须编写一个for循环，并在其中使用成员函数std::set::erase。有关详细信息，请参阅这个问题和这个接受的答案和另一个答案，但简而言之，循环如下所示

typename std::set::iterator set_iter; 

for( set_iter it = s.begin(); it != s.end(); /* blank */ ) {
    if( some_condition() ) {
        s.erase( it++ );       // Note the subtlety here
    }
    else {
        ++it;
    }
}
于 2010-09-25T05:29:40.533 回答
6

擦除删除习语不能与关联容器一起使用。关联容器不允许通过迭代器修改整个容器元素，这立即意味着std::remove不能对它们应用变异序列操作（如 ）。

于 2010-09-25T05:32:02.880 回答
2

如前所述，您的代码不起作用，因为您尝试修改关联容器内的序列，但您不能这样做，因为该序列是不可变的。基本原理：set 包含一个有序序列，通常在二叉树中。如果你被允许修改它，你可能会破坏容器并且程序会崩溃。顺便说一句，在某些情况下它仍然可能发生。

您可以将代码更改为：

test.erase(30);

或者使用 ArunSaha 的 (+1) 代码来获取更复杂的标准。

于 2010-09-25T08:41:46.550 回答
0

如果我没记错的话，std::remove 永远不会与 std::set 元素一起使用。

由于集合不是纯数组，因此您必须使用擦除。

于 2010-09-25T05:28:30.913 回答