链表在第二次迭代期间打印符号而不是单词

在第一次迭代期间释放了链表，因此第二次迭代具有未定义的行为，因为该指针已成为无效指针，并且列表节点的内存内容已更改。从它们读取具有未定义的行为，任何事情都可能发生，包括程序因无效的内存访问而停止或您观察到的随机输出。head

您应该编写函数：一个用于打印列表内容，另一个用于释放列表。将这些操作组合在一起会造成混淆，并导致诸如此类的逻辑错误。

另请注意以下备注：

• 的返回类型为 。修改语句，在错误和成功时返回，即：正常操作。mainintreturn10

• printf("File (%s) does not exist\n", argv[1])可能不是正确的诊断：文件可能存在，但如果进程没有适当的访问权限，则文件将失败。您可以使用并输出相应的错误消息：fopenerrno

   #include <errno.h>
   #include <string.h>
  
   [...]
  
       if (f == NULL) {
           fprintf(stderr, "cannot open %s: %s\n", argv[1], strerror(errno));
           return 1;
       }
  

• 而不是您应该测试执行两个转换是否成功：它应该返回 .更好的是：一次读取一行输入，并用于转换行内容，并使用包含违规输入行的显式消息报告转换错误。fscanf(f, "%s %d", line->word, &(line->frequency)) != EOFfscanf()2fgetssscanf()

• head被释放两次：最后必须删除。free(current)

这是修改后的版本：

#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define WORD_MAX_LENGTH 255

struct WordFreq {
    char word[WORD_MAX_LENGTH];
    int frequency;
    struct WordFreq *next;
} WordFreq;

struct WordFreq *head = NULL;

void insert(struct WordFreq *newNode) {
    if (head == NULL) {
        head = newNode;
    } else {
        struct WordFreq *current = head;
        while (current->next != NULL) {
            current = current->next;
        }
        current->next = newNode;
    }
    newNode->next = NULL;
}

void print_list(const struct WordFreq *node) {
    while (node != NULL) {
        printf("%s %d\n", node->word, node->frequency);
        node = node->next;
    }
}

void free_list(struct WordFreq *node) {
    while (node != NULL) {
        struct WordFreq *temp = node;
        node = node->next;
        free(temp);
    }
}

int main(int argc, char *argv[]) {
    char line[WORD_MAX_LENGTH + 20];

    if (argc != 2) {
        fprintf(stderr, "Please run as %s [filename]\n", argv[0]);
        return 1;
    }

    FILE *f = fopen(argv[1], "r");
    if (f == NULL) {
        fprintf(stderr, "Cannot open %s: %s\n", argv[1], strerror(errno));
        return 1;
    }

    printf("File content in %s:\n", argv[1]);

    while (fgets(line, sizeof line, f)) {
        struct WordFreq *node = malloc(sizeof(*node));
        if (node == NULL) {
            fprintf(stderr, "Cannot allocate node for %s\n", line);
            fclose(f);
            return 1;
        }
        if (sscanf(line, "%254s %d", node->word, &node->frequency) == 2) {
            printf("%s %d\n", node->word, node->frequency);
            insert(node);
        } else {
            fprintf(stderr, "invalid line: %s\n", line);
            free(node);
        }
    }
    fclose(f);

    printf("\nContent of linked list:\n");

    print_list(head);
    // ...
    free_list(head);
    return 0;
}