一道c语言的改错题目，请大家考虑考虑

jeffshia

#include<stdio.h>
#include<malloc.h>

void foo(int a,char *b)
{
     b = (char *)malloc(64);
     snprintf(b,"you are %d years old!\n",a);
}

int main(void)
{
     char *f;
     foo(23,f);
     printf("%s\n",b);
}

小锁

1. 
   
2. #include<stdio.h>
   
3. #include<malloc.h>
   
4. 
   
5. void foo(int a,char **b)
   
6. {
   
7.         (*b) = (char *)malloc(64);
   
8.         snprintf((*b),64,"you are %d years old!\n",a);
   
9. }
   
10. 
    
11. int main(void)
    
12. {
    
13.         char *f;
    
14.         foo(23,&f);
    
15.         printf("%s\n",f);
    
16. }
    

复制代码

jeffshia

我也看到了网上有这种改正的，但是你能说明为什么要这么改正么？

pupilzeng

最初由 小锁 发表

1. 
   
2. #include<stdio.h>
   
3. #include<malloc.h>
   
4. 
   
5. void foo(int a,char **b)
   
6. {
   
7.         (*b) = (char *)malloc(64);
   
8.         snprintf((*b),64,"you are %d years old!\n",a);
   
9. }
   
10. 
    
11. int main(void)
    
12. {
    
13.         char *f;
    
14.         foo(23,&f);
    
15.         printf("%s\n",f);
    
16. }
    

复制代码

没看出什么错误，除了一点不是很好的地方:最后没有free(f),但这在这里不是必须的.
运行起来也出什么错误

看错了，以为这个只是首贴的重新排版 :!!

bobhuang

在原程序上说明了

最初由 jeffshia 发表

1. 
   
2. #include<stdio.h>
   
3. #include<malloc.h>
   
4. 
   
5. void foo(int a,char *b) /*  *b是不能讲malloc的指针数组返回给调用函数的，所以用**b */
   
6. {
   
7.      b = (char *)malloc(64); /* 这里就要改成 (*b) */
   
8.      snprintf(b,"you are %d years old!\n",a); /* snprintf 的第2个参数应该是b的size，漏了*/
   
9. }
   
10. 
    
11. int main(void)
    
12. {
    
13.      char *f;
    
14.      foo(23,f); /* 修改了foo的接口，所以传个f的指针过去，&f */
    
15.      printf("%s\n",b); /* b在main函数中没有定义，应该是f */
    
16. }
    

复制代码

wuhu

**b,建议写成char *b[]，这样看起来一个指针数组多明了，呵呵

小锁

楼上解释的好。
大上楼的，没有好好看程序吧！
呵呵

lucifer

写**b是习惯问题

jeffshia

为什么说写**b是习惯问题呢?

lucifer

做个投票，看看程序员在这个问题上是怎么写就知道了。