パス

Question

が、私はその

class Base 
{ 
//some implementation 
}; 

class Deriv: public Base 
{ 
//implementation 
} 

class Case1 
{ 
    boost::scoped_ptr<A> a_ //polymorphic data member owned by C 
public: 
    Case1(A* a):a_(a) 
    { 

    } 
}; 
class Case2 
{ 
    boost::scoped_ptr<A> a_ //polymorphic data member owned by C 
public: 
    Case2(std::auto_ptr<A> a):a_(a.release()) 
    { 

    } 
}; 

のような多型クラス構造を持っていると私は、上記のもの多型のオブジェクトのいずれかを所有している第三級CASE1/2を持っているとしましょう。今は、このオブジェクトの所有権を取得するcase1/2クラスのコンストラクタにBase/Derivオブジェクトへのポインタを渡す必要があります。このオブジェクトをスマートポインタとして渡すべきですか？それを明確に私はあなたがスマートポインタを渡す必要があり、あなたはそれがスマートな名前を付ける必要があり、このオブジェクトの世話をターキン、または生のポインタ（ケース1）

Case1 c(new Deriv); 
//compared to 
Case2 c(std::auto_ptr<Base>(new Deriv)); 

Answer 1

のような非常に簡単な構文を許可することを可能にしていますようにするauto_ptrはポインタ（例えば、それは一時的なことはできません）：

std::auto_ptr<Base> p(new Deriv); 
Case2 c(p); // this constructor would need to take the `auto_ptr` by reference 
      // so that it can steal ownership. 

あなたの最初の例では、Case1 c(new Deriv);、new Derivが実行されたときに例外が間にスローされた場合、オブジェクトが漏洩する可能性があり、Case1オブジェクトはの所有権を取得する場合それはそのコンストラクタにあります。

スマートポインタの名前を付けない2番目の例では、状況によってはオブジェクトがリークする可能性があります。特に、これはif you have more than one argument to a functionで発生する可能性があります。

Answer 2

あなたのクラスが渡されたオブジェクトを完全に所有している場合は、該当するすべてのケースでauto_ptrを使用して明示的にすることをお勧めします。明示的にauto_ptrを構築するのが最善のケースです。これは、APIユーザーがあなたがそのオブジェクトを所有していることを知っていて、所有権ミックスアップの可能性を減らすためです。

クラスに変数の所有権がある場合は、通常、生ポインタと破棄関数を提供する方法があります。

class best_practices { 
    std::function<void(A*)> destructor; 
    A* ptr; 
public: 
    best_practices(A* a, std::function<void(A*)> destructfunc) 
     : ptr(a), destructor(destructfunc) {} 
    ~best_practices() { 
     destructor(ptr); 
    } 
}; 

Answer 3

私はこれがベストプラクティスであることのソースを持っていないが、あなたはいくつかの方法でリソースを格納するつもりなら、一般的に、私はそれが同じようにそのリソースを取得するのが最善です見つけます。

struct store_string 
{ 
    store_string(std::string s) : // potentially free, copy if not 
    s(std::move(s)) // free 
    {} 

    std::string s; 
}; 

やC +に：

この理由は、C++ 0xの中で、コピー/移動は引数が与えられたときに行われ、その後、あなただけのような、ストレージに移動ということです03、あなたのタイプが安く構築デフォルトにすることができた場合：

あなたのためにそう

struct store_string 
{ 
    store_string(std::string ss) // make necessary copy 
    { 
     s.swap(ss); // free 
    } 

    std::string s; 
}; 

、私はどうなる：

class Case2 
{ 
    boost::scoped_ptr<A> a_ //polymorphic data member owned by C 
public: 
    Case2(boost::scoped_ptr<A>& aa) 
    { 
     a.swap(aa); // take resource 
    } 
}; 

これは目を作りますあなたにとって単純なものであり、クライアントにリソースの管理方法を正確に知らせることができます。