用户制作的 fgets() 与标准库 fgets()

Question

标准库函数fgets()有两个缺点：

• 函数的第二个参数类型为int
• 它在提供的缓冲区中留下一个尾随换行符

我做了一个类似于fgets() 的简单函数，排除了上述缺点，以尝试提高我的一个程序的效率，该程序获取文本文件的行并在新的位置终止char 数组-line 字符使用函数strcspn()。

但它真的更有效吗？为什么标准库函数比以下幼稚的实现具有优势？

#include <stdio.h>


char *my_fgets(char *buf, size_t maxCount, FILE *stream);


int main(int argc, char **argv)
{
    if (argc < 2)
    {
        fprintf(stderr, "Usage: %s [filename]", argv[0]);
    }

    FILE *fp;
    fp = fopen(argv[1], "r");

    if (!fp)
    {
        perror(argv[1]);
        return 1;
    }

    char buf[256];
    /*while (fgets(buf, sizeof(buf), fp))
    {
        buf[strcspn(buf, "\n")] = '\0';
        // . . .
        puts(buf);
    }*/

    while (my_fgets(buf, sizeof(buf) - 1, fp))
    {
        puts(buf);
    }

    return 0;
}


char *my_fgets(char *buf,
    size_t maxCount, FILE *stream)
{
    int ch;
    size_t n = 0;

    while ((ch = fgetc(stream)) != EOF)
    {
        if (ch == '\n' || n == maxCount)
        {
            break;
        }
        else
        {
            buf[n++] = ch;
        }
    }
    if (n == 0 && ch == EOF)
    {
        return NULL;
    }
    buf[n] = '\0';
    return buf;
}

Answer 1

性能

您使用fgetc() 逐个获取字符保证my_fgets() 无法与经过合理优化的解决方案竞争。为了获得良好的性能，您需要通过read() 系统调用将一些字符读入缓冲区，然后直接从该缓冲区使用数据。对每个字节执行完整的函数调用是很多的开销。

安全

fgets() 对于一般用途不安全：它会分割长行。这种情况通常是出乎意料的（例如，您的示例代码也不处理这种情况），并且当向您的程序提供很长的输入行时，通常会导致错误的行为。尾随\n 的存在实际上可以帮助您处理该问题。 如果你想要安全的软件，你必须在使用fgets()时明确处理缓冲区不足的情况。如果不这样做，你肯定有一个等待攻击的错误。您的实现甚至没有尝试解决这个问题。

这就是为什么我强烈建议使用可以处理任何长度的输入行的输入函数。在getline() 的 POSIX 兼容系统上：此函数为您分配足够大的缓冲区，因此您唯一的限制是可用 RAM。如果getline() 在您的平台上不可用，我建议重新实现其功能，而不是“更好”的fgets() 版本。

Answer 2

真的更高效吗？

没有 - 没有更多在性能上有效。 my_fgets() 充其量可以有类似的效率，但即使这样也不太可能，因为 fgets() 可以访问精心制作的汇编代码，而 my_fgets() 不能 @Peter。

是的，如果功能目标是获取一行用户输入和not save the '\n'，它可能具有编码效率，因为这比fgets() ... code_to_rid_\n() 更少编码

---

但效率次于功能。

my_fgets() 与 input errors 有小问题。

my_fgets() 可以丢弃字符，不像fgets() @melpomene

size_t maxCount 的作用与 fgets() 的类似参数相差 1。

---

作为标准库fgets() 的良好尝试有各种弱点，迫使编码人员尝试更好的低级输入行函数。