我必须在链表上创建一个快速排序（在C中）。 我的第一个和最后一个指针是一个枢轴（在这段代码中它是列表的第一个元素）。 我必须使用的结构：
typedef struct list_element list_element;
struct list_element {
char *password;
int count;
list_element* next;
};
typedef struct list list;
struct list {
list_element* first;
list_element* last;
};
我有一个包含100个密码和计数的文件。 像这样： 密码1 123（下一行）密码2 435（下一行）密码3 133 ... 密码必须在此程序结束时（根据其计数）进行排序。 左右列表不需要任何额外的内存分配，因为我只需要使用下一个指针。 （这就是练习中暗示的内容。）
给定的主要功能：
int main(int argc, char** args)
{
if (argc != 2)
{
printf("Nutzung: %s <Dateiname>\n",args[0]);
return 1;
}
list mylist;
init_list(&mylist);
read_data(args[1],&mylist);
qsort_list(&mylist);
printf("Sortierte Liste:\n");
print_list(&mylist);
free_list(&mylist);
return 0;
}
我已经初步确定了我的清单：
void init_list(list* mylist)
{
mylist->first = NULL;
mylist->last = NULL;
}
在结尾插入一个新元素（passwort =文件中的密码，hauefigkeit =文件中的count）：
void insert_list(list_element* le, list* mylist)
{
if (mylist->first != NULL) {
le->next = mylist->last;
mylist->last = le;
le->next= NULL;
}
else {
mylist->last->next = le;
mylist->last = le;
mylist->last->next = NULL;
}
}
从文件中读取数据：
void read_data(char* filename, list* mylist)
{
FILE *file_in = fopen(filename, "r");
if (file_in == NULL) {
perror("Could not open input file!");
exit(1);
}
char buffer[999] = "0";
char *passwort = (char*) calloc(1,sizeof(passwort));
int haeufigkeit = 0;
while (fgets(buffer, sizeof(buffer), file_in) != NULL) {
sscanf(buffer, "%s %d", passwort, &haeufigkeit);
list_element* le = (list_element*)calloc(1,sizeof(list_element));
for(int i = 0; i <=100; i++) {
le->password[i] = passwort[i];
}
le->count = haeufigkeit;
le->next = NULL;
insert_list(le, mylist);
}
fclose(file_in);
}
列表的分区：
list_element* partition( list* input, list* left, list* right )
{
list_element* pivot = NULL;
if (input->first != NULL) {
list_element* temp;
pivot = input->first;
input->first = input->first->next;
pivot->next = NULL;
left->first = NULL;
right->first = NULL;
while (input->first != NULL) {
if((pivot->count)>(input->first->count)){
temp=input->first->next;
insert_list(input->first, left);
input->first=temp;
}
else {
temp = input->first->next;
insert_list(input->first, right);
input->first = temp;
}
}
}
return pivot;
}
实际的快速排序：
void qsort_list(list* mylist)
{
if(mylist->first == mylist->last){
}
else{
list* left = calloc(1,sizeof(list));
list* right= calloc(1,sizeof(list));
list_element* pivot = partition(mylist, left, right);
qsort_list(left);
qsort_list(right);
if(left->first == NULL){
mylist->first = pivot;
}
else{
mylist->first = left->first;
left->last->next = pivot;
}
if(right->first == NULL){
pivot->next = NULL;
mylist->last = pivot;
}
else{
pivot->next = right->first;
mylist->last = right->last;
}
free(right);
free(left);
}
}
在最后的打印列表中：
void print_list(list* mylist)
{
list_element *elem = mylist->first;
while (elem != NULL) {
printf("%s %d\n", elem->password, elem->count);
elem = elem->next;
}
}
免费清单：
void free_list(list* mylist)
{
list_element *current;
list_element *second;
current = mylist->first;
while (current != NULL) {
second = current->next;
free(current);
current = second;
}
}
语法应该没问题。 GCC（c99，Wall）编译没有任何问题。
但是存在分段错误。我一直在寻找几个小时，我不知道问题出在哪里。也许你可以帮我解决这个问题。
在前两个答案之后，没有任何分段错误。但是read_data函数仍然存在问题。 该程序无法正确读取passworts。也许我误解了你关于阅读功能的答案。 这是当前的功能：
void read_data(char* filename, list* mylist)
{
FILE *file_in = fopen(filename, "r");
if (file_in == NULL) {
perror("Could not open input file!");
exit(1);
}
char buffer[999] = "0";
int haeufigkeit = 0;
while (fgets(buffer, sizeof(buffer), file_in) != NULL) {
char passwort[100];
sscanf(buffer, "%s %d", passwort, &haeufigkeit);
list_element* le = (list_element*)
calloc(1,sizeof(list_element));
le->password = passwort;
le->count = haeufigkeit;
le->next = NULL;
insert_list(le, mylist);
}
fclose(file_in);
}
答案 0 :(得分：1)
正如莱昂纳多·阿尔维斯·马查多所指出的那样，当遇到C / C ++程序出现问题时，第一反应就是使用像gdb这样的调试器来运行它。以下是基础知识：
gcc -g main.c -o main
gdb main
(gdb) run
请注意-g编译标志：这会将调试信息添加到可执行文件中。
在read_data中，行
for(int i = 0; i <=100; i++) {
le->password[i] = passwort[i];
}
真的让我烦恼。您为passwort分配空间（顺便说一下，你永远不会释放它），并尝试将其复制到le->password，这是一个简单的指针（没有分配的空间）。你真正需要的是让le->password 指向 passwort，即
le->password = passwort;
在free_list中，在释放passwort空格之前，不要忘记取消分配list_element空格：
while (current != NULL) {
second = current->next;
free(current->password);
free(current);
current = second;
}
答案 1 :(得分：1)
您的程序遇到的第一个问题是read_data()没有为passwort分配足够的空间。实际上，目前还不清楚为什么你要动态分配这个，但鉴于你正在这样做，sizeof(passwort)是一个指向char的指针的大小（因为那是passwort） - 可能要么4或8个字节。稍后，当您（尝试）将其内容复制到列表元素时，您似乎假设分配的空间长度为100个字节。为什么不简单地将它声明为100-char数组？
char passwort[100];
实际上，如果你也以同样的方式声明list_element.passwort，那么你在循环中复制密码的代码是正确的，虽然有点非惯用。
正如@Derlin所观察到的那样，代码是有问题的。然而，他提议的解决方案不正确;你不能让列表元素指向本地passwort，只要它只为整个例程分配一次。然后所有列表元素将具有相同的密码字符串，这不是您想要的。如果您希望列表元素包含指向密码的指针，就像现在一样，那么您将需要在循环中移动passwort 的声明和分配，以便获得为每个列表元素分配的单独密码空间。然后，分配le->password = passwort的建议是正确的。
另一个早期问题是您的insert_list()功能严重受损。
首先考虑当您尝试将元素插入空列表时会发生什么，由init_list()初始化。列表的next和last成员都将为空，因此insert_list()将尝试执行此代码：
mylist->last->next = le;
mylist->last = le;
mylist->last->next = NULL;
注意mylist->last为空，因此第一行通过尝试取消引用空指针来调用未定义的行为。分段错误是一种非常合理的观察结果。您可以通过将第一行更改为
mylist->next = le;
现在考虑当您尝试插入非空列表时会发生什么。在这种情况下，您执行以下行：
le->next = mylist->last;
mylist->last = le;
le->next= NULL;
由于你的意图是在最后插入新元素（即追加它），你将新元素的next指针设置为列表的最后一个元素是很奇怪的。稍后用NULL覆盖该值会特别奇怪。你似乎把它向后：你想要将初始last元素设置为指向新元素作为它的下一个元素，而不是相反：
mylist->last->next = le;
实际上，这正是空列表案例错误的代码，但是当列表非空时它就没问题了。
总的来说，你的功能还有一个奇怪的缺乏并行性和一些隐藏的代码重复。我可能会写一些更像这样的整体功能：
void append_to_list(list_element* le, list* mylist)
{
le->next= NULL;
if (mylist->first != NULL) {
mylist->last->next = le;
mylist->last = le;
}
else {
mylist->first = le;
mylist->last = le;
}
}