リストからオブジェクトへのポインタを返す

Question

長時間の読者初回質問質問者。 私はfutureEvents.getNextEvent（）のようなものに行きたいと思うFutureEventクラスのオブジェクトのリストを持っています。これは現在、私はこれが私の主要なメモリリークの原因であると私は思う新しい呼び出しのために動作します。どんな助けでも大歓迎です。そうでなければ、ポインタを次のイベントに戻すことができます。あなたはこの関数の結果どこにもないdeleteを行う場合のおかげ

EVENT* FutureEvent::getNextEvent() 
{ 
EVENT* nextEvent = new EVENT; 
*nextEvent = futureEvents.front(); 
futureEvents.pop_front(); 
return nextEvent; 
} 

Answer 1

が、これは確かにメモリリークの原因です。

考えられる解決策は次のとおりです。

1. は、あなたもダイナミックアロケーション（すなわち、new）を使用する必要はありません。このアプローチでは値によって

   を返します。自動オブジェクトのみを使用するので、クリーンアップする必要はありません。使用するメモリは自動的に解放され、デストラクタは自動的に呼び出されます。

   EVENT FutureEvent::getNextEvent() { 
       EVENT nextEvent = futureEvents.front(); 
       futureEvents.pop_front(); 
       return nextEvent; 
   } 
   

2. スマートポインタを使用して、このアプローチは、まだ動的割り当てを使用しますが、自動的にクリーンアップの世話をすることができますクラスにそれが委譲されます。

   std::unique_ptr<EVENT> FutureEvent::getNextEvent() { 
       std::unique_ptr<EVENT> nextEvent(new EVENT(futureEvents.front())); 
       futureEvents.pop_front(); 
       return nextEvent; 
   } 
   

   あなたはboost::shared_ptrのように、ブーストでスマートポインタのいずれかを使用することができますstd::unique_ptrあなたのコンパイラがサポートしていない場合

   。