一个关于malloc函数的问题，新手求解答

初行者

先上代码

1. # include <stdio.h>
   
2. # include <malloc.h> void f(int * pArr, int i);
   
3. void g(int * pArr, int i); int main(void)
   
4. {
   
5. int i;
   
6. printf("请输入你要定义的数组大小：\n");
   
7. scanf("%d", &i);
   
8. int * p = (int *)malloc(sizeof(int) * i); free(p); f(p, i);
   
9. g(p, i); free(p); //在定义动态变量的时候，一定不要忘了释放内存// f(p, i);
   
10. // g(p, i); return 0;
    
11. }void f(int * pArr, int i)
    
12. {
    
13. printf("请输入数组每个数的值!\n");
    
14. int j;
    
15. for (j=0; j<i; ++j)
    
16.   scanf("%d", &pArr[j]);
    
17. printf("输入完毕！！\n");}void g(int * pArr, int i)
    
18. {// printf("%d\n", pArr[i-2]); int j = 0;  /*
    
19.      要是检查没有语法错误，就检查标点符号吧，
    
20.      尤其是逗号和分号，还要注意是在英文状态下输入的
    
21.      */
    
22. for (j=0; j<i; ++j) //
    
23.   printf("%d  ", pArr[j]);}
    
24. 
    

复制代码

在台式机上的在g();函数运行完之后就出
file:///C:/Users/Administrator/AppData/Roaming/Tencent/Users/312024054/QQ/WinTemp/RichOle/ZH1QSTTJ0TEJO6O]8OUK2OE.jpg
但在笔记本上的时候free();函数起不到释放内存的作用，就算吧free()后面的两个函数的注释去点仍能正常运行，不会出现内存的问题

求解答，说明一下，我是用的VC++6.0，

第一次发帖提问，勿喷，如果有什么没交代清楚的可以回贴问，但现在要睡觉了，等明天起来回答

向往

我已经对你的代码整理了一下,并加上了一些解释,如果有其他问题,请回复提问

1. 
   
2. # include <stdio.h>
   
3. # include <malloc.h>
   
4. void f(int *pArr, int i);
   
5. void g(int *pArr, int i);
   
6. int main(void)
   
7. {
   
8.     int i;
   
9.     printf("请输入你要定义的数组大小：\n");
   
10.     scanf("%d", &i);
    
11.     int *p = (int *)malloc(sizeof(int) * i);
    
12.     free(p); //上一行刚申请到的空间,这里马上就释放掉了???建议释放后要p=NULL;将p指向空指针
    
13.     f(p, i); /*上一行free(p);已经把p指向的空间释放了,为什么这里还要去使用释放掉的空间???
    
14.                为什么这里不出错?因为就算房子拆了,但是房子所在的那个地址,还是存在的
    
15.                所以仍然可以使用那块空间,但是这样做是很危险的,所以要p=NULL */
    
16.     g(p, i); //同上
    
17.     free(p); //释放内存是正确的,但是!!千万不要对同一块空间释放2次!!!!
    
18.     return 0;
    
19. }
    
20. void f(int *pArr, int i)
    
21. {
    
22.     printf("请输入数组每个数的值!\n");
    
23.     int j;
    
24.     for (j = 0; j < i; ++j)
    
25.         scanf("%d", &pArr[j]);
    
26.     printf("输入完毕！！\n");
    
27. }
    
28. void g(int *pArr, int i)
    
29. {
    
30.     int j = 0;
    
31.     for (j = 0; j < i; ++j) //
    
32.         printf("%d  ", pArr[j]);
    
33. }
    

复制代码

补充内容 (2012-11-4 01:19):
int *p = (int *)malloc(sizeof(int) * i);
free(p);
不是说申请了不能马上释放,这是绝对没问题的
问题就只有2个:
1.释放了的空间再次使用
2.对同一块空间释放2次

初行者

因为第一次在论坛贴代码，可能个是有点乱，见谅

初行者

图没贴上，另上传了

飘香夜梦

楼主的代码确实有点乱，我大概看了下，不明白楼主为什么要free两次，malloc和free必须成对出现的啊，你第一次free了，p所指向的对空间也就被释放了，就是说程序员不应该对这段空间在进行操作了，而且楼主的代码也非常不规范，命名就不说了，每次free(p)之后应该立马对p赋一个NULL值，这样可以起到检测错误的作用，因为在后面的程序中你仍然有可能对p进行操作，但p所指向的空间已经被释放了，这就不知道会造成什么后果了，后果可能是灾难性的，如果加上p = NULL这么一句的话，如果在之后的代码中你依旧对p进行了操作，程序一运行就会崩，会崩就是好事，说明你的程序有问题，弹出的错误代码会是C05，一看到这个错误代码有经验的人很快就能确定并修改代码。。。。

xiaowu88910

:@使用了一个释放的空间，程序员必须要注意的一点就是这个！！！

初行者

向往 发表于 2012-11-4 01:14

                               
登录/注册后可看大图

我已经对你的代码整理了一下,并加上了一些解释,如果有其他问题,请回复提问

补充内容 (2012-11- ...

1. # include <stdio.h>
   
2. # include <malloc.h>
   
3. 
   
4. void f(int * pArr, int i);
   
5. void g(int * pArr, int i);
   
6. 
   
7. int main(void)
   
8. {
   
9.         int i;
   
10.         printf("请输入你要定义的数组大小：\n");
    
11.         scanf("%d", &i);
    
12.         int * p = (int *)malloc(sizeof(int) * i);
    
13. 
    
14.         f(p, i);
    
15.         g(p, i);
    
16. 
    
17.         free(p);        //在定义动态变量的时候，一定不要忘了释放内存
    
18. 
    
19.         f(p, i);
    
20.         g(p, i);
    
21. 
    
22.         return 0;
    
23. }
    
24. 
    
25. void f(int * pArr, int i)
    
26. {
    
27.         printf("请输入数组每个数的值!\n");
    
28.         int j;
    
29.         for (j=0; j<i; ++j)
    
30.                 scanf("%d", &pArr[j]);
    
31.         printf("输入完毕！！\n");
    
32. 
    
33. }
    
34. 
    
35. void g(int * pArr, int i)
    
36. {
    
37. 
    
38. //        printf("%d\n", pArr[i-2]);
    
39. 
    
40.         int j = 0;                /*
    
41.                                         要是检查没有语法错误，就检查标点符号吧，
    
42.                                         尤其是逗号和分号，还要注意是在英文状态下输入的
    
43.                                         */
    
44.         for (j=0; j<i; ++j)        //
    
45.                 printf("%d  ", pArr[j]);
    
46. 
    
47. }
    

复制代码

可能是第一次传的时候弄错了代码，就这一段代码而言，free()函数扔起不到释放内存的作用，free()后面的函数仍可以执行，？？？，还有，能否加QQ详问

初行者

飘香夜梦 发表于 2012-11-4 00:48

                               
登录/注册后可看大图

楼主的代码确实有点乱，我大概看了下，不明白楼主为什么要free两次，malloc和free必须成对出现的啊，你第一 ...

1. # include <stdio.h>
   
2. # include <malloc.h>
   
3. 
   
4. void f(int * pArr, int i);
   
5. void g(int * pArr, int i);
   
6. 
   
7. int main(void)
   
8. {
   
9.         int i;
   
10.         printf("请输入你要定义的数组大小：\n");
    
11.         scanf("%d", &i);
    
12.         int * p = (int *)malloc(sizeof(int) * i);
    
13. 
    
14.         f(p, i);
    
15.         g(p, i);
    
16. 
    
17.         free(p);        //在定义动态变量的时候，一定不要忘了释放内存
    
18. 
    
19.         f(p, i);
    
20.         g(p, i);
    
21. 
    
22.         return 0;
    
23. }
    
24. 
    
25. void f(int * pArr, int i)
    
26. {
    
27.         printf("请输入数组每个数的值!\n");
    
28.         int j;
    
29.         for (j=0; j<i; ++j)
    
30.                 scanf("%d", &pArr[j]);
    
31.         printf("输入完毕！！\n");
    
32. 
    
33. }
    
34. 
    
35. void g(int * pArr, int i)
    
36. {
    
37. 
    
38. //        printf("%d\n", pArr[i-2]);
    
39. 
    
40.         int j = 0;                /*
    
41.                                         要是检查没有语法错误，就检查标点符号吧，
    
42.                                         尤其是逗号和分号，还要注意是在英文状态下输入的
    
43.                                         */
    
44.         for (j=0; j<i; ++j)        //
    
45.                 printf("%d  ", pArr[j]);
    
46. 
    
47. }
    

复制代码

free两次是代码上错了，我的问题是free函数没起到作用，没有释放内存，至于命名，刚学c，从网上下的教程还没看完，所以命名上有所失误，见谅，
十分感谢您的回答，以后注意

飘香夜梦

初行者 发表于 2012-11-4 12:30

                               
登录/注册后可看大图

free两次是代码上错了，我的问题是free函数没起到作用，没有释放内存，至于命名，刚学c，从网上下的教程还 ...

1. # include <stdio.h>
   
2. # include <malloc.h>
   
3. 
   
4. void f(int * pArr, int i);
   
5. void g(int * pArr, int i);
   
6. 
   
7. int main(void)
   
8. {
   
9.         int i;
   
10.         printf("请输入你要定义的数组大小：\n");
    
11.         scanf("%d", &i);
    
12.         int * p = (int *)malloc(sizeof(int) * i);
    
13. 
    
14.         f(p, i);
    
15.         g(p, i);
    
16. 
    
17.         //在定义动态变量的时候，一定不要忘了释放内存
    
18.         if (NULL != p)
    
19.         {
    
20.             free(p);
    
21.             p = NULL;
    
22.         }      
    
23. 
    
24.         f(p, i);
    
25.         g(p, i);
    
26. 
    
27.         return 0;
    
28. }
    
29. 
    
30. void f(int * pArr, int i)
    
31. {
    
32.         printf("请输入数组每个数的值!\n");
    
33.         int j;
    
34.         for (j=0; j<i; ++j)
    
35.                 scanf("%d", &pArr[j]);
    
36.         printf("输入完毕！！\n");
    
37. 
    
38. }
    
39. 
    
40. void g(int * pArr, int i)
    
41. {
    
42. 
    
43. //        printf("%d\n", pArr[i-2]);
    
44. 
    
45.         int j = 0;                /*
    
46.                                         要是检查没有语法错误，就检查标点符号吧，
    
47.                                         尤其是逗号和分号，还要注意是在英文状态下输入的
    
48.                                         */
    
49.         for (j=0; j<i; ++j)        //
    
50.                 printf("%d  ", pArr[j]);
    
51. 
    
52. }

复制代码

飘香夜梦

帮楼主修改了，现在的代码再在后面使用p变量就会报错了，这是好事，因为报错了我们就会去修改它，如果现在不报错指不定哪天报错，如果你开发的软件是这样的话也许就玩大了，因为你想，如果你开发一个飞机导航系统，测试的时候一切正常，但是飞机上天的时候突然软件崩了，那不玩大了吗？

其实我之前就已经回答了楼主这个问题，楼主第一次贴的代码不仅仅是两次free的问题，free的时候没有判断，而且free之后p变量没赋空值我都有提过，只是楼主没有认真看而已。。。。。

我们为什么要这么做，我向楼主解释一下，int * p = (int *)malloc(sizeof(int) * i);

这句代码是向堆空间申请一段内存，并且让p指向这段内存，而之后的free(p)是收回这段堆内存，但是你要注意，p变量依然指向这段堆内存，也就是说我们依然能用p变量对这段堆内存进行操作，难道楼主不觉得这是灾难性的吗？因为如果后面你再申请一个指针变量p2也指向一段堆内存，但是第一次申请的已经被释放了，却没有对p赋空值的话，那么第二次申请的堆内存可能就和第一次申请的有重叠，那么通过p也可以修改p2所指向的堆内存里面的值，那这个程序运行下去的结果也许就无法估计了，因为连内存中的值已经被非法修改了。。。。

向往

初行者 发表于 2012-11-4 12:18

                               
登录/注册后可看大图

可能是第一次传的时候弄错了代码，就这一段代码而言，free()函数扔起不到释放内存的作用，free()后面的函 ...

free(p);
已经把p指向的空间释放了,为什么这里还要去使用释放掉的空间???
为什么这里不出错???

因为就算房子拆了，但是房子所在的那个地址，还是存在的，所以仍然可以使用那块空间。
但是这样做是很危险的，所以要p=NULL;

程序在执行到 int *p = (int *)malloc(sizeof(int) * i); 的时候，
在内存中获得了一块内存空间（买了一块地，没有盖房子），指针p指向这块内存空间（在还买了另一个小房子，在这个小房子的墙上，写有你买的那块地的地址）。

到 f(p, i); 的时候，在f()函数（施工队）中，向p指向的内存空间写入数据（让请来的施工队把房子建好）。


到 free(p); 的时候，
把p指向的内存空间释放了，这块空间就还给操作系统去管理，不属于本程序了，但是p指针还是你的，p仍然保存着那个地址。
（请free()这个中介把房子卖给ZF了，这房子不是你的了，但是你还有另一个小房子呀，里面有你卖给ZF的房子的地址）


f(p, i);
（又把f()施工队找来，根据p的地址，再次找到那块地，不管ZF将那块地弄成怎样，是改成公园还是游泳池，你都不管！反正你就要施工队把地夷为平地，再按你的要求，新建其它的）

return 0;
程序退出。ZF发现那块地变样了。。。
所以，是不能这么做的，尽管你对ZF有不满的地方。。。

***建议：在free(p)之后，要p=NULL;，把墙上的地址擦掉。

补充内容 (2012-11-4 18:52):






补充内容 (2012-11-4 18:52):
ZF = 政。府

初行者

向往 发表于 2012-11-4 18:51

                               
登录/注册后可看大图

free(p);
已经把p指向的空间释放了,为什么这里还要去使用释放掉的空间???
为什么这里不出错???

可能理解错我的意思了
能否加QQ详问312024054
备注加   鱼C