Question

我花了很多时间在这上面，我终于让它工作了（因为它保存了值并打印出来）。我想按字母顺序对它进行排序。如果我将值添加到我的列表中，它们不会按照我添加它们的相同顺序打印出来，但它似乎有点随机......似乎strcpy不能按照我想象的方式工作。 ..

#include <iostream>
#include <cstdlib>
#include <string.h>
#include <stdio.h>
using namespace std;


typedef struct list list_t;
struct list{
    list *next;
    char *nam;
    int age;

};

static list_t *top = NULL;


void add_value(char * name1, int age){
    list * neww = (list_t*)malloc(sizeof( list_t));
    //tmp = top;
    neww->age = age;
    neww->nam = (char*)malloc(strlen(name1) + 1);
    neww->next = NULL;
    strcpy(neww->nam, name1);
    list * tmp = (list_t*)malloc(sizeof(list_t));
    tmp = top;
    if (tmp==NULL){
        top = neww;
        //printf("%s\n", top->nam);
    }else
    while (1){


        if (tmp->next == NULL){
            tmp->next = neww;
            break;
        }
        //printf("top - %s %d\n neww - %s %d\n tmp - %s %d", top->nam, top->age, neww->nam, neww->age, tmp->nam, tmp->age);
        if (strcmp(neww->nam, tmp->nam)>=0){
            neww->next = tmp->next;
            tmp->next = neww;
            break;
        }

        tmp = tmp->next;


    }

}


void print(){
    list * tmp = (struct list*)malloc(sizeof(struct list));
    tmp = top;
    while (tmp){
        printf("%s %d\n", tmp->nam, tmp->age);
        tmp = tmp->next;

    }


}


int main(){

    char namee[100];
    int age;
    for (int i = 0; i < 5; i++){
        scanf("%s %d", &namee, &age);
        add_value(namee, age);
    }

    print();

    return 0;
}

Answer 1

我发现了两件看似错误的事情。

a）这一行

list * tmp = (struct list*)malloc(sizeof(struct list));

错误（在两个函数中使用）。你需要一个指针 - 而不是一个新元素 - 所以不需要malloc。

b）此代码

    if (tmp->next == NULL){
        tmp->next = neww;
        break;
    }

当你到达最后一个元素时，

似乎变得活跃。但是，您仍必须检查是否要在列表中已有元素之前或之后插入新元素。您的代码总是将新元素放在后面。考虑一下您的列表只有1个元素的情况。

BTW - 如果你想编写c ++，你应该使用std::string而不是c风格的char数组。另外，请查看std::multimap而不是您自己的链接列表。

单链表，基于* char输入的自排序

1 个答案: