从文件打印字符串时遇到困难

Question

我的任务是从文件中读取单词并分别打印每个单词。 到目前为止，我得到了这个：

#define MAX 100000
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
typedef struct node *ptr;
typedef struct node {
    char *data;
    ptr next;
} node;

ptr head = NULL;

void add(char *data) {
        node *new_node = malloc(sizeof(node));
        new_node->data = data;
        new_node->next = head;
        head = new_node;
    printf("Adding new word:%s\n",data);
}

void print_list() {
    node *current = head;
        while (current != NULL) {
            printf("%s\n", current->data);
            current = current->next;
        }
}

void free_list(ptr *hnode)
{
    ptr p;
    while(*hnode){
        p = *hnode;
        *hnode = (*hnode)->next;
        free(p);
    }
}

int main(int argc, char *argv[]) {
    FILE *file;
    char buffer[MAX];
    char ch;
    int i = 0;

    if (argc != 2) {
        printf("Usage: %s <file>\n", argv[0]);
            exit(EXIT_FAILURE);
        }

        file = fopen(argv[1], "r");
        if (file == NULL) {
            printf("Error opening file\n");
            exit(EXIT_FAILURE);
        }

    while(i < MAX-1 && !feof(file))
    {
        if((ch = fgetc(file)) == ' '|| ch == '\n')
        {
            buffer[i] = '\0';
            add(buffer);
            i = 0;
        }
        else{
            if(ch == EOF)
                  break;
            buffer[i++] = ch;
        }
    }

        print_list();
        free_list(&head);
        exit(EXIT_SUCCESS);
}

当我尝试运行代码时，没有汇编或运行时错误，但是我只会得到最后一个单词和一个奇怪的符号，例如：

Adding new word:Hello
Adding new word:World!
Adding new word:Does
Adding new word:my
Adding new word:code
Adding new word:work?
work?
work?
work?
work?
work?
work?

添加新单词正确并正确读取文件时的部分，我的问题是打印部分。 编辑：现在我有了适当的EOF检查，仍然无法按预期工作。

Answer 1

当您调用添加（Buffer）时，您每次都会传递完全相同的指针（从技术上讲，数组不是指针，但对于此讨论而言足够接近）。该指针被分配给节点的数据字段。这意味着每个节点都包含对完全相同的字符串的引用。这就是为什么每个节点都打印出相同的值的原因。此外，随着您的代码变得更加复杂，如果buffer的终生终止，则可能会冒着这些参考文献无效的风险。

您要做的是使用malloc给出每个节点自己的字符串副本。

void add(char *data) {
    size_t len = strlen(data);
    node *new_node = malloc(sizeof(node)); // Note: You should check this return value.
    new_node->data = malloc(len+1); // Note: Ditto.
    memcpy(new_node->data, data, len+1);
    new_node->next = head;
    head = new_node;
    printf("Adding new word:%s\n",data);
}

void free_list(ptr *hnode)
{
    ptr p;
    while(*hnode){
        p = *hnode;
        *hnode = (*hnode)->next;
        free(p->data);
        free(p);
    }
}

Answer 2

在函数中，添加所有节点的数据成员数据指向MAIM中声明的同一数组缓冲区，

void add(char *data) {
        node *new_node = malloc(sizeof(node));
        new_node->data = data;
        new_node->next = head;
        head = new_node;
    printf("Adding new word:%s\n",data);
}

因此函数print_list将输出最后存储在缓冲区中的内容。

您需要动态地将传递的字符串的副本制作。

同时，循环

while(i < MAX-1 && !feof(file))
{
    if((ch = fgetc(file)) == ' '|| ch == '\n')
    {
        buffer[i] = '\0';
        add(buffer);
        i = 0;
    }
    else{
        buffer[i++] = ch;
    }
}

并没有很大的意义。首先，变量CH应声明为具有int的类型。循环中的条件

while(i < MAX-1 && !feof(file))

可能会导致将eof存储在缓冲区中。输出证明了这一点。

的问题中的此更新

       if(ch == EOF)
              break;

（编辑：您回答后进行

。）另外，按一个角色读取字符串效率低下。您应该使用函数fscanf而不是fgetc。

注意定义全局变量head

ptr head = NULL;

以及某些功能取决于全局变量是一个坏主意。

同样，在程序的某些部分中，您正在使用typedef名称ptr，而在程序的其他部分中，则使用type node *。这仅使代码的读者感到困惑。