这个函数有错么？？？？怎么改？？？
int Locate(NODE *head, NODE *data){
    NODE *p;
    int n;
    for (p = head, n = 0; p->link != NULL;){
        if (strcmp(p->name,data->name))return n;
        else p = p->link; n++;
    }
}
目的：查找链表中元素，并输出所在位置

如果没错的话那是不是调用的语句错了？？？下面是完整代码

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
typedef struct node{
    char name[20];
    struct node *link;
}NODE;
void creat(NODE *head, int n){
    NODE *p;
    for (; n > 0; n--){
        p = (NODE *)malloc(sizeof(NODE));
        gets(p->name);
        p->link = NULL;
        head->link = p;
        head = p;
    }
}
void output(NODE *head){
    NODE *p;
    p = head->link;
    while (p != NULL){
        puts(p->name);
        p = p->link;
    }
}

int Locate(NODE *head, NODE *data){
    NODE *p;
    int n;
    for (p = head, n = 0; p->link != NULL;){
        if (strcmp(p->name,data->name))return n;
        else p = p->link; n++;
    }
}

void main(){

    NODE *head;
    int n;
    head = (NODE *)malloc(sizeof(NODE));
    head->link = NULL;

    printf("结点个数为:");
    scanf("%d", &n);
    getchar();
    creat(head, n);
    printf("\n\n");
    printf("输出;\n");
    output(head);

    NODE *p;
    p = (NODE *)malloc(sizeof(NODE));
    printf("请输入要查找的数据：");
    gets(p->name);
    printf("所在位置为：%d", Locate(head, p));
}

用数组做不是挺好的

我们学校刚讲完链表，让用链表练习，，，，求解

strcmp(p->name,data->name)，这一句，如果两个字符串相等，它返回零，所以应该在它前面加上取反 ! 。

加了，程序还是有问题呀

嗯，往下一看，问题还真不少。给个建议你吧，把下面功能单独写成子函数：1、创建表头；2、插入节点；3删除节点（当然，本题来说不需要）；4、查找节点。你的creat函数将创建表头和插入节点写到一起了，结果却把表头给丢了……

int Locate(NODE *head, NODE *data)
这个函数你有返回值么？
在函数尾加上
return n;
你再试试

另外提醒一下
记得
free(head);
free(p);

locate函数后面加个return 0；，在把main函数中p和head指针一起定义放在main函数的第一行，然后再运行程序试试