1

我已经盯着这个有一段时间了，我不明白为什么它会失败。我有一个 CMatrix 类，实现一个二维矩阵，如下所示：

class CMatrix {
public:
    CMatrix(int height, int width);
    ~CMatrix();
    // more stuff here
private:
    void deleteData();
    void allocData(int height, int width);

    int rows, cols;
    double** data;
};

CMatrix::CMatrix(int height, int width) {
    allocData(height, width);
}

CMatrix::~CMatrix() {
    deleteData();
}

它工作正常，直到我试图摧毁它。有两种方法负责分配和释放，定义如下：

void CMatrix::allocData(int height, int width) {
    this->rows = height;
    this->cols = width;

    data = new double*[rows];
    for (int i = 0; i < rows; i++) {
        data[i] = new double[cols];
        for (int j = 0; j < cols; j++)
            data[i][j] = 0;
    }
}

void CMatrix::deleteData() {
    for (int i = 0; i < rows; i++) {
        delete data[i];
    }
    delete [] data;
}

这个简单的main代码会导致它发生故障：

int main(int argc, char** 

    CMatrix a(2, 3);
    a[0][0] = 1;
    a[0][1] = 2;
    a[0][2] = 3;
    a[1][0] = 4;
    a[1][1] = 5;
    a[1][2] = 6;

    return 0;
}

valgrind 说：

==21005== Mismatched free() / delete / delete []
==21005==    at 0x4C2A44B: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==21005==    by 0x402B5C: CMatrix::deleteData() (main.cpp:375)
==21005==    by 0x401731: CMatrix::~CMatrix() (main.cpp:138)
==21005==    by 0x402F5D: main (main.cpp:598)
==21005==  Address 0x5a06090 is 0 bytes inside a block of size 24 alloc'd
==21005==    at 0x4C2AAA4: operator new[](unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==21005==    by 0x402A84: CMatrix::allocData(int, int) (main.cpp:366)
==21005==    by 0x4015FB: CMatrix::CMatrix(int, int) (main.cpp:123)
==21005==    by 0x402D63: main (main.cpp:415)

在方法deleteData()和中打印此消息i=0。i对于其他s 和 delete[]，它什么也没说。

我究竟做错了什么？

4

3 回答 3

3

您需要delete[]在每一行上使用，因为它们被分配了new[]：

for (int i = 0; i < rows; i++) {
    delete[] data[i];
}

请注意，[i]末尾的 是数组下标，与 . 无关delete[]。想一想：

for (int i = 0; i < rows; i++) {
    double* p = data[i];
    delete[] p;
}

由于p指向动态分配数组中的第一个元素，因此必须对其进行delete[]编辑。

于 2013-03-25T11:05:46.313 回答
1

除了先前确定delete[]与delete问题的答案之外，请注意，您还应该为您的类定义一个复制构造函数（或声明它们以禁止复制语义），因为您直接管理您的原始资源（即动态分配的内存）班级。有关详细信息，请参阅三法则。operator=CMatrixprivate

或者您可以简单地使用直接资源管理器，std::vector而不是使用new[]. 您还可以定义嵌套向量来构建 2D 矩阵，例如在vector<vector<double>>. 这样，std::vector将正确地自动进行内存分配、清理和复制。

于 2013-03-25T11:29:49.660 回答
1

您的删除功能不会删除您为每行中的列分配的内存块

代码应该是这样的

void CMatrix::deleteData() {
    for (int i = 0; i < rows; i++) {
        delete[] data[i];
    }
    delete [] data;
}
于 2013-03-25T11:43:57.623 回答