1

ポインタのstd::listを使用していくつかのテストを行っています。リストのいくつかの要素を削除するためにremove_ifアルゴリズムを使用しています。しかし、私はいくつかの問題に遭遇しました。remove_ifはポインタを破壊しないため、メモリリークを引き起こしています(私は思います)。

私は解決策を見つけましたが、それがうまく作られているのか、正しいのか、少なくとも許容できるのかわかりません...

コードは次のとおりです。

#include <algorithm>
#include <iostream>
#include <list>

using namespace std;

class Object
{
 private:
         int intData;   
 public:
        Object(int n) : intData(n) { };
        int getIntData(void) { return intData; };
        void setIntData(int n) { intData = n; };
};

/** Functor */
struct listFunctor
{ 
 bool operator()(Object* obj1, Object* obj2) const
 {
  return (obj1->getIntData() < obj2->getIntData());
 }
};

class removeFunctor
{
 private:
         int remover;
 public:
        removeFunctor(int n) : remover(n) { };
        bool operator()(Object* obj)
        {
         bool res = (obj->getIntData() != remover);

         if(res)
          delete obj;

         return res; 
        }
};

typedef list<Object*> objList;
typedef list<Object*>::iterator objectListIter;

int main(int argc, char** argv)
{
 objList objectList;

 objectList.push_back(new Object(8));
 objectList.push_back(new Object(0));
 objectList.push_back(new Object(2));

 /** sort elements. */
 objectList.sort(listFunctor());

 /** print. */
 for(objectListIter it = objectList.begin(); it != objectList.end(); ++it)
  cout<<*it<<"  "<<(*it)->getIntData()<<'\n';   

 /** remove. */
 objectListIter iter = remove_if(objectList.begin(), objectList.end(), removeFunctor(8));

 /** print. */
 for(objectListIter it = objectList.begin(); it != iter; ++it)
  cout<<*it<<"  "<<(*it)->getIntData()<<'\n';   

 /** delete list. */
 for(objectListIter it = objectList.begin(); it != iter; ++it)
  delete *it;   

 objectList.clear(); //IS THIS NECESSARY?

 return 0;
}

プログラムは最初にリストを作成し、それをソートしてからいくつかの要素を削除します。

このコードは、この問題に対する優れた実行可能な解決策ですか?Valgrindのデフォルトのスキャンでは問題は報告されませんが、さらにテストを行っています。

ありがとう。

4

1 に答える 1

10

aから削除するlist<Object*>と、リストからポインタのみが削除されます。スマートポインタがリストから削除されたときにポイントされたオブジェクトを自動的に削除するalist<unique_ptr<Object>>またはaを選択する必要があります。list<shared_ptr<Object>>

于 2012-05-10T18:24:31.510 に答える