用于在链表中按升序添加和删除节点的C程序

Question

执行以下代码时出现分段错误。 仅当我在数据大于第一个节点项的节点中添加项时才会发生分段错误。

在此代码中，当我尝试添加20时，我遇到了分段错误，因为20大于第一个节点中的项目。 为什么会出现这种情况以及如何防止出现此错误？

代码：

#include <stdio.h>

struct node
{
    int data;
    struct node *link;
};

int main()
{
    struct node *p;
    p = NULL;

    add(&p,10);
    add(&p,9);
    add(&p,1);
    add(&p,20);

    display(p);
    printf("\nNo of elements in Linked List=%d",count(p));

    delete(&p,7);
    delete(&p,4);
    delete(&p,5);
    delete(&p,9);

    display(p);
    printf("\nNo of elements in Linked List=%d",count(p));

}
/* adds a node to an ascending order linked list */
add(struct node **q,int num)
{

    struct node *r,*temp=*q;
    r = malloc(sizeof(struct node));
    r->data = num;

    /* If list is empty or if new node is to be inserted before */
    if(*q==NULL || (((*q)->data) > num))
    {
        *q = r;
        (*q)->link = temp;
    }
    else
    {
            /* traverse the list to search the position to insert the new node */
        while(temp!=NULL)
        {

            if(temp->data <=num && (temp->link->data > num || temp->link == NULL))
            {
                r->link = temp->link;
                temp->link = r;
                return;
            }
            temp = temp->link; /* go to next node */
        }

    }       
}

display(struct node *q)
{
    printf("\n");

    while(q!=NULL)
    {
        printf("%d ",q->data);
        q = q->link;
    }
}

count(struct node *q)
{
    int c=0;
    while(q!=NULL)
    {
        q = q->link;
        c++;
    }
    return c;
}

delete(struct node **q,int num)
{
    struct node *old,*temp;
    temp = *q;

    while(temp!=NULL)
    {
        if(temp->data == num)
        {
            if(temp == *q)
            {
                *q =temp->link;
                free(temp);
                return;
            }
            else
            {
                old->link = temp->link;
                free(temp);
                return;

            }   
        }
        else
        {
            old = temp;
            temp = temp->link;
        }
    }
    printf("\nElement %d not found",num); 
}

Answer 1

在add方法中尝试更改行：

if(temp->data <=num && (temp->link->data > num || temp->link == NULL))

到

if(temp->data <=num && (temp->link == NULL || temp->link->data > num))

由于C是short cutting布尔表达式，因此将在另一个运算符之前计算NULL检查，并在为true时停止执行。

Answer 2

更改行

if(temp->data <=num && (temp->link->data > num || temp->link == NULL))

到

if(temp->data <=num && (temp->link == NULL || temp->link->data > num))

这应解决您的问题（它不会尝试从不存在的节点访问数据）