一个常见的指针操作错误问题

converse

原帖由 flw2 于 2008-9-5 11:26 发表
free(p); p = NULL;
我认为这么写不是个C语言程序员, 这也会招flw鄙视的
free之后的做法是保证不用*p

好了,借这个帖子讨论一下这个问题,我是支持free之后置NULL的,除非确认是不再使用的指针才不会进行这个操作.
比如flw2的说法,保证不用*p是很难百分之百做到的,问题在于,有人错误用了*p的时候不一定会造成程序出错,用flw的话说"错误的程序不一定有错误的结果",为了让这个操作有错误的结果,就得让p为NULL.

[ 本帖最后由 converse 于 2008-9-5 11:32 编辑 ]

flw2

原帖由 converse 于 2008-9-5 11:30 发表


好了,借这个帖子讨论一下这个问题,我是支持free之后置NULL的,除非确认是不再使用的指针才不会进行这个操作.
比如flw2的说法,保证不用*p是很难百分之百做到的,问题在于,有人错误用了*p的时候不一定会造成程 ...

如过free了还*p，定义一个FREE宏好了

flw

原帖由 converse 于 2008-9-5 11:30 发表

好了,借这个帖子讨论一下这个问题,我是支持free之后置NULL的,除非确认是不再使用的指针才不会进行这个操作.
比如flw2的说法,保证不用*p是很难百分之百做到的,问题在于,有人错误用了*p的时候不一定会造成程 ...

我理解你，要让你现在明白这个道理，的确有点困难，慢慢来吧。能做到哪个阶段就先做到哪个阶段。

nicozhou

原帖由 lenky0401 于 2008-9-5 11:19 发表


不对哟 我把p=NULL后 至少下次判断if(p==NULL)的时候可以判断出P已经无效了
如果不p=NULL后 下次判断if(p==NULL)的时候还会把它当有效指针使用

你都free它了，还用它，那还有效个鬼啊。

converse

原帖由 flw 于 2008-9-5 11:44 发表

我理解你，要让你现在明白这个道理，的确有点困难，慢慢来吧。能做到哪个阶段就先做到哪个阶段。

flw,我不想骂人,请你自重,谢谢.
讨论问题应该有起码的姿态,你知道你为什么总是被人骂吗?

[ 本帖最后由 converse 于 2008-9-5 12:18 编辑 ]

waker

原帖由 converse 于 2008-9-5 12:17 发表


flw,我不想骂人,请你自重,谢谢.
讨论问题应该有起码的姿态,你知道你为什么总是被人骂吗?

因为那人是你的马甲？

mik

free(p) 后再将 p = NULL 有时候是很有用的。特别在遍历 free 整条深度链表，至少保证前提不出错

blizzard213

原帖由 flw 于 2008-9-5 11:16 发表

严重支持！

那个题我做了80分啊
去哪里找林博士把我招进上海朗讯贝尔？

一仓麻叶

1. 
   
2. #include <stdio.h>
   
3. #include <stdlib.h>
   
4. #include <string.h>
   
5. #include <assert.h>
   
6. int main(void)
   
7. {
   
8.         char *str = (char *) malloc(20);  //使用malloc在堆中申请20个字节的空间，返回首地址。
   
9.         strcpy(str, "hello");                    //把hello放入申请的空间内，占用6个字节。
   
10.         free(str);                                    //释放内存，内存管理器只是做标记，并不做额外的操作
    
11.                                           //就和在硬盘删除文件一样。标记这段空间可以使用。
    
12.                                           //str的值不会改变，还是指向那段空间的首地址。
    
13.         strcpy(str, "world");                   //这种情况下再写入数据不会出现内存访问异常，因为空间并没有释放。
    
14.                                           
    
15.         char *cc = (char *) malloc(20);   //再次申请相同的空间，好的 不用费力运算了，直接把刚才释放的再给你
    
16.         printf(cc);                                    //打印cc 发现内容是world。 证实以上内存分配释放策略。
    
17.                                           //不过这个依赖于底层实现，不一定是这样子，只是解释你问题中的现象。
    
18.                                              //另外释放之后 把指针赋0，是个好的编码习惯，后面你再用时就是访问地址0，
    
19.                                           //直接抛异常。
    
20.         return 0;
    
21. }
    
22. 
    

复制代码

sep

原帖由 flw 于 2008-9-5 09:23 发表

对不起，看来我真的冤枉你了。
看来你真的不是骗分，
而是水平太差，建议你从现在开始再不要回答任何问题了，
好好学学别人都是怎么回答的，希望你的水平能够有所提高，不要再误导人了，谢谢！

flw，我对你的敬仰犹如滔滔江水连绵不绝啊
其实在嵌入式版，我也觉得有这么一个人的，老是给别人错误的信息，误导别人从错误的方向入手，简直浪费别人的生命
我很早就想跟他这样说了，但就是没有这样的勇气啊