0

我正在寻找一种使用C中独特的void函数读取文件和标准输入流的方法。我正在尝试使用此功能：

#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';
}

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

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

并且仅在第一种情况下有效。

PS：最初， inputString 一个 fileContent 已被声明为

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

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

4

2 回答 2

5

它无法正常工作，因为您重新分配了字符串但没有将指针返回到新分配，并且因为您处理错误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);

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

  1. 将指针传递给指针，以便更改可以反映在调用代码中。在原始代码中，如果调用可以增大当前块，您可能会侥幸逃脱realloc()，但是当它必须移动内存时，移动的位置在调用函数中不可用。

  2. 的返回类型fgetc()是int，不是char。

  3. 您应该更喜欢while循环而do ... while不是循环。如果您的代码遇到 EOF，它会尝试将存储的字符值放入数组中。

  4. （仍然是一个问题）最初的分配在string很大程度上被忽略了；它被视为零长度分配。

  5. 您一次增加一个字符存储的空间——这是低效的。修改后的代码每次大约使可用空间增加一倍。它还为终止的空字节分配足够的空间。

  6. 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;
}

您还应该查找 POSIXgetline() 函数。

于 2013-08-31T05:57:00.497 回答
0

那么你的功能工作正常。查看以下代码：

#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而不是 a file pointer to the file。此外，您需要传递指针的引用以在上应用 realloc p。希望这可以帮助。

于 2013-08-31T06:10:51.213 回答