C++ベクトル内のオブジェクトを削除する

Question

現在、プログラム内の人を保持するためのベクトルを使用しています。私は関数内のベクトルと同様に、私は削除したい要素を渡す

vectorname.erase(index);

でそれを削除しようとしています。私の主な問題は、コンパイル速度に関してコードをどのように改善できますか？

#include <iostream> 
#include <string> 
#include <vector> 
using namespace std; 

class person { 
    private: 
     string name; 
    public: 
     person() {} 
     person(string n):name(n){} 
     const string GetName() {return name;} 
     void SetName(string a) { name = a; } 
}; 

void DeleteFromVector(vector<person>& listOfPeople,person target) { 
    for (vector<person>::iterator it = listOfPeople.begin();it != listOfPeople.end();++it) {//Error 2-4 
     if (it->GetName() == target.GetName()) { 
      listOfPeople.erase(it); 
      break; 
     } 
    } 
} 

int main(){ 
    //first group of people 
    person player("Player"), assistant("Assistant"), janitor("Janitor"), old_professor("Old Professor"); 

    //init of vector 
    vector<person> listOfPeople = { player, assistant, janitor, old_professor }; 

    DeleteFromVector(listOfPeople, janitor); 
} 

Answer 1

indexを定義する必要がイテレータは、ベクター内のオブジェクトにアクセスするために使用することができ、ありません：

for (vector<person>::iterator it = listOfPeople.begin(); it != listOfPeople.end(); ++it) {//Error 2-4 
    if (it->GetName() == target.GetName()) { 
     listOfPeople.erase(it); 
     break; 
    } 
} 

次の行は、forループを分割することですので、私たちは、無効なイテレータを考慮していませんここの問題。

Answer 2

ベクターからオブジェクトを削除するためにループを実行する必要はありません。ちょうどstd::find_if：

#include <algorithm> 
//... 
void DeleteFromVector(vector<person>& listOfPeople, const person& target) 
{ 
    // find the element 
    auto iter = std::find_if(listOfPeople.begin(), listOfPeople.end(), 
          [&](const person& p){return p.GetName() == target.GetName();}); 

    // if found, erase it 
    if (iter != listOfPeople.end()) 
     listOfPeople.erase(iter); 
}