我想知道你们中是否有人能确认我是否正确地删除了一些动态分配的内存。
下面的TileWrapper被初始化为一个二维指针数组:
private:
TileWrapper*** mLayout;我已经简化了它的初始化,向你展示了重要的部分:
void generateLayout() {
mLayout = new TileWrapper**[mRows];
for(int i = 0; i < mRows; i++) {
mLayout[i] = new TileWrapper*[mColumns];
}
for(int i = 0; i < mRows; i++) {
for(int j = 0; j < mColumns; j++) {
mLayout[i][j] = new TileWrapper();
}
}
}我需要确认的部分是销毁,如下所示:
~Destructor() {
for (int i = 0; i < mRows; i++) {
for (int j = 0; j < mColumns; j++) {
delete mLayout[i][j];
}
delete[] mLayout[i]; // CONFIRM THIS
}
delete[] mLayout; // CONFIRM THIS
}我特别关注由于[]字符而导致//确认这一点的删除。我的代码是防内存泄漏的吗?谢谢。
发布于 2016-12-23 03:23:11
就所显示的代码而言,它是安全和健全的。混淆可能源于使用第二个for循环的方式。它实际上等同于
for(int i = 0; i < mRows; i++) {
for(int j = 0; j < mColumns; j++) {
*(*(mLayout + i) + j) = new TileWrapper();
}
}使用三重指针的结构有一个甚至在K&R时代就已知的缺点:它不能保证占用整体内存区域,因为您在代码中单独分配每个“列”,因此它不是内存模型的指针的多维数组,这应该声明给这个“数组”的用户。在C++中,在不创建辅助模板类型或重新定义operator[]的情况下,不可能一次性分配整个数组并为其使用下标
https://stackoverflow.com/questions/41289699
复制相似问题