struct list* creat()
{
    struct list *r,*p,*l;
    int x;
    r=l=(list*)malloc(LEN);
    l->next=NULL;
    scanf("%d",&x);
    while(x!=0)
    {
        p=(list*)malloc(LEN);
        p->data=x;
        p->next=NULL;
        r->next=p;
        r=p;
        scanf("%d",&x);
    }
    return l;
}

在一个书上看到这个函数，感觉有问题，大家帮忙看看，是不是第一个节点里的数据（data）没有被赋值啊？

这个创建的是带头结点的链表，头结点不需要赋值

头结点特殊 只有指向下一结点的指针 没有数据
PS 尾结点也特殊 指向下一结点的指针指向NULL

头节点是必须空着吗？那查找和输出的时候是不是要注意绕开头结点？

相反 头结点是链表的唯一的入口 你对链表的所有操作几乎都是要从头结点开始的

头节点不是必须空着，很多情况下用它的数据域来存储链表的长度等信息，打印链表的时候不需要打印出头结点的数据域信息

其实头节点也可以赋值的。
这句r=l=(list*)malloc(LEN);
后面添加：
scanf("%d",&x);
r->data=x;
就可以了，
个人觉得没必要空着头节点，这样在进行链表查询等操作的时候就不用刻意避开头节点的操作了。