我想在链接列表上实现QuickSort,但是我无法从文本文件读取到列表中并打印所有节点。我只将最后一个元素打印为输出。我究竟做错了什么?

我的文本文件如下所示(密码及其使用频率):

asdfgh 31554
snoopy1 15637
qwertyuiop 24372
soccer 21208
.
.


这是我的结构

struct list_element {
char *password;
int count;
list_element* next;
};

struct list {
list_element* first;
list_element* last;
};


ReadfromData()

void read_data(char* filename, list* mylist)
{

FILE *fp;

char password[128];
int freq;

fp = fopen(filename, "r");

if(fp == NULL)
{
 perror("Error opening file");
 return;
}


while(fgets(password, sizeof(password), fp))
{

 list_element *node = malloc(sizeof(list_element));

 char *token;
 token = strtok(password, " ");
 node->password = strdup(token);

 if( token != NULL ){
 token = strtok(NULL, " ");
 }

 freq = atoi(token);
 node->count = freq;
 node->next = NULL;

 insert_list(node, mylist);
 }

 fclose(fp);
}


在列表中插入infront

void insert_list(list_element* le, list* mylist)

if((mylist->first = NULL)){
mylist->first = le;
}else{
le->next = mylist->first;
mylist->first = le;
}


打印清单

void print_list(list* mylist)

list_element *temp;
temp = mylist->first;

while(temp != NULL)
{
printf("pass %s and count %d \n", temp->password, temp->count);
temp = temp->next;
}


我还编写了一个小函数,在程序的开头调用该函数以详细列出该列表:

void init_list(list* mylist){
mylist = (list*)malloc(sizeof(list));
mylist->first = mylist->last = NULL;
}


但是我也不认为在这里进行malloc也没有意义,因为我已经一个接一个地创建了节点,对吗?有点困惑。

任何建议都很好!

最佳答案

让我们仔细看看init_list函数:

void init_list(list* mylist){
mylist = (list*)malloc(sizeof(list));
mylist->first = mylist->last = NULL;
}


参数mylist是局部变量。这样,当函数结束时它将超出范围,并且对它的所有更改都将丢失。因此,在init_list调用之后将要使用的指针实际上不会被初始化,并且使用时将具有未定义的行为。

有两种解决方案:让init_list不带任何参数,而是返回新列表。或者,您可以通过从调用函数传递指向列表变量的指针来模拟按值传递,这意味着init_list函数采用指向结构的指针的指针。

第二种选择可能看起来像这样

void init_list(list **mylist)
{
    *mylist = malloc(sizeof **mylist);
    (*mylist)->first = (*mylist)->last = NULL;
}


然后使用address-of运算符调用它:

list *mylist;
init_list(&mylist);

关于c - 链表仅打印C中的最后一个元素,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/41898754/

10-12 12:20
查看更多