C语言自排序链表运行异常：执行create_a_node后程序停滞无报错

从你给出的代码片段来看，程序出现两次输入提示+停滞的问题，主要有几个明显的逻辑和设计问题，我帮你拆解一下：

1. 函数职责混淆导致重复输入 #

你的create_a_node函数参数是int input，但函数内部又写了scanf("%i",&NTA)来获取输入——这就会导致两种输入来源：调用这个函数时传入的input，加上函数内部主动让用户输入的NTA，自然会出现两次输入提示。

如果这个函数的目的是创建单个节点，那它应该只负责根据传入的数值生成节点，而不是自己去读取用户输入；如果是要处理用户输入并构建链表，那函数名和参数设计就不合理，应该改成比如build_sorted_list这类名字，并且不需要传入input参数。

2. 未初始化指针与潜在死循环 #

你声明了node *prvptr;但没有初始化它，这是个野指针。在自排序链表的插入逻辑中，你需要用prvptr跟踪前驱节点，如果它是野指针，后续的指针操作可能会导致程序逻辑混乱，甚至陷入死循环（这就是你看到程序停滞的大概率原因）。

另外，代码片段里的while(NTA != -1)循环，如果循环内部没有再次调用scanf来更新NTA的值，那当用户输入的不是-1时，循环会一直执行下去，直接导致程序停滞。比如你的代码可能漏掉了循环末尾的输入提示和scanf，这会让程序卡在循环里永远出不来。

3. 局部head指针的问题 #

你在create_a_node函数内部声明了node *head = NULL;，这是个局部变量。如果你的意图是让这个函数返回构建好的链表头节点，那没问题，但如果是想修改外部的链表头，那局部的head不会影响外部变量，后续调用这个函数后，外部的链表头还是空的，这会导致链表操作无效。如果需要修改外部的头指针，应该传入指针的指针（node **head）作为参数。

修复建议示例 #

给你一个简化的正确思路，把创建节点和链表插入分开：

第一步：单独的创建节点函数 #

node* create_a_node(int value) {
    node *newptr = (node*)malloc(sizeof(node));
    if (newptr == NULL) {
        printf("Memory allocation failed!\n");
        exit(1);
    }
    newptr->data = value;
    newptr->next = NULL;
    return newptr;
}

第二步：自排序插入函数（处理用户输入+构建链表） #

node* build_sorted_list() {
    node *head = NULL;
    int NTA;
    printf("please enter a number to add (enter -1 to stop): ");
    while (scanf("%i", &NTA) == 1 && NTA != -1) {
        node *new_node = create_a_node(NTA);
        // 插入到排序后的正确位置
        if (head == NULL || NTA < head->data) {
            new_node->next = head;
            head = new_node;
        } else {
            node *curptr = head;
            while (curptr->next != NULL && curptr->next->data < NTA) {
                curptr = curptr->next;
            }
            new_node->next = curptr->next;
            curptr->next = new_node;
        }
        printf("please enter a number to add (enter -1 to stop): ");
    }
    return head;
}

这样拆分后，职责清晰，也不会出现重复输入和死循环的问题。

内容的提问来源于stack exchange，提问作者Adam