在C++中复制并遍历矢量

Question

我尝试将一个矢量上一个矢量复制到另一个矢量中currentVector in C++。

我的代码如下。 我想浏览currentVector并根据值的条件，在之前的矢量中添加元素。我不确定这是正确的做法。

int ChooseElement(std::vector<Powers>* previousVector) { 
    std::vector<Powers> currentVector(*previousVector); 
    for(auto iter = currentVector.begin(); iter != currentVector.end(); iter++) { 
     if(condition on (*iter)) 
     (*previousPowers).push_back(someValue); 
    } 
    return 0 
} 

main(int argc, const char * argv[]){ 
    std::vector<myObjectType> listOfElements; 
    myObjectType result = myObjectType(1,2); 
    listOfElements.push_back(result); 
    result = ChooseElements(&listOfElements); 
    return 0; 
} 

Answer 1

我想浏览currentVector和基于值条件下，previousVector添加元素。

如果你的目的是要增加，你迭代元素，那么这正是std::copy_if是：

std::copy_if(currentVector.begin(), currentVector.end(), 
      std::back_inserter(previousVector), condition_functor); 

如果您打算做你的代码做什么，然后你根本不需要currentVector，因为它的值似乎不用于任何事情。你的代码的作用是：它将someValue添加到previousVector，并将它n次，其中n是条件为真的previousVector中的元素数。如果这是真的，你想要什么，那么这是一个简单的方法来做到这一点：

void ChooseElement(std::vector<Powers>& v) { 
    auto count = std::count_if(v.begin(), v.end(), condition_functor); 
    v.insert(v.end(), count, someValue); 
} 

Answer 2

你push_back()要细，如currentVector是原始载体的深层副本，等previousVector操作并不影响它在所有。但这种方法是非常无效，则应使用临时的向量：

int ChooseElement(std::vector<Powers> &previousVector) { 
    std::vector<Powers> tmp; 
    for(auto iter = previosVector.begin(); iter != previousVector.end(); iter++) { 
     if(condition on (*iter)) 
     tmp.push_back(someValue); 
    } 
    std::move(tmp.begin(), tmp.end(), std::back_inserter(previousVector));  
    return 0; 
} 

如果无法C++ 11使用替代std::move()有：

previosVector.insert(previosVector.end(), tmp.begin(), tmp.end());