0

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

在开始时,initial()将调用 then loadtempdata()thenaveragetemp()和 finally 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;
    }

}
4

2 回答 2

2

在调用 Cleanup() 之前,您或法官系统是否有可能多次调用 LoadTempData?

于 2013-03-18T20:41:13.200 回答
2

根据您的前提条件,我能想到的内存泄漏的唯一原因如下:多次调用LoadTempData. LoadTempData您应该在函数的开头添加以下代码:

if (table != 0)
    Cleanup();

否则 2 次调用LoadTempData将导致第一个数据指针被简单地覆盖而不实际释放它。

于 2013-03-18T20:44:24.977 回答