0

读后： C++指针数组：delete还是delete[]？ （由 shai vashid 回复）和 http://www.cplusplus.com/forum/beginner/6651/

我实现了以下内容：

内核.h

unsigned int **ConfigMeM;
//.....
~Kernel(){ //destructor
    for (unsigned int i=0; i<MeMSize; i++)
        delete [] MeM[i]; //Valgrind:- Invalid read of size 4
                                       - Invalid free() / delete / delete[] / realloc()
    delete [] MeM; //Valgrind: Invalid free() / delete / delete[] / realloc()

    for (unsigned int i=0; i<_item_tot; i++)
        delete [] ConfigMeM[i]; //Valgrind: Same comments as above
    delete [] ConfigMeM; //Valgrind: same as above
};

内核.cpp

//.......
MeM = new unsigned int*[MeMSize];
for (unsigned int i=0; i<MeMSize; i++){
    MeM[i] = new unsigned int[11]; //Valgrind: 14,608 bytes in 332 blocks are definitely lost in loss record 2,021 of 2,036
}
for (unsigned int i=0; i<MeMSize; i++){
    for (int j=0; j<10; j++){
        MeM[i][j] = 0;
    }
}
 //.....
 ConfigMeM = new unsigned int*[_item_tot];
for (unsigned int i=0; i<_item_tot; i++){
    ConfigMeM[i] = new unsigned int[3]; //Valgrind: 1,200 bytes in 100 blocks are definitely lost in loss record 1,131 of 2,036 
}
for (unsigned int i=0; i<_item_tot; i++){
    for (int j=0; j<3; j++){
        ConfigMeM[i][j] = 0;
    }
}
  //.....

我不确定我做错了什么。

请问有什么建议吗？

谢谢你。

4

1 回答 1

1

我最好的猜测是使用隐式生成的复制构造函数或复制赋值运算符复制您的类的对象。这将留下两个具有指向同一内存的指针的对象；两者都会尝试在销毁时删除此内存，从而导致双重删除错误。

最简单的解决方法是通过删除这些函数来防止复制：

Kernel(Kernel const&) = delete;
void operator=(Kernel const &) = delete;

或者，如果您被一个古老的编译器困住，请将它们声明为没有函数体的私有。

如果您需要复制这些对象，那么您将需要实现这些对象，也许是为了执行分配新内存块的“深度复制”。

std::vector或者，使用它来管理动态数组可能更容易；这已经具有正确的复制语义，并且可以非常简单地初始化：

std::vector<std::vector<unsigned int>> MeM, ConfigMem;

(MemSize, std::vector<unsigned int>(11));
于 2012-11-27T13:43:03.520 回答