getterメソッドのconst参照

Question

このコードを確認してください。私はここに何も間違っていると信じています。

テンプレートが使われるクラスのデキュー機能を見るのが好きです。

void enqueue(const T &data) 
    { 
     _mutex.lock(); 
     _queue.push(data); 
     _mutex.unlock(); 
    } 

T const& dequeue() 
    { 
     _mutex.lock(); 
     T &data = _queue.back(); 
     _queue.pop(); 
     _mutex.unlock(); 

     return data; 
    } 

Answer 1

dequeueには、ダングリングリファレンスが返されます。 popが オブジェクトをペダルにすると、そのオブジェクトは存在しなくなります。

Answer 2

まず、ミューテックスをスラップしてスレッドセーフと呼ぶことはできません。それが不必要な場合は恐ろしいオーバーヘッドが発生するだけでなく、特定の操作のアトミック性も破られます。たとえば、キュ​​ーのサイズを確認してその間に誰かがそれを空にしている可能性があるため、キューから1つを取り出してください。それとも、オブジェクトを取り除いてから、それがポップされたのでしょうか？おっとっと。

スレッドセーフはでデータに同時にアクセスしません。は、データ構造にミューテックスをチャッキングして呼び出すだけではありません。

第2に、がのコンカレントコンテナを構築しようとしている場合は存在し、それらは必要です。インテルのTBBとMicrosoftのPPLの同時データ構造を見てください。彼らはあなたのslap-a-mutex-on-itハッキングよりもずっと速く、かなり少ないバグになる同時使用用に設計されたインターフェースを持っています。