ポインタのベクトル内のポインタが指すオブジェクトを削除する

Question

私は動的に割り当てられた火星の構造体を指すポインターのベクトルを持っています。私の関数を追加するとうまくいくようですが、私はオブジェクトとポインタの適切な削除をしない私の削除関数を恐れています。あなたは削除機能を見て、どこで問題を解決するか教えてください。あなたは先の尖った（実際にはメモリが解放され、それが再利用可能になります言わせれることはありません）が使用するメモリ領域を解放しますポインタに削除を呼び出すことによって

struct Martian 
{ 
    string fname, lname, ssid; 
    Martian(string fname, string lname, string ssid) : fname(fname), lname(lname), ssid(ssid){} 
}; 

class martianDatabase 
{ 
    public: 
     vector<Martian*> database; 
     martianDatabase(); 
     void deleteMartian(string deletedID); 
     void addMartian(int &i, string f, string l, string id); 
     int iterator = 0, size = 0; 
}; 

void martianDatabase::addMartian(int& i, string f, string l, string id) 
{ 
    this->database.push_back(new Martian(f, l, id)); 
    i++; 
} 

void martianDatabase::deleteMartian(string deleteID) 
{ 
    int i = 0; 
    while (i < size +1) 
    { 
     if (this->database[i]->ssid == deleteID){ 
     delete this->database[i]; 
     size--; 
     break; 
     } 
     else ++i; 
    } 
} 

Answer 1

。

void martianDatabase::deleteMartian(string deleteID) 
{ 
    auto martianIterator = database.begin(); 

    while(martianIterator != database.end()) 
    { 
     if((*martianIterator)->ssid == deleteID) 
     { 
      delete *martianIterator; 
      martianIterator = database.erase(martianIterator); 
     } 
     else 
     { 
      ++martianIterator; 
     } 
    } 
} 

Answer 2

。

あなたのプログラムで遭遇する問題は、あなたのエントリを削除してからベクトルのサイズを小さくすることですが、ポインタは削除されず、nullptrに設定されません。つまり、逆参照しようとすると問題が発生します。コードは、最後の要素を削除した場合にのみ機能します）。

また、C++ 11で導入された新しいスマートポインタを使用することもできます（あなたの場合は、生ポインタをstd :: shared_ptrに置き換えることができます）。詳細について

：これはssid値が一致するすべてのMartianオブジェクトを削除

http://en.cppreference.com/w/cpp/memory/shared_ptr