一段简单的读文件c程序. 有些疑问.

fenglue

cfg文件只有一行ss.
我把问题写在注释中.

1. 
   
2. #include <stdio.h>;
   
3. 
   
4. int main(){
   
5.         FILE *dat;
   
6.         dat=fopen("./cfg","rt");
   
7.         fseek(dat,0,SEEK_END);
   
8.         long int k=ftell(dat);                            //这里是3
   
9. 
   
10.         char*  input=(char*)malloc(k*sizeof(char));   //分配3byte, 起始地址是xxxx80
    
11.         rewind(dat);
    
12.         *input=fgetc(dat);
    
13.         while(!feof(dat)){
    
14.                 *(++input)=fgetc(dat);
    
15.         }
    
16. //在这里发现input总会跳到第4个地址0xxxx83,可这个                                                                            //地址不是他的啊.奇怪的是结果倒是对的
    
17.         fclose(dat);
    
18.         printf("%s",input-k);                  //一直觉得应该是input-k+1
    
19.         return 1;
    
20. }
    
21. 
    

复制代码

看了半天,不明所以.是不是有什么概念错误?   

1. [/code][code]

复制代码

gadfly

1. 
   
2. *(++input)=fgetc(dat);
   

复制代码

改成

1. 
   
2. *(input++)=fgetc(dat);
   

复制代码

就好了。

个人觉得，为了代码清晰，用以下的方式好：

1. 
   
2. *input=fgetc(dat);
   
3. input++;
   

复制代码

fenglue

这样改就不对了.因为如此就会无条件地input++而很可能会超出分配给她的内存了.
input++;
*input=fgetc(dat);
就是和我的一样.
我的问题是:
对于3个byte 文件 cfg中仅有一行的 ss来说, 为什么我在读完了3byte后,还会再次进入循环,读一个?字符.
晕啊.

gadfly

？？
怎么会一样，你以前的是地址先往前移动，再赋值，当然会浪费一个了。
你在仔细看看区别。
另外你的程序也没有初始化这块分配的内存，为了放置结束符，应该多分配一个字符空间。读完之后，需要结束。

1. 
   
2. int main(){
   
3.         FILE *dat;
   
4.         dat=fopen("./cfg","rt");
   
5.         fseek(dat,0,SEEK_END);
   
6.         long int k=ftell(dat);                            //这里是3
   
7. 
   
8.         char*  input=(char*)malloc((k+1)*sizeof(char));   //分配3byte, 起始地址是xxxx80
   
9.         rewind(dat);
   
10.         *input=fgetc(dat);
    
11.         while(!feof(dat)){
    
12.                 *input=fgetc(dat);
    
13.                 input++;
    
14. 
    
15.         }
    
16.              *input = '\0';
    
17.         fclose(dat);
    
18.         printf("%s",input-k);                  //一直觉得应该是input-k+1
    
19.         return 1;
    
20. }
    

复制代码

自己试试看吧

fenglue

先谢谢你的回答.我还是初学者,所以想把一些细节搞搞清楚.
在我的例中.我先*input=fgetc(dat),然后再进入循环,所以首先得input++,再*input=fgetc(dat).
而你的例中,看起来好象第一个字符被第二个字符覆盖了.

不知道我说的对不对.
另外一个问题关于"\0"这个结束符,在存成文件的时候,该不该存进去啊?

JohnBull

你的程序没有在字符串后面补\0；
malloc内存的时候没有为\0多申请一个char；
不应该在feof监测之前就fgetc，文件可能是0字节长；
取得malloc对分配的指针不应该改变；
堆使用完后没有free；
程序有点乱（可能咱俩思维习惯不同），我宁愿这样写：

1. 
   
2. #include <stdio.h>;
   
3. 
   
4. int main(){
   
5.         FILE *dat;
   
6.         long int k,i=0;
   
7.         char tmp,*input;
   
8. 
   
9.         dat=fopen("./cfg","rt");
   
10.         fseek(dat,0,SEEK_END);
    
11.         k=ftell(dat);
    
12.         input=(char*)malloc((k+1)*sizeof(char));
    
13.         rewind(dat);
    
14.         while(!feof(dat)){
    
15.                 tmp=fgetc(dat);
    
16.                 if (tmp==EOF) tmp='\0';
    
17.                 input[i++]=tmp;
    
18.         }
    
19.         fclose(dat);
    
20.         printf("%s\n",input);
    
21.         free(input);
    
22.         return 1;
    
23. }
    

复制代码

gadfly

哦，没注意while前面还有个fgetc，去掉，我那程序就正确了。

为了malloc空间的释放和指针的完整，确实应该采用JohnBull的风格。

xjleijie

可按JohnBull 的程序编译通过
但执行时总是提示 “Segmentation fault"
能说明是什么原因么？

JohnBull

原帖由 "xjleijie" 发表：
可按JohnBull 的程序编译通过
但执行时总是提示 “Segmentation fault"
能说明是什么原因么？

   

因为我为了不对他的程序做太大的改动所以没有检查fopen是否成功。你肯定没建cfg。

fenglue

感谢johnbull大牛的提醒. 你的回贴向来很精彩,一锤定音.
如此看来在文件的末尾还有一个EOF字符. 而feof()只有在这个字符之后,才会返回true.