C语言 将节点插入到链接列表中的最后一个位置

Question

我需要在链表的最后一个位置插入一个节点。这就是我想出的：

#include<stdio.h>
#include<stdlib.h>
struct node { float data;
                struct node * next;
};
struct node* makenode(float item){
    struct node* p=(struct node*)malloc(sizeof (struct node));
    if(p) p->data = item;
    return p;
}
void init (struct node **p){
    *p=0;
}
int addlast(struct node **ptr, float item){
    struct node* p=makenode(item);
    if(!p) return 0;
    struct node* temp=*ptr;
    while(temp->next)temp = temp->next;
    p->next=0;
    temp->next=p;
    return 1;
}      
float delfirst(struct node **ptr){
    struct node* p =*ptr;
    *ptr=(*ptr)->next;
    float temp=p->data;
    free(p);
    return temp;
}
void main(){
    struct node *list,*list2;
    init (&list);
    int i;
    for(i=0;i<10;i++)addlast(&list,i);
    while(list)printf("%4.2ft",delfirst(&list));
    getchar();
}

但是当我编译代码时，它不断崩溃，错误出在addlast函数中。 但我找不到我错在哪里。谁能告诉我我在addlast功能中哪里出错了？

Answer 1

你的makenode函数有缺陷，它不会初始化结构中的所有元素。

您的addlast也有缺陷，因为当您添加第一个节点时，*ptr NULL，并且您在temp->next中取消引用此NULL指针，从而导致未定义的行为。

Answer 2

struct node* temp=*ptr;
while(temp->next)temp = temp->next;

当您调用 API 时addlast()传递一个NULL的指针，您可以使用此指针初始化 temp 并开始使用 temp。

访问/取消引用NULL指针将导致未定义的行为，从而导致崩溃。

Answer 3

在使用 temp->next 之前测试temp是否不为 NULL