关于二重指针释放的顺序

C语言 码拜 7年前 (2015-07-10) 769次浏览 0个评论
 

使用 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协议进行授权 , 转载请注明关于二重指针释放的顺序
喜欢 (0)
[1034331897@qq.com]
分享 (0)

文章评论已关闭!