#include<stdio.h>
#include<malloc.h>
typedef struct node
{
    int data;
    int site;
    struct node *link;
}NODE;

NODE *creat()
{
    NODE *p,*q,*head,*tail;                                                                                                                                                                                                                                                                                             
    int num,i=0;
    head=tail=NULL;
    printf("int put num end with -1:\n");
    scanf("%d",&num);
    while(num!=-1)
    {
        i++;
        p=(NODE *)malloc(sizeof(NODE));
        if(p==NULL)
        {
            printf("fail");
            return NULL;
        }
        p->data=num;
        p->site=i;
        p->link=0;
        if(head==NULL)
        {
            head=p;
        }
        else
        {
            tail->link=p;
        }
        tail=p;
        scanf("%d",&num);
    }
    return head;
}

int countlist(NODE *head)
{
    int i=0;
    while (head)
    {
        i++;
        head=head->link;
    }
    return i;
}

NODE *insertlist(NODE *head,int num,int i)
{
    NODE *p,*q;
    int j;
    p=q=NULL;
    p=head;
    for(j=0;;j++)
    {
        if(p->site!=i)
        {
            p=p->link;
            continue;
        }
        else
        {
            q->data=num;
            q->site=p->site+1;
            q->link=p->link;
            p->link=q;
            return head;
        }
    }
}

void printlist(NODE *head)
{
    while (head)
    {
        printf("%d",head->data);
        head=head->link;
    }
    return;
}

main()
{
    NODE *a;
    a=creat();
    insertlist(a,88,2);
    printlist(a);
}

1、首先我吐槽一下，既然是提问题，你先把你的想法什么的说一下，然后加上注释，没注释贴这样一串代码除了我这种半夜睡不着闲的蛋疼的谁有耐心看完啊。
2、既然你写了countlist干嘛还要在结构体里多加个site（话说你这命名真心挺醉的）

以下是问题：
3、你p->link=0是什么鬼，link指向的是node类型吧，而tail->link=p，第一次tail都还是null哪来的link嘛。其他的函数应该都没错，但create问题挺多的。
4、如果我没猜错你应该是要弄个正序输入创建的单链表。如果是的话建议使用一个NewNode指针变量来生成新结点，将第一个节点同时使用Head和Tail指向，接着使用Tail将一个个NewNode连接起来就行了