内存分配的专用功能会导致内存泄漏吗？

Question

内存分配的专用功能会导致内存泄漏吗？

Hy all,

我相信下面这段代码会产生内存泄漏？

    /* External function to dynamically allocate a vector */
    template <class T>
            T *dvector(int n){
            T *v;

            v = (T *)malloc(n*sizeof(T));

            return v;
    }


    /* Function that calls DVECTOR and, after computation, frees it */
    void DiscontinuousGalerkin_Domain::computeFaceInviscidFluxes(){
            int e,f,n,p;
            double *Left_Conserved;

            Left_Conserved = dvector<double>(NumberOfProperties);

            //do stuff with Left_Conserved
            //

            free(Left_Conserved);

            return;
    }

Run Code Online (Sandbox Code Playgroud)

我认为,通过将指针传递给DVECTOR,它将分配它并返回正确的地址,以便free(Left_Conserved)成功解除分配.但是,似乎并非如此.

注意:我也测试过new/delete替换malloc/free也没有成功.

我有一个用于分配二维数组的类似代码.我决定管理这样的矢量/数组,因为我经常使用它们,而且我也想了解使用C++进行更深入的内存管理.

所以,我非常希望保留一个外部函数来为我分配向量和数组.这里有什么能避免内存泄漏的问题？

编辑

我一直在使用DVECTOR函数来分配用户定义的类型,所以这可能是一个问题,我想,因为我没有调用构造函数.

即使在我释放Left_Conserved向量之前的代码段中,我也想分配一个向量并将其保持为"open"以通过其指针通过其他函数进行评估.如果使用BOOST,它会在函数结束时自动清除分配,所以,我不会得到一个带有BOOST的"公共"数组,对吗？我想用NEW很容易解决这个问题,但对于矩阵来说,更好的方法是什么？

我刚刚发现我将指针作为参数传递给其他函数.现在,BOOST似乎没有那么多享受它,并且编译以错误退出.

因此,我需要一个指向矢量或矩阵的指针,它接受用户定义的类型,它们将作为参数传递给其他函数.矢量(或矩阵)很可能在外部函数中分配,并在另一个合适的函数中释放.(我只是不想复制循环和新的东西,在代码中的任何地方分配矩阵!)

这是我想做的事情:

    template <class T>
    T **dmatrix(int m, int n){
            T **A;

            A = (T **)malloc(m*sizeof(T *));
            A[0] = (T *)malloc(m*n*sizeof(T));

            for(int i=1;i<m;i++){
                    A[i] = A[i-1]+n;
            }

            return A;
    }


    void Element::setElement(int Ptot, int Qtot){

            double **MassMatrix;

            MassMatrix = dmatrix<myT>(Ptot,Qtot);

            FillInTheMatrix(MassMatrix);

            return;
    }

Run Code Online (Sandbox Code Playgroud)

Answer 1

Eva*_*ran 7

那里没有内存泄漏,但你应该使用new/delete []而不是malloc/free.特别是因为你的功能是模板化的.

如果您想要使用具有非平凡构造函数的类型,则基于malloc的函数会被破坏,因为它不会调用任何构造函数.

我只需要这样做就可以取代"dvector":

void DiscontinuousGalerkin_Domain::computeFaceInviscidFluxes(){
        double *Left_Conserved = new double[NumberOfProperties];

        //do stuff with Left_Conserved
        //

        delete[] Left_Conserved;
}

Run Code Online (Sandbox Code Playgroud)

它在功能上是等效的(除了它可以为其他类型调用构造函数).它更简单,需要更少的代码.此外,每个c ++程序员都会立即知道发生了什么,因为它不涉及额外的功能.

更好的是,使用智能指针完全避免内存泄漏:

void DiscontinuousGalerkin_Domain::computeFaceInviscidFluxes(){
        boost::scoped_array<double> Left_Conserved(new double[NumberOfProperties]);

        //do stuff with Left_Conserved
        //
}

Run Code Online (Sandbox Code Playgroud)

许多聪明的程序员都喜欢说"最好的代码就是代码,你不必写"

编辑:为什么你认为你发布的代码泄漏内存？

编辑:我看到你对另一篇文章的评论说