NULLポインターを削除しても安全ですか？

NULLポインターを削除しても安全ですか？

そしてそれは良いコーディングスタイルですか？

deleteとにかくチェックを実行するので、あなたの側でそれをチェックすると、オーバーヘッドが追加され、醜く見えます。あ非常に良い練習をした後、NULLへのポインタを設定していますdelete（二重欠失および他の同様のメモリ破損の問題回避に役立ちます）。

deleteデフォルトでパラメータをNULLに設定していた場合も好きです

#define my_delete(x) {delete x; x = NULL;}

（私はRとLの値について知っていますが、いいと思いませんか？）

C ++ 0xドラフト標準から。

$ 5.3.5 / 2-「[...]どちらの方法でも、deleteのオペランドの値はnullポインター値である可能性があります。[... '」

もちろん、NULL値を持つポインターを「削除」することは決してできませんが、安全に行うことができます。理想的には、NULLポインタの削除を行うコードを含めないでください。しかし、ポインター（コンテナー内など）の削除がループで発生する場合に役立つことがあります。NULLポインタ値の削除は安全であるため、削除するNULLオペランドを明示的にチェックしなくても、削除ロジックを実際に作成できます。

余談ですが、C標準$ 7.20.3.2は、NULLポインターの 'free'は何もしないと言っています。

free関数は、ptrが指すスペースの割り当てを解除します。つまり、追加の割り当てに使用できるようになります。ptrがNULLポインターの場合、アクションは発生しません。

はい、安全です。

nullポインタを削除しても害はありません。割り当てられていないポインタがゼロに初期化され、その後単に削除された場合、関数の末尾でのテストの数を減らすことがよくあります。

前の文は混乱を引き起こしているので、説明されているものの例-これは例外的に安全ではありません-

void somefunc(void)
{
    SomeType *pst = 0;
    AnotherType *pat = 0;

    …
    pst = new SomeType;
    …
    if (…)
    {
        pat = new AnotherType[10];
        …
    }
    if (…)
    {
        …code using pat sometimes…
    }

    delete[] pat;
    delete pst;
}

サンプルコードで選択できるすべての種類のnitがありますが、概念は（私が望む）明確です。ポインタ変数はゼロに初期化されるのでdelete、関数の最後の操作は、ソースコードでnullでないかどうかをテストする必要がありません。とにかく、ライブラリコードはそのチェックを実行します。

nullポインタを削除しても効果はありません。必要がないため、必ずしも良いコーディングスタイルではありませんが、悪くもありません。

優れたコーディング方法を探している場合は、代わりにスマートポインタを使用することを検討してください。そうする必要はまったくありませんdelete。

ruslikの答えを補足するために、C ++ 14では次の構文を使用できます。

delete std::exchange(heapObject, nullptr);

削除演算子をオーバーロードしない限り安全です。削除演算子をオーバーロードしてnull条件を処理しないと、安全ではありません。

私はそうではないことを経験しましたdelete [] NULL（配列構文など）安全（VS2010）。これがC ++標準に準拠しているかどうかはわかりません。

NULL（スカラー構文）を削除しても安全です。