c 语言中fread()函数使用时应该注意的地方

nello

用c语言做了个通讯录，系统一运行时便将数据文件加载进内存，并用链表存储。退出系统时，自动将链表中的所有节点再存入文件。

可是现在，每次退出系统，文件里都会比链表多存储一条记录。
如：现在只有两条记录，退出后在启动时一查询，就会多一条乱记录（系统自己加的）。
加载文件的部分代码如下：
/*判断文件是否为空*/
  fseek(groupfp,0,2);/*指针指向group.dat文件尾部*/
  fseek(linkmanfp,0,2);/*指针指向linkman.dat文件尾部*/
  if(ftell(groupfp)>0){ /*文件非空*/
  rewind(groupfp);/*文件指针重新指向文件开始处*/
  /*下面要装载该文件到group链表中*/
  while(!feof(groupfp)){
      p=(GroupNote)malloc(sizeof(struct groupnote));
      fread(p,sizeof(struct groupnote),1,groupfp);
      p->next=NULL;/*刚读取进来的指针是上一次的，所以重新赋值为空*/
      gn->next=p;
  gn=p;/*尾插法*/
  /*注意：p->next上面已经赋值为空了，所以这里不需要再次赋值了*/
  group.length++; /*链表长度加1*/
  }/*while*/
  group.tail=gn;/*将尾指针指向group链表的结尾*/
把
fread(p,sizeof(struct groupnote),1,groupfp);
改成
if (fread(p,sizeof(struct groupnote),1,groupfp) != 1) {
  free(p);
  break;
}
这是因为有些库函数的写法，是在fread失败之后才设文件终结符的，这样在最后一个数据读完后还会再执行一次循环.