求教如何不漏释放内存，谢谢

shiqgnefu

回复 9# cobras

个人倾向对有内存分配与释放的函数借助goto实现只有一个出口设计，这样不会加深代码的层次，比如按照你的设计思路，在超过两个内存分配时会加深深度，容易遗漏内存释放

1. 
   
2. #define FREE_RES(res)        if (res) free(res), res = NULL
   
3. void fun(void) {
   
4.         a = NULL, b = NULL;
   
5.        
   
6.         if (NULL == (a = calloc())){
   
7.                 ...
   
8.                 goto fun_err;
   
9.         }
   
10.        
    
11.         if (NULL == (b = calloc())){
    
12.                 ...
    
13.                 goto fun_err;
    
14.         }
    
15.        
    
16.         ...
    
17. fun_err:
    
18.         FREE_RES(b);
    
19.         FREE_RES(a);
    
20. }
    

复制代码

纯属讨论，个人观点而已

zsszss0000

这种结构确实比较简洁，我也觉得栈式的处理方法不太合适，如果申请的变量多了的话，结构会变得非常复杂。回复 11# shiqgnefu


   

bsdc

回复 11# shiqgnefu


    嗯，多谢你的肯定，但是我的场景是：

    a=malloc();//分配的内存很大，几十M，而且是多线程的，每台机器有n个这样的程序
    处理很长一段时间。。。。
    b=malloc();
   
所以我需要及时释放，如果都留到goto可能内存比较紧张。所以我只能到处加Free，即使多加了也无所谓

shiqgnefu

回复 13# bsdc

场景决定代码该怎么设计，刚回头看了下你的帖子，没想到我们思路相近。
个人认为，实际的代码是由具体业务决定，代码是死的，思路是活的。
   

jd808

shiqgnefu 发表于 2015-01-26 10:26
回复 9# cobras

个人倾向对有内存分配与释放的函数借助goto实现只有一个出口设计，这样不会加深代码的层 ...

不是说珍惜生命,远离goto么,goto是什么?从来不知道哦

bskay

实现好像有问题啊
C里面是按值传递的

bsdc

回复 16# bskay


    确实，你说得没错，所以我传了指针的地址过去，还是楼上一位老兄的宏用起来顺手，我已经改成此宏。

einsnabuck

我一般是用do {} while(0);

bsdc

回复 18# einsnabuck


    你是指宏里面的do {} while(0)吗？不过感觉这里用逗号表达式也够简洁，牛逼啊，我总是简单事情复杂化