帮忙看看这个getLine函数

iwinux

因为刚刚学 C 语言，经常做练习的时候都需要用到getchar()之类的函数，为了方便，我参考《C语言的科学和艺术》这本书的代码，实现了一个getLine函数，这里我把代码贴出来，大家帮忙看看有什么bug，谢谢！

1. 
   
2. #define BUFSIZE 255
   
3. /*reference:The Art and Science of C*/
   
4. int getLine(char ** line) {
   
5.         char * tmp;
   
6.         int c, n = 0, size = BUFSIZE;
   
7. 
   
8.         if (*line) {
   
9.                 free(*line);
   
10.         }
    
11. 
    
12.         *line  = (char *) malloc(size + 1);
    
13. 
    
14.         if (!*line) {
    
15.                 return 0;
    
16.         }
    
17. 
    
18.         while ((c = getchar()) != '\n' && c != EOF) {
    
19.                 if (n == size) {
    
20.                         size *= 2;
    
21.                         tmp = (char *) malloc(size + 1);
    
22.                         strncpy(tmp, *line, n);
    
23.                         free(*line);
    
24.                         *line = tmp;
    
25.                 }
    
26.                 *(*line + n) = c;
    
27.                 n++;
    
28.         }
    
29. 
    
30.         if (n == 0 && c == EOF) {
    
31.                 free(*line);
    
32.                 return 0;
    
33.         }
    
34.        
    
35.         *(*line + n)= '\0';
    
36.         tmp = 0;
    
37. 
    
38.         return n;
    
39. }

复制代码

flw10000

原帖由 iwinux 于 2006-8-2 10:06 发表
因为刚刚学 C 语言，经常做练习的时候都需要用到getchar()之类的函数，为了方便，我参考《C语言的科学和艺术》这本书的代码，实现了一个getLine函数，这里我把代码贴出来，大家帮忙看看有什么bug，谢谢！


        if (*line) {
                free(*line);
        }
[c ...

这里还没有malloc就free，会有问题吧！

flw10000

明白你的意思了

如果调用时已经malloc了，要先free，重新malloc，因为没法知道调用者malloc了多大空间！

MackedNice

if (*line) {
                free(*line);
        }
有问题. line 只是一个指针的地址。这个指针所指的内存是分配在函数栈上还是分配在堆上或者可能这个指针什么也没有指向，
你都是不知道的。所以你这样做是错误的。
当然你可以判断一下指针的地址是不是有效
if(line == NULL)
     return -1;


其他的好象没有问题。写得不错。
-------------------------------------
我学c的时候还写不出这样代码。

Bayweb

原帖由 flw10000 于 2006-8-2 11:18 发表
明白你的意思了

如果调用时已经malloc了，要先free，重新malloc，因为没法知道调用者malloc了多大空间！

>>
>>
>>一般malloc之前都会检查一下是否为空，或者size是否为零。如果不是十分必要，
>>在函数的结尾务必free一下。
>>
>>

flw10000

原帖由 Bayweb 于 2006-8-2 13:00 发表

>>
>>
>>一般malloc之前都会检查一下是否为空，或者size是否为零。如果不是十分必要，
>>在函数的结尾务必free一下。
>>
>>

咋俩的说法并不矛盾

我说的是楼主在程序开始处就判断*line如果为真就free，可能考虑的是并不知道该函数的调用者为*line申请了多大空间，所以free了，重新malloc了（size + 1）个内存空间！

默难

LZ看以下几个函数：
fgets realloc

yuxh

没有大的问题.
但在最后

1. 
   
2.         if (n == 0 && c == EOF) {
   
3.                 free(*line);
   
4.                 return 0;
   
5.         }

复制代码

如果不加上一句*line = NULL;的话,再次调入函数会重复free,程序会core哦.

flw

man 3 getline
man 3 fgets
man 3 realloc

iwinux

Google 了一下
这篇文章说realloc有潜在的隐患
http://www.csdn.net/develop/article/27/27950.shtm

不过C-FAQ又说：

Since you always want each block of dynamically-allocated memory to be contiguous (so that you can treat it as if it were an array), you and realloc have to worry about the case where realloc can't make the old block of memory bigger ``in place,'' but rather has to relocate it elsewhere in order to find enough contiguous space for the new requested size. realloc does this by returning a new pointer. If realloc was able to make the old block of memory bigger, it returns the same pointer. If realloc has to go elsewhere to get enough contiguous memory, it returns a pointer to the new memory, after copying your old data there. (In this case, after it makes the copy, it frees the old block.) Finally, if realloc can't find enough memory to satisfy the new request at all, it returns a null pointer. Therefore, you usually don't want to overwrite your old pointer with realloc's return value until you've tested it to make sure it's not a null pointer. You might use code like this:

看起来很安全 -____-

[ 本帖最后由 iwinux 于 2006-8-2 21:57 编辑 ]