亲历fread函数的错误用法及修改验证过程------台上1分钟， 台下1天功

        我曾经在http://blog.csdn.net/stpeace/article/details/42847697中说过：fwrite和fread函数针对的是字符（无边界）， 而不是字符串（以'\0'作为边界）

        今天呢， 某模块出了一个问题。 据说， 这个问题搞了较长时间。我准备晚上去打球的，毕竟感觉好久没有打了。 结果一个电话过来， 说有个紧急问题需要我协助定位， 我就乖乖返回了。经了解问题现象并查看源代码， 我立即发现了fread函数用法错误。后来修改并验证， 果然如此。

        fread函数错误用法的程序抽象为：

 
 

   
   

    
    

   
   
   
   

    
    

     
     #include <iostream>
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     using 
     
     namespace 
     
     std;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     typedef 
     
     struct _node
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     {
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     char name[
     
     100];
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     int score;
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     }Student;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     int main()
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     {
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	Student s;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&s, 
     
     0, 
     
     sizeof(s));
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     strncpy(s.name, 
     
     "Eric", 
     
     sizeof(s.name) - 
     
     1);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	s.score = 
     
     99;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	FILE *fp = fopen(
     
     "taoge", 
     
     "wb");
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fwrite(&s, 
     
     sizeof(s), 
     
     1, fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fwrite(&s, 
     
     sizeof(s), 
     
     1, fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fwrite(&s, 
     
     sizeof(s), 
     
     1, fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fclose(fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	Student t;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&t, 
     
     0, 
     
     sizeof(t));
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fp = fopen(
     
     "taoge", 
     
     "rb");
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fread(&t, 
     
     sizeof(t) - 
     
     1, 
     
     1, fp); 
     
     // danger, logical error
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.name << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.score << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&t, 
     
     0, 
     
     sizeof(t));
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fread(&t, 
     
     sizeof(t) - 
     
     1, 
     
     1, fp); 
     
     // danger, logical error
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.name << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.score << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&t, 
     
     0, 
     
     sizeof(t));
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fread(&t, 
     
     sizeof(t) - 
     
     1, 
     
     1, fp); 
     
     // danger, logical error
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.name << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.score << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fclose(fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     return 
     
     0;
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     }
    
    
   
   
 
 

       结果为：

Eric
99


25344


6488064

      我们看到， fread少读了一个字符， 结果会导致后面读取的时候出现整体偏移错误。 应该修改为：

 
 

   
   

    
    

   
   
   
   

    
    

     
     #include <iostream>
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     using 
     
     namespace 
     
     std;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     typedef 
     
     struct _node
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     {
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     char name[
     
     100];
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     int score;
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     }Student;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     int main()
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     {
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	Student s;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&s, 
     
     0, 
     
     sizeof(s));
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     strncpy(s.name, 
     
     "Eric", 
     
     sizeof(s.name) - 
     
     1);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	s.score = 
     
     99;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	FILE *fp = fopen(
     
     "taoge", 
     
     "wb");
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fwrite(&s, 
     
     sizeof(s), 
     
     1, fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fwrite(&s, 
     
     sizeof(s), 
     
     1, fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fwrite(&s, 
     
     sizeof(s), 
     
     1, fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fclose(fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	Student t;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&t, 
     
     0, 
     
     sizeof(t));
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fp = fopen(
     
     "taoge", 
     
     "rb");
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fread(&t, 
     
     sizeof(t), 
     
     1, fp); 
     
     // ok
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.name << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.score << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&t, 
     
     0, 
     
     sizeof(t));
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fread(&t, 
     
     sizeof(t), 
     
     1, fp); 
     
     // ok
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.name << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.score << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     memset(&t, 
     
     0, 
     
     sizeof(t));
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fread(&t, 
     
     sizeof(t), 
     
     1, fp); 
     
     // ok
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.name << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     cout << t.score << 
     
     endl;
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     	fclose(fp);
    
    
   
   

   
   

    
    

   
   
   
   

    
    
 
    
    
   
   

   
   

    
    

   
   
   
   

    
    
	
     
     return 
     
     0;
    
    
   
   

   
   

    
    

   
   
   
   

    
    

     
     }
    
    
   
   
 
 

     结果：

Eric
99
Eric
99
Eric
99

      正常。

      为什么写代码的人当时要用sizeof(...) - 1呢？ 

      根本原因： 没有搞清楚fread函数的用法。在这里， 我再次强调一遍: fread, read, recv, fwrite, write, send这类函数针对的是字符（无边界）， 而不是字符串（以'\0'作为边界）， 与'\0'没有半毛钱的关系。

      间接原因： 我们经常强调数组szTest必须以'\0'结尾， 可能是当时写该代码的人受了这个思想的影响， 以为任何缓冲区的最后一定是'\0'.

       我以前总结过类似问题， 所以今天很快帮助其他同事解决了此问题， 好开心， 尽管最后球没打成。 

       在其位， 谋其事， 体现自身价值。

       台上1分钟， 台下1天功啊， 以后继续坚持学习、实践、总结、分享。  好吧， 本文就总结到这里。