void f(boost::shared_ptr<int> ptr)
{
if (ptr) // should we check?
// do something
}
void f2(int *p)
{
if (p) // good practice to check before using it
// do something
}
質問：shared_ptrを使用する前に検証する必要がありますか？boost :: shared_ptrまたはstd :: shared_ptrを使用する前にチェックする必要がありますか？
はい。 shared_ptrは、割り当てられたメモリを解放するタイミングを知って、そのメモリへの参照をカウントすることによって、あなたに利益をもたらします。あらかじめヌルではないことを事前に知る方法がないと仮定して、使用前に有効かどうかを検証する必要がありますが、shared_ptrを使用しているかどうかの問題の横にあります。
普通のプログラム実行中に実際にshared_ptrがヌルであるかどうかによって異なります。そうでない場合は、ユーザーの前提条件として責任を押してfとし、おそらくassert(ptr);とします。それはshared_ptrにのみ適用されるのではなく、実際にポインタが使用されることに注意してください。おそらくここでの参照を使用してください。
私は特に参照パラメータb/cを持つ関数を提供することによって質問を列挙しません。参照は空のポインタを指すべきではなく、常に参照を確認する必要はないと仮定します。しかし、それでも何とか無効にすることができます。 – q0987
は、シナリオによって異なります。
f()は、誰かがそうあなたがチェックする必要があり、あなたを渡すかもしれないものを制御することはできませんパブリック関数/ API関数である場合。
この関数がプライベートメンバー関数であるか、有効なポインタを必要とすると文書化されている場合は、assertを使用します。リリースビルドでオーバーヘッドが発生せず、デバッグビルドですぐに問題を表示します。
void f(shared_ptr<T> ptr)
{
assert(ptr && "ptr is null!");
.....
}
デバッグとリリースで異なる動作をするため、ランタイム値の検証のためのアサートを嫌う。アサーションは、ランタイムチェック用ではなくアプリケーションインバリアント用に予約する必要があります。 –
レッツは言う：あなたのvoid foo(shared_ptr<int> ptr)は汎用ライブラリであり、あなたは、実行時にユーザーに知らせたいnullptr値がサポートされていないことを（例外をスロー）、またはそれは異なるコードパスで処理されている場合、はい、それは正しいですそれを確認する。 shared_ptrをfoo()のどこかに格納する必要がなく、その関数がヌルポインタをサポートしないようにする必要がある場合は、リファレンスを渡すだけです（例：void foo(int &integer)）。私が知っているコンパイラでは、参照はほとんどの場合、null以外のポインタです（多くの場合、最適化の選択に依存します）。
参照の数をチェックするのに** use_count **関数を使用するだけで、チェックすることはありません。 – softghost