Code Bye

关于二重指针释放的顺序

码拜
 

使用 malloc 来申请内存，申请的代码部分如下：

float *fp = (float*)malloc(sizeof(float)* h * w);
float **cost = (float**)malloc(sizeof(float*)* h);
for (int i = 0; i < h; ++i) {
    cost[i] = &fp[i * w];
}

那我释放的时候应该如何释放呢？
我现在采用的办法是：

for (int i = 0; i < h; ++i) {
    free(cost[i]);
    cost[i] = NULL;
}
free(fp);

显然，这个代码在循环的时候就报错了。
 
应该是先释放一级指针，再释放二级指针吧？

10分
有几次malloc就使用几次free；而且二者的地址必须相同；

malloc分配的内存，是有簿记的；只能在分配的地址上释放；否则没法找到大小信息（free用来释放）；

所以，不能够“free(cost[i]);”；
而只能：free(fp);（或者等价的cost[0]）；

10分

 

free (fp);
free (cost);

或者

free (*cost);
free (cost);

也就是free的次数必须和malloc的次数相同。虽然你分配内存时在for循环中的行为很奇怪（通常二重指针不是这么申请内存的）。
 
引用 3 楼 JiangWenjie2014 的回复: 

free (fp);
free (cost);

或者

free (*cost);
free (cost);

也就是free的次数必须和malloc的次数相同。虽然你分配内存时在for循环中的行为很奇怪（通常二重指针不是这么申请内存的）。

这样释放的时候不就是一个语句就行了嘛～
当然，这种写法是看 paper 配的 code 学到的，是不是这样写会存在问题？

20分
仅供参考： 
#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
int **newarr2d(int rows,int cols) {
    int **p,i;

    p=(int **)malloc(rows*sizeof(int *));
    if (NULL==p) exit(1);
    for (i=0;i<rows;i++) {
        p[i]=(int *)malloc(cols*sizeof(int));
        if (NULL==p[i]) exit(1);
    }
    return p;
}
void deletearr2d(int **p,int rows) {
    int i;

    for (i=0;i<rows;i++) {
        free(p[i]);
    }
    free(p);
}
int main() {
    int **arr2d,i,j,r,c;

    r=4;
    c=5;
    //在堆中开辟一个4×5的二维int数组
    arr2d=newarr2d(r,c);
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            arr2d[i][j]=i*c+j;
        }
    }
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            printf(" %2d",arr2d[i][j]);
        }
        printf("\n");
    }
    deletearr2d(arr2d,r);

    r=6;
    c=3;
    //在堆中开辟一个6×3的二维int数组
    arr2d=newarr2d(r,c);
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            arr2d[i][j]=i*c+j;
        }
    }
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            printf(" %2d",arr2d[i][j]);
        }
        printf("\n");
    }
    deletearr2d(arr2d,r);

    return 0;
}
//  0  1  2  3  4
//  5  6  7  8  9
// 10 11 12 13 14
// 15 16 17 18 19
//  0  1  2
//  3  4  5
//  6  7  8
//  9 10 11
// 12 13 14
// 15 16 17
//
 
引用 5楼赵4老师 的回复:

仅供参考：

#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
int **newarr2d(int rows,int cols) {
    int **p,i;

    p=(int **)malloc(rows*sizeof(int *));
    if (NULL==p) exit(1);
    for (i=0;i<rows;i++) {
        p[i]=(int *)malloc(cols*sizeof(int));
        if (NULL==p[i]) exit(1);
    }
    return p;
}
void deletearr2d(int **p,int rows) {
    int i;

    for (i=0;i<rows;i++) {
        free(p[i]);
    }
    free(p);
}
int main() {
    int **arr2d,i,j,r,c;

    r=4;
    c=5;
    //在堆中开辟一个4×5的二维int数组
    arr2d=newarr2d(r,c);
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            arr2d[i][j]=i*c+j;
        }
    }
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            printf(" %2d",arr2d[i][j]);
        }
        printf("\n");
    }
    deletearr2d(arr2d,r);

    r=6;
    c=3;
    //在堆中开辟一个6×3的二维int数组
    arr2d=newarr2d(r,c);
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            arr2d[i][j]=i*c+j;
        }
    }
    for (i=0;i<r;i++) {
        for (j=0;j<c;j++) {
            printf(" %2d",arr2d[i][j]);
        }
        printf("\n");
    }
    deletearr2d(arr2d,r);

    return 0;
}
//  0  1  2  3  4
//  5  6  7  8  9
// 10 11 12 13 14
// 15 16 17 18 19
//  0  1  2
//  3  4  5
//  6  7  8
//  9 10 11
// 12 13 14
// 15 16 17
//

感谢赵老师～
 
我那种申请方式有没有问题？因为在 free 的时候，如果按照您这样的，得有一个循环，而按照我写的那种，不需要循环，可能效率会高点？
 
free(cost);
free(fp);
CodeBye 版权所有丨如未注明 , 均为原创丨本网站采用BY-NC-SA协议进行授权 , 转载请注明关于二重指针释放的顺序！