求教C语言高手：版我看一下这段代码

[codetown]

菜鸟求教：以下这段代码不知错在哪里？请高手帮忙

#include<stdio.h>
#include<malloc.h>
typedef struct s1
{
    int data;
    struct s1* next;
} NODE;
push(NODE* head,int value)
{
    NODE* tmp=head;
    head=(NODE*)malloc(sizeof(NODE));
    head->data=value;
    printf("%d",head->data);
    head->next=tmp;
}
print(NODE* head)
{
    NODE* itNode=head;
    while(itNode)
    {
        printf("%d\n",itNode->data);
        itNode=itNode->next;
    }
}
int main(void)
{
    int i;
    NODE* mylist=0;
    for(i=1;i<10;++i)
        push(mylist,i);
    print(mylist);
    return 0;
}

改一下

[relaxisland]

push(NODE* head,int value)
{
NODE* tmp=head;
head=(NODE*)malloc(sizeof(NODE));
head->data=value;
printf("%d",head->data);
head->next=tmp;
}的函数必须采用二级指针，采用把 head传回来

push(NODE** head,int value)
{
NODE* tmp=*head;
*head=(NODE*)malloc(sizeof(NODE));
*head->data=value;
printf("%d",*head->data);
*head->next=tmp;
}

[leo_dengzx]

楼上正确。也可以提前分配好一个节点，每次push自动分配下一个，这样就可以不用二级指针。

[zhongjilei]

[Quote=引用 1 楼 relaxisland 的回复:]
push(NODE* head,int value)
{
NODE* tmp=head;
head=(NODE*)malloc(sizeof(NODE));
head->data=value;
printf("%d",head->data);
head->next=tmp;
}的函数必须采用二级指针，采用把 head传回来

push(NODE** head,……
[/Quote]

顶，另外有个问题，函数要有返回类型，虽然有的编译器会
默认给你填上返回int，有时候还是会出问题的
所以一定要将返回类型写上，没有的话写void
要养成好习惯

[AnYidan]

[Quote=引用 1 楼 relaxisland 的回复:]
push(NODE* head,int value)
{
NODE* tmp=head;
head=(NODE*)malloc(sizeof(NODE));
head->data=value;
printf("%d",head->data);
head->next=tmp;
}的函数必须采用二级指针，采用把 head传回来

push(NODE** head,……
[/Quote]

原因，lz 可以读一下林锐博士的 高质量c/c++编程