std :: list的連續使用導致崩潰

我有代碼是爲了做內存管理，但它在我從'活'列表中刪除一個對象並將它放到'死'之一：std :: list的連續使用導致崩潰

class MemoryObject { 
private: 
    static std::list <MemoryObject *> alive, dead; 
    long references; 
public: 
    MemoryObject() { 
     alive.push_back(this); 
     references = 0; 
    } 

    static void deepClean() { 
     clean(); 
     std::list<MemoryObject *>::iterator iterator; 
     for(iterator = alive.begin(); iterator != alive.end(); iterator ++) { 
      MemoryObject *object = *iterator; 
      Log::instance().write(DEBUG_LOG, "\nObject still active at the end of the program, check for memory leaks." 
        "\nSize: %d", 
        alive.size()); 
      delete object; 
     } 
     alive.clear(); 
    } 

    void reference() { 
     references ++; 
    } 

    void release() { 
     references --; 
     if(references <= 0) { 
      dead.push_back(this); 
      alive.remove(this); 
     } 
    } 

    static void clean() { 
     std::list<MemoryObject *>::iterator iterator; 
     for(iterator = dead.begin(); iterator != dead.end(); iterator ++) 
      delete(&iterator); 
     dead.clear(); 
    } 

    ~MemoryObject() { 
     clean(); 
    } 
}; 

std::list <MemoryObject *> MemoryObject::alive, MemoryObject::dead;

Eclipse的調試顯示它未能在釋放（），總是在第二列表相關的現場 - 我試圖把它們（alive.remove(this) and dead.push_back(this)）以不同的順序，這改變不了什麼。然而有趣的是，如果我把在中間的東西他們，就像一個printf（）語句，它不會崩潰...

這裏就是我從調用它：

#include <stdlib.h> 
#include <stdio.h> 

#include "log/log.hpp" 
#include "memory/object.hpp" 

int main(int argc, char *argv[]) { 
    MemoryObject foo; 
    foo.release(); 
    MemoryObject::deepClean(); 
    return 0; 
}

來源

2017-07-06 Inigo Selwood

在你的'clean'功能你'刪除（迭代器）'。這是不正確的。 –

請顯示你是如何調用這些函數的。 –

請仔細閱讀：[mcve] –

你delete只有通過new分配的內容。

MemoryObject foo; 
foo.release();

而且內部的清潔

for(iterator = dead.begin(); iterator != dead.end(); iterator ++) 
     delete(*iterator); //I am assuming you have * instead of &(which is incorrect as mentioned in another answer).

那麼，你是呼籲這是不是通過new分配試試這個東西delete()。

MemoryObject *foo=new MemoryObject(); 
foo->release();

編輯：這將仍然不起作用的原因如下。

內clean()要刪除的dead元素，這就是當前object.So的this總之你正在做這樣的事情

class A 
{ 
    void delete_this() 
    { 
     delete(this); 
     //At this point "this" is a dangling pointer and you should not use it. 
    } 
};

來源

2017-07-06 09:02:10

在你clean功能你有：

delete(&iterator);

這將編譯，但會嘗試刪除迭代器本身 - 這是在堆棧（這將崩潰）。

我懷疑你想要的東西：

delete(*iterator);

來源

2017-07-06 08:40:27

您不能刪除不是用new分配的對象。 MemoryObject foo;不是用新的allocatetd。

來源

2017-07-06 08:56:00

std :: list的連續使用導致崩潰

回答

相關問題