c - 从文件加载到链表

Question

我正在尝试将未知数量的数据从文件加载到链接列表中

加载函数

void load(FILE *file, Node **head) // while feof somewhere
{
    char tempArtist[30]={'\0'}, tempAlbum[30]={'\0'}, tempTitle[30]={'\0'}, tempGenre[30]={'\0'},tempSpace='\0';
    char tempPlay[100]={'\0'}, tempRating[6]={'\0'}, tempMins[8]={'\0'}, tempSecs[8]={'\0'};

    Data *temp;

    temp=(Data*)malloc(sizeof(Data));

            while(!feof(file))
            {
            fscanf(file,"%s",tempArtist);
            fscanf(file,"%s",tempAlbum);
            fscanf(file,"%s",tempTitle);
            fscanf(file,"%s",tempGenre);
            fscanf(file,"%s",tempMins);
            fscanf(file,"%s",tempSecs);
            fscanf(file,"%s",tempPlay);
            fscanf(file,"%s",tempRating);

            temp->mins=strdup(tempMins);
            temp->secs=strdup(tempSecs);
            temp->album=strdup(tempAlbum);
            temp->artist=strdup(tempArtist);
            temp->genre=strdup(tempGenre);
            temp->song=strdup(tempTitle);
            temp->played=strdup(tempPlay);
            temp->rating=strdup(tempRating);    

            insertFront(head,temp);

            }

}

我遇到的问题是，当我打印列表时，所有条目都与从文件中读取的最后一个条目相同。这与 strdup() 有关，但我无法在不发生访问冲突的情况下将其复制到数据类型。

我可以用来复制从文件中读取的字符串并将其传递给插入的另一种方法(正确的方法)是什么？

插入前面

void insertFront(Node **head, Data *data)
{
    Node *temp=makeNode(data);

    if ((*head) == NULL)
    {
        *head=temp;
    }
    else
    {
        temp->pNext=*head;
        *head=temp;
    }

}

数据结构

typedef struct data
{
    char *artist;
    char *album;
    char *song;
    char *genre;
    char *played;
    char *rating;
    char *mins;
    char *secs;
}Data;

测试文件

snoop
heartbeat
swiggity
rap
03
10
25
4
hoodie
cakeboy
birthday
hiphop
02
53
12
5

Answer 1

您对所有行使用相同的 temp 实例。分配应该在循环内部进行，理想情况下是在您确定整个条目已成功读取之后。

顺便说一下，feof 是 not a good way to control input loops 。您应该检查 fscanf 的返回值。

while (1) {
    if (fscanf(file,"%s",tempAlbum) < 1 ) break;
    if (fscanf(file,"%s",tempArtist) < 1) break;
    // ...

    temp = (Data *) malloc(sizeof(Data));

    temp->album=strdup(tempAlbum);
    temp->artist=strdup(tempArtist);
    // ...

    insertFront(head, temp);
}

请注意，只有在读取整个记录后，新节点的分配才会发生。

还有其他方法可以改进代码。例如，包含数字数据的字符串的缓冲区非常短。如果出现行错误并且您将较长的行读入如此短的缓冲区怎么办？另外，您的输入似乎是逐行的，因此最好使用 fgets 而不是 fscan(f, "%s")，后者只会读取 "单词”，并且在处理带有空格的行时会出现问题。