我想在链接列表上实现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/