C语言 逐个字符读取流字符

Question

我正在使用这个函数来读取，逐个字符，文本文件或标准输入

void readLine(FILE *stream, char **string) {
    char c;
    int counter = 0;
    do {
        c = fgetc(stream);
        string[0] = (char *) realloc (string[0], (counter+1) * sizeof(char));
        string[0][counter++] = c;
    } while(c != ENTER && !feof(stream));
    string[counter-1] = '';
}

但是当我调用它时，我的程序崩溃了，我真的不知道为什么，因为我不会忘记 0 终止符，而且我相信我正确存储了 char 序列。我已经验证了字符串长度，但它看起来没问题。

Answer 1

这是一个错误：

do {
    c = fgetc(stream);
    // What happens here?!? 
} while(c != ENTER && !feof(stream));

"这里发生的事情"是您在检查 EOF 之前将c添加到string，哎呀。

这是非常不好的：

    string[0] = (char *) realloc (string[0], (counter+1) * sizeof(char));

在一个循环中。 realloc是一个可能很昂贵的调用，您可以为每个字节的输入执行此操作！请求一个(显然(没有分配任何东西的指针参数也是一个愚蠢而混乱的接口——传入指针通常表示已经完成。 如果string是一个静态数组呢？ 相反，以块为单位分配并返回一个指针：

char *readLine (FILE *stream) {
// A whole 4 kB!
    int chunksz = 4096;
    int counter = 0;
    char *buffer = malloc(chunksz);
    char *test;
    int c;
    if (!buffer) return NULL;
    while (c = fgetc(stream) && c != ENTER && c != EOF) {
        buffer[counter++] = (char)c;
        if (counter == chunksz) {
            chunksz *= 2;
            test = realloc(buffer, chunksz);
        // Abort on out-of-memory.
            if (!test) {
                free(buffer);
                return NULL;
            } else buffer = test;
        }
    }
// Now null terminate and resize.
    buffer[counter] = '';
    realloc(buffer, counter + 1);
    return buffer;
}

这是一个标准的"2次方"分配方案(它翻了一番(。如果你真的想提交一个指针，预先分配它，并提交一个"最大长度"参数：

void *readLine (FILE *stream, char *buffer, int max) {
    int counter = 0;
    int c;
    while (
        c = fgetc(stream)
        && c != ENTER
        && c != EOF
        && counter < max - 1
    ) buffer[counter++] = (char)c;
// Now null terminate.
    buffer[counter] = '';
}       

Answer 2

此代码中存在一些问题：

1. fgetc()返回int .
2. 不要在 C 中强制转换 malloc() 和朋友的返回值。
3. 避免使用sizeof (char)，这只是一种非常笨拙的1写法，所以乘以它是非常多余的。
4. 通常，缓冲液一次生长超过 1 个字符，realloc()可能很昂贵。
5. string[0]写成*string会更清楚，因为它不是一个数组，而只是一个指向指针的指针。
6. 您在文件末尾的逻辑意味着它将存储EOF的截断版本，不是很好。

Answer 3

更改此行

string[counter-1] = '';

自

string[0][counter-1] = '';

您希望终止存储在 string[0] 的字符串。