c - 使用 fgets 将行复制到字符串数组中？

Question

我正在尝试编写一个 C 程序，该程序旨在反转文件的行。我在 C 方面仍然很无能(尽管我有 Java 背景)，所以我很可能在指针等方面犯错误，但我已经尝试每次都查阅手册。这有点像是一项任务。

该程序的要点是反转文件的内容，数量最多为 MAX_LINES，每行长度不超过 MAX_CHARS。我想尝试的方法如下:使用 fgets 从文件中读取 80 个字符或直到 EOL，存储该字符串，然后使用侧面的计数器重复该过程，直到达到 EOF 或 MAX_LINES。之后，我只是将相同的字符串放入不同的数组中，从 secondary_array[counter] 到 0。但是，我在实际将字符串放入第一个数组时遇到了问题。这是我到目前为止所拥有的:

1 #include <stdio.h>
2 #include <string.h>
3
4 #define MAX_LINES 100
5 #define MAX_CHAR 80
6
7 int main(int argc, char *argv[])
8 {
9         if(argc != 2)
10                 goto out;
11         FILE *fp;
12         char *str[MAX_CHAR];
13         char buffer[MAX_CHAR];
14         char *revstr[MAX_CHAR];
15         int curr_line = 0, i = 0, j =0;
16
17         fp = fopen(argv[1],"r");
18
19         out:
20         if (fp == NULL && argc != 2){
21                 printf("File cannot be found or read failed.\n");
22                 return -1;
23         }
24
25         /* printf("This part of the program reverses the input text file, up to a  maximum of 100 lines and 80 characters per line.\n The reversed file, from the last (or    100th) line to the first, is the following:\n\n"); */
26
27         /* fgets reads one line at a time, until 80 chars, EOL or EOF */
28         while(curr_line < MAX_LINES && ((fgets(buffer, MAX_CHAR, fp)) != NULL)){
29
30                 str[i] = buffer;
31                 ++j;
32                 ++i;
33         }
34
35         for(i = 0; i < 4; ++i) 
36                 printf("%s \n", str[i]);
37
38
39         printf("END OF PROGRAM RUN.");
40         return 0;
41 }

在同一目录中，我有一个“txt”文件，其中包含以下行:

is this a test
this is a test
this is not a test

当我编译并运行该程序(./a.out txt)时，我得到以下输出:

this is not a test

this is not a test

this is not a test

END OF PROGRAM RUN.

显然，这意味着正在覆盖相同的位置，但我不确定如何纠正这个问题(如上所述，指针对我来说仍然很陌生)。谁能澄清这里发生了什么？我需要使用二维数组吗？任何帮助将不胜感激。

Answer 1

滥用缓冲区

问题在于，您一直将指针 buffer 存储在 str[i] 中，但 buffer 只包含最后读取的一行。

您必须复制读取的行，可能使用strdup():

str[i++] = strdup(buffer);

需要担心错误检查，但从概念上讲，这是处理它的一种简单方法。

Could you please clarify as to why that is? The declaration of fgets() being char *fgets(), I got the impression that it momentarily returns a pointer to the line currently read from the file!

实际上，fgets() 返回两个值之一:

• EOF 或其他 I/O 错误为 NULL，或者
• buffer，指向传递给函数的字符串的指针。

特别是，它不会自动为您创建新的存储空间。如果您想要的话，您可能需要查看 readline()来自 POSIX。

因此，对 fgets() 的调用每次都使用相同的数组 buffer，以便每个连续的行读取都会覆盖前一行。这就是为什么您需要以某种方式复制每一行，也是为什么我建议使用 strdup() 作为实现这一点的简单方法。

<小时/>

strdup()

函数 strdup() 是 POSIX 的一部分，但不是标准 C 的一部分。但是，它很容易实现:

char *strdup(const char *str)
{
    size_t len = strlen(str) + 1;
    char  *dup = malloc(len);
    if (dup != 0)
        memmove(dup, str, len);
    return(dup);
}

<小时/>

错误报告

您应该尽可能避免goto，使用此代码可以轻松完成此操作。您的双重用途错误消息实际上最终会让省略参数或提供太多参数的用户感到困惑。提供两条单独的错误消息，一条是程序参数数量不正确，另一条是无法打开文件。

if (argc != 2)
{
    fprintf(stderr, "Usage: %s file\n", argv[0]);
    exit(1);
}
fp = fopen(argv[1], "r");
if (fp == 0)
{
    fprintf(stderr, "%s: failed to open file %s (%d: %s)\n",
            argv[0], argv[1], errno, strerror(errno));
    exit(1);
}

请注意，错误应该在 stderr 上报告，而不是在 stdout 上；这就是错误输出 channel 的用途。