ポインタのベクトルを削除する

Question

クラスのインスタンスのポインタを作成する必要があり、プログラムはコンパイル時に作成するポインタの数を知らない。 削除のために、私はベクトルにポインタを格納し、それを一つずつ削除することを検討していました。 スマートポインタの使用はよりクリーンな方法でしょうか？ スマートポインタを使用したくない場合、このベクトルの使用はきれいだと考えられますか？

最小コード：

#include <vector> 
using namespace std; 

class Foo { 
public: 
    Foo(); 
}; 
Foo::Foo(){} 
void createFooVector(int nb, std::vector<Foo*> &v){ 
    for(int i=0;i<nb;i++){ 
     Foo* f = new Foo(); 
     v.push_back(f); 
    } 
} 
int main(int argc, char *argv[]){ 
    std::vector<Foo*> v; 
    createFooVector(5,v); 
    while (!v.empty()){ 
     Foo* f = v.back(); 
     v.pop_back(); 
     delete f; 
    } 
} 

Answer 1

私は示唆しているいずれかのboost::pointer_vector、std::vector<std::unique_ptr<Foo>>、またはvector<Foo*>を保持し、コンストラクタで削除の世話をするあなた自身のFooマネージャークラスをロールアウトを使用します（「エキスパート」解決策、そして唯一の試みとしてこれを見るべきですあなたが例外安全性を完全に理解しているならば）。手動で削除を実行したくない場合は、簡単にエラーにつながる可能性があります。

Answer 2

あなたのコードは大丈夫です。しかし、スマートポインタを使用することが望ましい選択肢です（書き込むためのコードが少なく、メモリバグの機会がはるかに少ない）。

Answer 3

は、スマートポインタの使用は行くためのクリーンな方法でしょうか？

はい。

スマートポインタを使用したくない場合は、 ベクトルのこの使用はクリーンであると見なされますか？

私はアイデアを持っていない、誰かがそれは宿題ではない場合は、C++でのスマートポインタを使用したくない理由... しかし、私が思うに、それはこの場合にはboost::pointer_containersのようなものを使用することをお勧めします。

Answer 4

Fooから派生したクラスを使用しておらず、Fooが構造をコピーするのに比較的安価な場合は、vector<Foo>を使用してください。

コンパイラが移動セマンティクスをサポートしている場合、問題はありません。