c - 从数组中读取数字然后将它们插入列表的程序不起作用

Question

我必须编写一个程序，从数组中读取 int 数字，然后如果它们大于限制数字，则将它们一个接一个地插入。我写的是:

#include <stdio.h>
#include <stdlib.h>

#define SIZE 10

struct number_list{

    int high_number;
    struct number_list *next;
    };

typedef struct number_list number_list;

void insert_in_list(int *array, int lim);

int main(void){

    int number_seq[SIZE]={1, 2, 3, 4, 5, 6, 7, 8, 9};
    int limit;
    scanf("%d\n", limit);
    insert_in_list(number_seq, limit);
    }

void insert_in_list(int *array, int lim){

    int i;
    number_list *start_node;
    for(i=0; i<SIZE; i++){

        if(array[i]>lim){

            start_node=malloc(sizeof(number_list));
            start_node->high_number=array[i];
            printf("%d\n", start_node->high_number);
            start_node->next=start_node;
            }

        }

    }

发生的事情是程序没有任何编译错误(除了 %d 期望参数类型为 *int 而不是 int 的警告，我也不明白，所以如果有人能帮助我也将非常友好)但是在运行时，在插入限制数字后，显示段错误(已创建核心转储)。

我想问题出在 insert_in_list 函数中，但我不明白它是什么。

Answer 1

这里有一些错误:

(a) insert_in_list 函数需要一个附加参数来告知要插入哪个列表。

(b) 为什么将 number_list 结构的成员命名为“high_number”。难道它不应该只包含一个数字并因此简单地称为“数字”吗？

(c) 您始终将next 指针设置为结构本身。如果要形成一个列表，它应该指向下一个节点。

(d) 在 main 中，您需要定义一个指向列表第一个节点的指针，只要列表为空，它就应该包含 null。这个“ anchor ”代表您的列表。

(e) 作为提示:将指向该 anchor 的指针传递给您的插入函数，因为一旦插入函数创建了第一个节点，它就必须将第一个节点的地址存入 anchor 。因此，您的插入函数应如下所示:

void insert_in_list (number_list** pAnchor, int number)

(f) 将输入数组的大小定义为 SIZE 并在 insert_in_list 函数中使用该符号并不是一个好主意。取而代之的是，将数组及其长度作为两个参数传递给函数，或者——就像我在上面的代码行中所做的那样——每次调用插入函数时只传递一个数字，并为每个要插入的数字调用它。