Question

我编写了一个代码来计算地图中区域温度的平均值。

一开始，initial()将被调用，然后loadtempdata()然后averagetemp()，最后cleanup()。

我使用了一个全局指针来指向函数中的堆动态数组，以便它可以在其他函数中使用。最后，我使用delete[] table来回收内存，但结果表明我仍然有内存泄漏，我发现它很长时间，但找不到原因。

为什么还有内存泄漏？

提前谢谢。

#include<iostream>
namespace TEST
{
    double *table = 0;
    const double *mtx = 0;
    int n = 0, m = 0;

    void LoadTempData(const double* matrix_row_major, int M, int N){
        table = new double [(M + 1) * (N + 1)];
        mtx = matrix_row_major;
        m = M;
        n = N;
        for(int i = 1; i < M + 1; i ++){
            for(int j = 1; j < N + 1; j ++){
                table[j + i * (N + 1)] = matrix_row_major[(j - 1) + (i - 1) * N]
                + table[j - 1 + i * (N + 1)]
                + table[j + (i - 1) * (N + 1)]
                - table[j - 1 + (i - 1) * (N + 1)];
            }
        }
    }
    double RegionAvgTemp(int y1, int x1, int y2, int x2){
        if(y1 == y2){
        return (table[x2 + 1 + (y2 + 1) * (n + 1)]
        - table[x2 + 1 + (y2 - 1 + 1) * (n + 1)]
        - table[x1 - 1 + 1 + (y1 + 1) * (n + 1)]
        + table[x1 - 1 + 1 + (y1 - 1 + 1) * (n + 1)]) / (x2 - x1 + 1);
        }
        else if(x1 == x2){
        return (table[x2 + 1 + (y2 + 1) * (n + 1)]
        - table[x1 + 1 + (y1 - 1 + 1) * (n + 1)]
        - table[x2 - 1 + 1 + (y2 + 1) * (n + 1)]
        + table[x1 - 1 + 1 + (y1 - 1 + 1) * (n + 1)]) / (y2 - y1 + 1);
        }
        else if ((y1 == y2) && (x1 == x2)){
            return mtx[x1 + y1 * n];
        }
        else{
        return (table[x2 + 1 + (y2 + 1) * (n + 1)]
        + table[x1 - 1 + 1 + (y1 - 1 + 1) * (n + 1)]
        - table[x2 + 1 + (y1 - 1 + 1) * (n + 1)]
        - table[x1 - 1 + 1 + (y2 + 1) * (n + 1)]) / ((y2 - y1 + 1) * (x2 - x1 + 1));
        }
        return 0;
    }


    void Init()
    {

    }

    void Cleanup()
    {
        delete[] table;
    }

}

Answer 1

在调用Cleanup（）之前，您或者判断系统是否有可能多次调用LoadTempData？

Answer 2

由于您的前置条件是内存泄漏的唯一原因，我可以想到以下内容：多次调用LoadTempData。您应该在LoadTempData函数的开头添加以下代码：

if (table != 0)
    Cleanup();

否则对LoadTempData的2次调用将导致第一个数据指针被简单地覆盖而不实际解除分配。

使用operator删除动态但仍然是内存泄漏

2 个答案: