4

単一のデザインパターンを実装するクラスがあります。一部の人々はそれが良い考えだとは思わないことを私は知っていますが、それは大いに役立ちます、

とにかく-メモリリークがあり、vlagrindは次の行を示しています。

_singleton = new Manager(); //Manager::instance() (Manager.cpp:18)

Manager::Manager() : _file(new ofstream), _tasks(new map<int, Task *>()),
        _idState(new map<int, int>()), _closing(false), _pending(false),
        _lock(new pthread_mutex_t), _endLock(new pthread_mutex_t),  _cond(new pthread_cond_t),
        _flushCond(new map<int, pthread_cond_t *>()), _attr(new pthread_attr_t) {
//The last line is line 25 in Manager::Manager

現在、Managerのデストラクタでは、ばかげたループが作成されるため、明示的に削除できません(_singletonを削除するとデストラクタが呼び出され、無限ループが発生するため)。このリークを取り除くにはどうすればよいですか?ありがとう!

ここでのPsはValgrindの出力です:

==17823== 512 bytes in 1 blocks are definitely lost in loss record 2 of 2
==17823==    at 0x4C27297: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17823==    by 0x40151E: Manager::Manager() (Manager.cpp:25)
==17823==    by 0x4014DB: Manager::instance() (Manager.cpp:18)
==17823==    by 0x406475: initdevice(char*) (outputdevice.cpp:66)
==17823==    by 0x4061D5: main (driver.cpp:21)
==17823== 
==17823== LEAK SUMMARY:
==17823==    definitely lost: 512 bytes in 1 blocks
=    =17823==    indirectly lost: 0 bytes in 0 blocks
==17823==      possibly lost: 288 bytes in 1 blocks
==17823==    still reachable: 0 bytes in 0 blocks
==17823==         suppressed: 0 bytes in 0 blocks

追加:マネージャーを作成するコードは次のとおりです。

Manager.h:
class Manager {
public:
    ~Manager();
    static Manager * instance();
private:
    Manager();
    static Manager * _singleton;
};

そして実装:

Manager.cpp:
Manager * Manager::_singleton = 0;

Manager * Manager::instance() {
    if (!_singleton)
        _singleton = new Manager();
    return _singleton;
}
4

2 に答える 2

4

C ++でシングルトンを実装する一般的な方法の1つは、インスタンスstd::unique_ptr<T>をクラス静的変数ではなく、インスタンスゲッター内で関数静的にすることです。これにより、プログラムの完了時にデストラクタが確実に呼び出され、抽象基本クラスへのポインタなどを介して多態的にアクセスされるインスタンスを作成できます。

Scott Meyersは、彼の「より効果的なC ++」の本で、このトピックに関する優れた議論を提供しました。

于 2012-05-13T14:50:21.507 に答える
2

静的Managerオブジェクトを作成すると、そのコンストラクタとデストラクタが自動的に呼び出されます。または、演算子newを使用して割り当てる必要がある場合は、ポインターが存在するときに破棄されるように、スマートポインター(可能な場合はunique_ptr、それ以外の場合はauto_ptr)に配置します。

于 2012-05-13T14:46:46.493 に答える