读取不同流的独特功能答案

【问题标题】：Unique function to read different streams读取不同流的独特功能
【发布时间】：2013-09-03 21:24:09
【问题描述】：

我正在寻找一种使用 C 中独特的 void 函数来读取 file 和 stdin 流的方法>。我正在尝试使用此功能：

#define ENTER 10 //'\n' ASCII code

........

void read(FILE *stream, char *string) {
    char c;
    int counter = 0;

    do {
        c = fgetc(stream);
        string = realloc(string, (counter+1) * sizeof(char));
        string[counter++] = c;
    } while(c != ENTER && !feof(stream));

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

但它仅适用于 stdin 流。当我使用文本文件时，文件内容在函数之外不可见。我这样调用这个函数：

read(stdin, inputString);
read(inputFile, fileContent);

并且只适用于第一种情况。

P.S.：最初，inputString 一个文件内容被声明为

char *inputString = malloc(sizeof(char));
char *fileContent = malloc(sizeof(char));

而且我知道 fgetc 返回 int，char by char 重新分配很昂贵（但我只需要使用必要的内存）并且 EOF 或 '\n' 是存储在字符串中（但之后被 0 终止符替换）。

【问题讨论】：

你还没有初始化计数器变量。
对不起..这里写错了，但我现在已经编辑了。

标签： c pointers stream

【解决方案1】：

由于您重新分配了字符串但没有返回指向新分配的指针，并且您错误处理了fgetc()，因此它无法正常工作。

你可能需要：

void read(FILE *stream, char **string)
{
    int c;
    int counter = 0;
    int available = 0;

    while ((c = fgetc(stream)) != EOF && c != ENTER)
    {
        if (counter >= available)
        {
            int new_size = (available + 2) * 2;
            void *space = realloc(*string, new_size);
            if (space == 0)
                // Handle out of memory reporting?
                return;
            *string = space;
            available = new_size;
        }
        (*string)[counter++] = c;
    }
    (*string)[counter] = '\0';
}

你会这样称呼它：

read(stdin, &inputString);
read(inputFile, &fileContent);

这里有很多更正；仍然可以并且应该进行改进。

将指针传递给指针，以便更改可以反映在调用代码中。在原始代码中，如果realloc() 调用可以增大当前块，您可能会侥幸逃脱，但是当它必须移动内存时，移动的位置在调用函数中不可用。
fgetc() 的返回类型是int，而不是char。
您应该更喜欢 while 循环而不是 do ... while 循环。如果您的代码遇到 EOF，它会尝试将存储的字符值放入数组中。
（仍然是个问题）string 的初始分配在很大程度上被忽略了；它被视为零长度分配。
您一次增加一个字符存储的空间 - 这是低效的。修改后的代码每次大约使可用空间增加一倍。它还为终止的空字节分配足够的空间。
未修复：名称read() 不被标准 C 保留，但如果您使用任何 POSIX 函数，则有效保留。

您（仍然）无法报告错误。我实际上更喜欢这样的功能：

char *read_line(FILE *stream)
{
    int c;
    int counter = 0;
    int available = 2;
    char *string = malloc(available);

    if (string == 0)
        return string;

    while ((c = fgetc(stream)) != EOF && c != ENTER)
    {
        if (counter >= available)
        {
            int new_size = (available + 2) * 2;
            void *space = realloc(string, new_size);
            if (space == 0)
            {
                free(string);
                return 0;
            }
            string = space;
            available = new_size;
        }
        string[counter++] = c;
    }
    string[counter] = '\0';
    return string;
}

您还应该查找 POSIX getline() 功能。

【讨论】：

返回string而不是void不是更好吗？
是的 - 我或多或少保留了原始界面，但它有问题（尤其是初始分配无关紧要）。
您在这个答案中投入的时间确实令人印象深刻。
是的 - 我可能应该去寻找答案的先前版本；我可以肯定这不是我第一次回答同样的问题，更不用说其他人这样做了。
问题解决了！ @JonathanLeffler 感谢您的帮助和耐心。

【解决方案2】：

您的功能运行良好。查看以下代码：

#include <stdio.h>
#define ENTER 10 //'\n' ASCII code

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

(*string)[counter-1] = '\0';
}
int main()
{
char *p = malloc(1);
read(fopen("sam.txt", "r"), &p);
printf("%s\n", p);
return 0;
}

sam.txt 的内容打印准确。我猜你是通过file name 而不是file pointer to the file。您还需要传递指针的引用以在 p 上应用 realloc。希望这会有所帮助。

【讨论】：