#include <stdio.h>
#include <stdlib.h>
typedef struct LNode
{
    int data;
    struct LNode *next;
}LNode,*Linklist;
void initial(Linklist L)
{
    Linklist p;
    p=(Linklist)malloc(sizeof(LNode));
    if(!p) exit(0);
    L=p;
    L->next=NULL;
}
void creatlist(Linklist L)
{
    Linklist p,q;
    int m;
    q=L;
    printf("请输入若干个正整数，输入非正整数时结束:\n");
    while(1)
    {
        scanf("%d",&m);
        if(m>0)
        {
            p=(Linklist)malloc(sizeof(LNode));
            p->data=m;
            q->next=p;
            q=p;
        }
        else break;
    }
    p->next=NULL;
}
void displaylist(Linklist L)
{
    Linklist p=L;
    for(p=L->next;p!=NULL;p=p->next)
        printf("%d  ",p->data);
    printf("\n");
}
void destory(Linklist L)
{
    Linklist temp;
    temp=L;
    while(temp!=NULL)
    {
        free(temp);
        temp=temp->next;
    }
}
void main()
{
    LNode A;
    initial(&A);
    creatlist(&A);
    printf("输出单向链表A的元素:\n");
    displaylist(&A);
    destory(&A);
}
帮忙看看我的destory()函数问题在哪里？

    while(temp!=NULL)
    {
        free(temp);
        if (temp->next != NULL)
            temp=temp->next;
    }

还是不对啊！运行后还是弹出警告。

代码写的很多问题，指针使用不当，改好了

#include <stdio.h>
#include <stdlib.h>
typedef struct LNode
{
    int data;
    struct LNode *next;
}LNode,*Linklist;
void initial(Linklist L)
{
    L->data = 0;
    if(!L) exit(0);
    L->next=NULL;
}
void creatlist(Linklist L)
{
    Linklist p,q;
    int m;
    q=L;
    p=L;
    printf("请输入若干个正整数，输入非正整数时结束:\n");
    while(1)
    {
        scanf("%d",&m);
        if(m>0)
        {
            p=(Linklist)malloc(sizeof(LNode));
            p->data=m;
            q->next=p;
            q=p;
        }
        else break;
    }
    p->next=NULL;
}
void displaylist(Linklist L)
{
    Linklist p=L;
    for(p=L->next;p!=NULL;p=p->next)
        printf("%d  ",p->data);
    printf("\n");
}
void destory(Linklist L)
{
    Linklist temp;
    while(L!=NULL)
    {
        temp=L->next;
        free(L);
        if (temp != NULL)
            L=temp;
        else
            break;
    }
}
void main()
{
    Linklist A;
    A=(Linklist)malloc(sizeof(LNode));
    initial(A);
    creatlist(A);
    printf("输出单向链表A的元素:\n");
    displaylist(A);
    destory(A);
}

[ 本帖最后由 秀痘魔导士 于 2010-3-26 16:33 编辑 ]

再麻烦你解释下为什么不能直接删除头结点，而必须先让temp=L->next;再 free(L);直接free(L)为什么不对呢？在网上搜索了一下原因，感觉还是不太明白。谢了！这个问题解决了就结贴！

直接删除头结点，头结点就不存在了，那么他的next当然是一个错误的值，所以必须先备份一份next的指针，这样在删除之后才能定位到next