ポインタを削除してリセットするためのC++テンプレート関数

Question

ポインタに関する小さな問題が発生しました。 私は1行でこのタスクを実行するために迅速かつ簡単な方法を持っていると思ったよう：

... 
delete pointer; 
pointer = 0; 
... 

私はすぐにポインタのいずれかのタイプを受け入れるテンプレートメソッドを持つ単純なヘッダファイルを設定します。これは次のようになります：

#ifndef P_DELETE_H 
#define P_DELETE_H 

template <typename T> 
void pDelete(T* pointer) { 
    if (pointer) { 
    delete pointer; 
    pointer = 0; 
    } 
} 

#endif 

しかし、結果は、オブジェクトbeeingが削除され、ポインタbeeingリセットの私の期待を満たしていません。代わりに、オブジェクトだけが削除されたようですが、ゼロに設定しても何の効果もありませんでした。誰かがここで私を少し明るくしてこの行動を説明できたら、私はそれほど気にしません！

UPDATE：

答えがのstd :: unique_ptrをとstdを使用して、説明したよう:: shared_ptrのは、このポインタをより無効化して行くための安全な方法です。

しかし、あなたは本当に私が要求されるような道を行く必要がある場合は、次のコードは、トリックを行います：

template <typename T> 
inline void pDelete(T*& pointer) { 
    if (pointer) { 
    delete pointer; 
    pointer = 0; 
    } 
} 

しかし、これは正しく削除し、ポインタを無効にもかかわらず、使用することが一般的に安全ではありませんすべてのインスタンスで（クイックテストはこれを明らかにした）。

有益な回答をいただき、ありがとうございます。あなたが探している

Answer 1

解決策は以下のとおりです。

delete pointer; 
pointer = nullptr; 

うん、ちょうどインラインそのコードを記述します。他の関数テンプレートを導入して、実際に何を理解して、1行のコードを保存するかを人々が調べなければならない理由はまったくありません。

上記は完全に読みやすく、わかりやすいです。

Answer 2

あなたがしようとしていることはすべて嘘です。これがなぜ自動的に起こっていないのか考えましたか？なぜランタイムは削除後にポインタを無効にしませんか？ C++標準委員会は十分にスマートではなかったので？

いいえ、あなたはほとんどの時間のためによくありません。はい、あなたはこのポインタのインスタンスを無効にします - しかし、あなたがポインタを持っていた全理由が、このポインタが他の場所にうまく生きている可能性が高いです！ （もちろんこれには例外があります）。その結果、無効化されたすべての無効化はポインタのこのインスタンスでの不正アクセスですが、他のインスタンスでは無効になります。本質的に、それはあなたに誤った安全感を与えます。

自動ポインタの使用を誤った感じにする代わりに、std::unique_ptrを最初に選択してください。十分でない場合は、std::shared_ptr - 本当に必要なことを確認してください。

Answer 3

所有権を示すために生ポインタを使用しないようにしてください。ポインタを使用してオブジェクトの所有権を取得する場合は、std::unique_ptrに移動する必要があります。

あなたの状況はpointer.reset()です。

あなたが所有するオブジェクトを他のコードが観察できるようにするには、pointer.get()を使用して非所有ポインターを取得します。 *pointerを使用して参照を取得することもできます。

生ポインタ（私はあなたのQtを見ています）を使用するインターフェイスでその所有権を渡す必要がある場合は、pointer.release()を使用してください。