在 C 中逐行浏览文本文件

Question

我一直在为我的 CIS 课程做一个小练习，并且对 C 用于读取文件的方法感到非常困惑。我真正需要做的就是逐行读取文件，并使用从每一行收集的信息进行一些操作。我尝试使用 getline 方法和其他没有运气的方法。 我的代码目前如下：

int main(char *argc, char* argv[]){
      const char *filename = argv[0];
      FILE *file = fopen(filename, "r");
      char *line = NULL;

      while(!feof(file)){
        sscanf(line, filename, "%s");
        printf("%s\n", line);
      }
    return 1;
}

现在我遇到了 sscanf 方法的 seg 错误，我不知道为什么。我是一个总 C 菜鸟，只是想知道我是否遗漏了一些重要的事情。 谢谢

Answer 1

这么几行就这么多问题。我可能忘记了一些：

• argv[0]是程序名，不是第一个参数；
• 如果你想读入一个变量，你必须分配它的内存
• 一个永远不会在 feof 上循环，一个在 IO 函数上循环直到它失败，然后 feof 用于确定失败的原因，
• sscanf是用来解析一行的，如果你想解析一个文件，使用fscanf，
• “%s”将作为 ?scanf 系列的格式在第一个空格处停止
• 读取一行，标准函数是fgets，
• 从 main 返回 1 表示失败

所以

#include <stdio.h>

int main(int argc, char* argv[])
{
    char const* const fileName = argv[1]; /* should check that argc > 1 */
    FILE* file = fopen(fileName, "r"); /* should check the result */
    char line[256];

    while (fgets(line, sizeof(line), file)) {
        /* note that fgets don't strip the terminating \n, checking its
           presence would allow to handle lines longer that sizeof(line) */
        printf("%s", line); 
    }
    /* may check feof here to make a difference between eof and io failure -- network
       timeout for instance */

    fclose(file);

    return 0;
}

Answer 2

要从文件中读取一行，您应该使用fgets 函数：它从指定文件中读取一个字符串，直到换行符或EOF。

在您的代码中使用sscanf 根本不起作用，因为您使用filename 作为格式字符串，用于从line 读取到常量字符串文字%s。

SEGV的原因是你写入了line指向的未分配内存。

Answer 3

除了其他答案之外，在最近的 C 库（符合 Posix 2008）上，您可以使用getline。请参阅this answer（相关问题）。

Answer 4

假设您正在处理一些其他分隔符，例如 \t 制表符，而不是 \n 换行符。

一种更通用的分隔符方法是使用getc()，它一次抓取一个字符。

请注意，getc() 返回一个int，以便我们可以测试与EOF 是否相等。

其次，我们定义了一个char 类型的数组line[BUFFER_MAX_LENGTH]，以便在堆栈上最多存储BUFFER_MAX_LENGTH-1 个字符（我们必须将最后一个字符保存为\0 终止符）。

使用数组避免了使用malloc 和free 在堆上创建正确长度的字符指针的需要。

#define BUFFER_MAX_LENGTH 1024

int main(int argc, char* argv[])
{
    FILE *file = NULL;
    char line[BUFFER_MAX_LENGTH];
    int tempChar;
    unsigned int tempCharIdx = 0U;

    if (argc == 2)
         file = fopen(argv[1], "r");
    else {
         fprintf(stderr, "error: wrong number of arguments\n"
                         "usage: %s textfile\n", argv[0]);
         return EXIT_FAILURE;
    }

    if (!file) {
         fprintf(stderr, "error: could not open textfile: %s\n", argv[1]);
         return EXIT_FAILURE;
    }

    /* get a character from the file pointer */
    while(tempChar = fgetc(file))
    {
        /* avoid buffer overflow error */
        if (tempCharIdx == BUFFER_MAX_LENGTH) {
            fprintf(stderr, "error: line is too long. increase BUFFER_MAX_LENGTH.\n");
            return EXIT_FAILURE;
        }

        /* test character value */
        if (tempChar == EOF) {
            line[tempCharIdx] = '\0';
            fprintf(stdout, "%s\n", line);
            break;
        }
        else if (tempChar == '\n') {
            line[tempCharIdx] = '\0';
            tempCharIdx = 0U;
            fprintf(stdout, "%s\n", line);
            continue;
        }
        else
            line[tempCharIdx++] = (char)tempChar;
    }

    return EXIT_SUCCESS;
}

如果你必须使用char *，那么你仍然可以使用这个代码，但是你strdup()line[]数组，一旦它被填满了一行的输入。完成后必须free这个重复的字符串，否则会出现内存泄漏：

#define BUFFER_MAX_LENGTH 1024

int main(int argc, char* argv[])
{
    FILE *file = NULL;
    char line[BUFFER_MAX_LENGTH];
    int tempChar;
    unsigned int tempCharIdx = 0U;
    char *dynamicLine = NULL;

    if (argc == 2)
         file = fopen(argv[1], "r");
    else {
         fprintf(stderr, "error: wrong number of arguments\n"
                         "usage: %s textfile\n", argv[0]);
         return EXIT_FAILURE;
    }

    if (!file) {
         fprintf(stderr, "error: could not open textfile: %s\n", argv[1]);
         return EXIT_FAILURE;
    }

    while(tempChar = fgetc(file))
    {
        /* avoid buffer overflow error */
        if (tempCharIdx == BUFFER_MAX_LENGTH) {
            fprintf(stderr, "error: line is too long. increase BUFFER_MAX_LENGTH.\n");
            return EXIT_FAILURE;
        }

        /* test character value */
        if (tempChar == EOF) {
            line[tempCharIdx] = '\0';
            dynamicLine = strdup(line);
            fprintf(stdout, "%s\n", dynamicLine);
            free(dynamicLine);
            dynamicLine = NULL;
            break;
        }
        else if (tempChar == '\n') {
            line[tempCharIdx] = '\0';
            tempCharIdx = 0U;
            dynamicLine = strdup(line);
            fprintf(stdout, "%s\n", dynamicLine);
            free(dynamicLine);
            dynamicLine = NULL;
            continue;
        }
        else
            line[tempCharIdx++] = (char)tempChar;
    }

    return EXIT_SUCCESS;
}