コンストラクタとデストラクタの共有リソースをロックする

Question

私はC++のマルチスレッドの基本については、少なくとも上手く取り組んでいると思いますが、共有リソースの周りにミューテックスをロックするという明確な答えは得られませんでしたコンストラクタまたはデストラクタで宣言します。私はあなたが両方の場所でロックしなければならないという印象を受けていましたが、最近の同僚は反対しています。以下のクラスは、複数のスレッドによってアクセスされたふり：もちろん、すべての

class TestClass 
{ 
public: 

    TestClass(const float input) : 
     mMutex(), 
     mValueOne(1), 
     mValueTwo("Text") 
    { 
     //**Does the mutex need to be locked here? 
     mValueTwo.Set(input); 
     mValueOne = mValueTwo.Get(); 
    } 

    ~TestClass() 
    { 
    //Lock Here? 
    } 

    int GetValueOne() const 
    { 
     Lock(mMutex); 
     return mValueOne; 
    } 

    void SetValueOne(const int value) 
    { 
     Lock(mMutex); 
     mValueOne = value; 
    } 

    CustomType GetValueTwo() const 
    { 
     Lock(mMutex); 
     return mValueOne; 
    } 

    void SetValueTwo(const CustomType type) 
    { 
     Lock(mMutex); 
     mValueTwo = type; 
    } 

private: 

    Mutex mMutex; 
    int mValueOne; 
    CustomType mValueTwo; 
}; 

を初期化リストを通じて安全であるが、何コンストラクタ内のステートメントについてのでしょうか？デストラクタでは、非スコープのロックを行い、ロックを解除することは有益でしょう（基本的にはpthread_mutex_destroyを呼び出します）。

Answer 1

複数のスレッドが同じオブジェクトを構築することはできません。オブジェクトを完全に構築する前に、そのスレッドを使用することもできません。したがって、正当なコードでは、ロックなしの構築は安全です。

破壊は少し難しい場合があります。しかし、やはり、オブジェクトの適切なライフタイム管理によって、スレッドがまだそれを使用している可能性がある場合にオブジェクトが破壊されることはありません。

共有ポインタは、例えばこれを達成するのに役立ちます。 ：

• 場合、すべてのスレッド（必要であれば、それを構築したスレッドを含む）オブジェクトへのアクセスを必要とするすべてのスレッドへ
• パス共有ポインタ
• オブジェクトが破壊される特定のスレッドでオブジェクトを構築共有ポインタを解放しました

もちろん、他の有効なアプローチが存在します。重要な点は、オブジェクトの存続期間の3つの主要段階（建設、使用、破壊）の境界を適切に保つことです。これらの段階の間に重複を許さないでください。

Answer 2

コンストラクタ内でロックを解除する必要はありません。外部の誰かがその時点でそのデータにアクセスできる唯一の方法は、コンストラクタ自体から渡す場合です（または呼び出しなどの未定義の動作仮想メソッド）。

は、[編集：デストラクタ程度除去された部分、あなたが死んだかもしれないオブジェクトからリソースにアクセスしようとしている場合、コメントは当然主張するよう以来、あなたは大きな問題を抱えている]