C语言 从.txt文件创建字符串列表

Question

我正在尝试创建一个字符串列表，从.txt文件中读取单词。我的代码仅在.txt文件包含少量单词并且我不知道为什么时才有效，我认为这是我的代码内存分配问题。

#include <stdio.h>
#include <stdlib.h> struct s_nodo {
    char* word;
    struct s_nodo*sig; }; typedef struct s_nodo* t_nodo;
void add (t_nodo*,char*); void print(t_nodo);
int main() {
    char aux[30];
    t_nodo lista=NULL;
    FILE*fd;
    fd=fopen("c:\texto.txt","r");
    while(!feof(fd))
    {
        fscanf(fd,"%s",aux);
        add(&lista,aux);
    }
     print(lista);
    return 0; }

void add (t_nodo*lista,char *aux) {
    if(*lista==NULL)
    {
        *lista=malloc(sizeof(t_nodo));
        (*lista)->word=malloc((strlen(aux+1))*sizeof(char));
        strcpy((*lista)->word,aux);
        (*lista)->sig=NULL;
    }
    else add (&(*lista)->sig,aux);
}
void print (t_nodo lista) {
    if(lista!=NULL)
    {
        printf("-%s-",lista->word);
        print(lista->sig);
    }
}

Answer 1

您正在为指针指向结构的大小分配内存，而您需要为结构本身的大小分配内存。

改变

  *lista=malloc(sizeof(t_nodo));

自

  *lista=malloc(sizeof(struct s_nodo));

此外，您使用了错误的表达式将内存分配给word。

(*lista)->word=malloc((strlen(aux+1))*sizeof(char));

应该是

(*lista)->word=malloc( (strlen(aux) + 1 );  //sizeof(char) == 1 in C

也就是说，请参阅为什么"while （ ！feof （file））"总是错误的？

Answer 2

你的编码风格在这里导致了这个错误

(*lista)->word=malloc((strlen(aux+1))*sizeof(char));
                        //       ^

1. 不要使用 sizeof(char)，因为它是 1 并且是强制性的，它只是帮助您忽略了这个问题。
2. 使用更多的空白，这将在您眼前轻松分隔令牌。
3. 在使用指针之前，请始终检查malloc()是否未返回NULL。

所以应该是

(*lista)->word = malloc(strlen(aux) + 1);

看看现在怎么清楚了，不是吗？