复制对的矢量

时间:2011-06-12 13:45:53

标签: c++ vector c++11

我正在尝试将一对矢量复制到另一个:

vector<pair<int,int>> vp = {pair<int,int>(1,1), pair<int,int>(2,2)};
vector<pair<int,int>> vp2;
for_each(vp.begin(), vp.end(), [vp2](pair<int,int> p){
                                               if(/*some condition*/){
                                                vp2.push_back(p);
                                               }
                                              });

我收到此编译器错误:

error: passing ‘const std::vector<std::pair<int, int> >’ as ‘this’ argument of ‘void std::vector<_Tp, _Alloc>::push_back(value_type&&) [with _Tp = std::pair<int, int>, _Alloc = std::allocator<std::pair<int, int> >, value_type = std::pair<int, int>]’ discards qualifiers

在ubuntu上使用gcc 4.5.1。

2 个答案:

答案 0 :(得分:7)

作为康拉德回答的替代方案;如果目标只是复制第一个向量的元素,如果它们匹配某个条件,那么std::copy_if不是更好的匹配吗?

vector<pair<int,int>> vp = {pair<int,int>(1,1), pair<int,int>(2,2)};
vector<pair<int,int>> vp2;

copy_if(vp.begin(), vp.end(), back_inserter(vp2), [](pair<int,int> p) { return /* some condition */; });

答案 1 :(得分:5)

复制比这容易得多:

vector<pair<int,int>> vp2(vp.begin(), vp.end());

甚至:

vector<pair<int,int>> vp2 = vp; // or
vector<pair<int,int>> vp2(vp);

您的代码中的错误是您通过值捕获vp2 ,这有效地使您的匿名方法成为const,并且您无法调用push_back一个const vector。以下应该有效:

for_each(vp.begin(), vp.end(), [&vp2](pair<int,int> p){vp2.push_back(p);});

但是没有理由使用它代替更简单的代码。