下面是一个建立链表的程序，老师写的，我觉得有点小问题，大家帮我看看。

# include "stdio.h"
struct student{
  int num;
  float score;
  struct student * next;
};
struct student *creat();
main()
{
  struct student *head;
  head=creat();
}

struct student *creat()
{
  struct student *head,*tail,*p;
  float score;
  int num,size=sizeof(struct student);
  head=tail=NULL;
  scanf("%d%f",&num,&score);
  while(num){
    p=(struct student *)malloc (size);
    p->num=num;p->score=score;
    p->next=NULL;
    if(head==NULL) head=p;
    else tail->next=p;
    tail=p;
    scanf("%d%f",&num,&score);
  }
  return head;
}

这段程序建立了单向链表，但好像链表的尾指针不是NULL，我觉得应该在return head的前面加上一句:tail->next=NULL
高手帮我看看是不是这样，谢谢！

[[it] 本帖最后由 ronaldfree 于 2008-8-14 15:34 编辑 [/it]]

我看错了，老师是对的。我没看到p->next=NULL这句

如果从效率上看，把p->next=NULL这句删了，然后在return head前面加上tail->next=NULL可能效率更高，请问高手是不是这样？

我就觉得在main里还应该加上这个

void *p;
p=create;
h=p();

struct student *()

你们觉得这个链表的程序没有问题吗？

如果这位老师写这段代码的目的是要学生找出错误并改正的话那还差不多，如果是作为正确的程序来讲解的话，这样的老师可以去另谋生路了，否则会误人子弟！

有点问题：struct student *()这行应该是struct student *creat()，抄漏了creat。现在没有问题了，谁说有问题我跟谁急

正确性没什么问题
效率另议。

？？？
看不懂！