c 的自定义 getLine() 函数

Question

我需要一个函数/方法，它将接收一个 char 数组并将其设置为从标准输入读取的字符串。它需要返回读取的最后一个字符作为它的返回类型，所以我可以确定它是到达行尾还是文件尾标记。

这是我目前所拥有的，我有点基于 here 的代码

更新：我更改了它，但现在它只是在文本后按 Enter 时崩溃。我知道这种方式效率低下，并且 char 不是 EOF 检查的最佳选择，但现在我只是想让它返回字符串。我需要它以这种方式而不是其他方式来做到这一点。我需要字符串是行的确切长度，并返回一个值，即换行符或 EOF int，我相信它仍可用于 char 值。

这个程序是 C 而不是 C++

char getLine(char **line);

int main(int argc, char *argv[])
{
    char *line;
    char returnVal = 0;

    returnVal = getLine(&line);
    printf("%s", line);

    free(line);

    system("pause");
    return 0;
}

char getLine(char **line) {
    unsigned int lengthAdder = 1, counter = 0, size = 0;
    char charRead = 0;

    *line = malloc(lengthAdder);
    while((charRead = getc(stdin)) != EOF && charRead != '\n')
    {
        *line[counter++] = charRead;
        *line = realloc(*line, counter);
    }

    *line[counter] = '\0';

    return charRead;
}

提前感谢您的帮助！

Answer 1

您将malloc() 的结果分配给line 的本地副本，因此在getLine() 函数返回后它不会被修改（尽管您认为它是）。您要做的就是返回它（而不是使用输出参数）或传递它的地址（通过引用传递它）：

void getLine(char **line)
{
    *line = malloc(length);

    // etc.
}

然后这样称呼它：

char *line;
getLine(&line);

Answer 2

您使用realloc 的方式不正确。如果返回NULL，则内存块将丢失。

这样使用realloc比较好：

char *tmp;
...
tmp = realloc(line, counter);
if(tmp == NULL)
   ERROR, TRY TO SOLVE IT
line = tmp;

Answer 3

您的关键问题是 line 指针值不会传播到 getLine() 函数之外。解决方案是将 指向 line 指针 的指针作为参数传递给函数 - 像 getLine(&line); 一样调用它，而函数将被定义为采用参数 char **line .在函数中，在您现在使用 line 的所有地方，您将使用 *line 代替，即取消对指针的引用并使用ma​​in() 中指针指向的变量。希望这不会太混乱。 :-) 试着把它画在一张纸上。

（一个棘手的部分 - 您必须将 line[counter] 更改为 (*line)[counter] 因为您首先需要取消引用指向字符串的指针，并且只有 然后 才能访问字符串中的特定字符。）

您的代码还有其他几个问题：

• 您使用 char 作为 charRead 的类型。但是，EOF 常量不能使用 char 表示，您需要使用 int - 两者都作为 charRead 的类型> 和 getLine() 的返回值，这样您就可以真正区分换行符和文件结尾。
• 您忘记返回从 getLine() 函数读取的最后一个字符。 :-)
• 您在每次添加字符后重新分配缓冲区。这不是非常有效，因此是一种相当丑陋的编程实践。使用另一个变量来跟踪分配的空间量并不太难，然后 (i) 从分配合理的内存块开始，例如64 字节，因此理想情况下，您永远不会重新分配 (ii) 仅当您需要基于比较计数器和分配大小跟踪器时才扩大分配。两种重新分配策略很常见 - 将分配的大小加倍或按固定步长增加分配。