free后，指针不为null？

yecheng_110

俺一般定义一个宏
blazewater 发表于 2009-10-12 15:57

那你还不如写成这样

1. #define MY_FREE((point)) \
   
2. 
   
3. do\
   
4. 
   
5. {\
   
6. 
   
7.     if(point!=NULL){free(point); \
   
8. 
   
9.     point = NULL; }\
   
10. 
    
11. }while(0);
    

复制代码

有意义吗？

yecheng_110

free（）只是把指针指向的内存区域清空，至于指针又没有什么关系
smcree 发表于 2010-11-14 20:45

？
有这么傻的free

apollolegend

包装一下free不就得了

void myfree(void** p){
free(*p);
*p=NULL;
}

zhangsuozhu

以前不知跟谁学的free后一定置NULL。

后来写了很长时间代码了。想了想。确实没有什么必要。

如果连指针目前什么状态都没搞清楚。逻辑流程都稀理糊涂。还何谈写出健状的代码呢？
想想为什么free为什么不设计成
void FREE（void **p)
{
    free(*p);
    *p = NULL;
}

FREE(&p)
的原应就明白了。因为free后跟本不用 p=NULL;

狗气球

不谈需不需要置NULL,  我觉得如果一个指针跨越几百万行代码，那一定是有严重的设计问题。
donotblock 发表于 2010-11-15 13:07

    re。

    代码腐烂以后，你不置空那就是个访问野指针，置空那就是访问空指针。不论哪种，因为代码足够烂，都很难定位问题成因。解决不了咋办？使用之前加空指针判断。结果代码就更烂了。

    只不过访问空指针一般来说确实比访问野指针造成的问题容易定位一点。

lnitcscq

不谈需不需要置NULL,  我觉得如果一个指针跨越几百万行代码，那一定是有严重的设计问题。
donotblock 发表于 2010-11-15 13:07

    你理解错我的意思了，是整个产品的代码有那么多，出错了怎么排查？

zhangsuozhu

1. 置不置空也分情况的。
   
2. 比如：
   
3. statuc char *array[i] = {NULL};
   
4. 然后可能存在重多的array[i]  = malloc(xxxx);
   
5. 
   
6. 然后有的成功，有的没成功。然后做一个操作，就释放了。如：
   
7. if (!array[i])
   
8. {
   
9.      .............
   
10.      free(array[i]);
    
11.      array[i] = NULL;
    
12. }
    
13. 
    
14. 
    
15. 最后可能有一个全部回收的函数如:
    
16. void  distrory()
    
17. {
    
18.      int i;
    
19.     for(i = 0; i < sizeof(array); i++)
    
20.     {
    
21.            if (!array[i])
    
22.                  free(array[i]);
    
23.     }
    
24. }
    
25. 
    
26. 其中每一步都在不同的函数里。所以最后的destory是要判断空的。

复制代码

newmax123

想想C语言的法则,你就会明白无论free怎么实现也无法把它变成NULL,旁门左道不算
cjaizss 发表于 2010-11-15 12:45

对了， 函数里的用的p 和实质的p 是2个东西。

所以free 是不能把p = NULL的。

zhangsuozhu

设计成free(&p)就可以

flw

以前不知跟谁学的free后一定置NULL。

后来写了很长时间代码了。想了想。确实没有什么必要。

如果连指针目前什么状态都没搞清楚。逻辑流程都稀理糊涂。还何谈写出健状的代码呢？
zhangsuozhu 发表于 2010-11-16 09:57

重复一下让更多的人看看。