fprintf 和 fscanf 存读问题求高人指点

vfcv

这个代码一开始只要求输入员工，工号，名字，工资，只存储工号，工资。
没要求取出在进行操作的。后来一个朋友说加上一个在取出后删除的操作

结果就在用fscanf取出数据后发现悲催了。

求高人指点一下

下面是代码略长有问题的就最后一段

1. #define _CRT_SECURE_NO_WARNINGS
   
2. #include<stdio.h>
   
3. #include<stdlib.h>
   
4. 
   
5. #define ZJ sizeof(data)
   
6. 
   
7. struct data
   
8. {
   
9.         int a;
   
10.         char m[20];
    
11.         int g;
    
12.         struct data *e;
    
13. };
    
14. 
    
15. void main()
    
16. {
    
17.         struct data *lianbiao();
    
18.         void baochun(struct data *p);
    
19.         void puchu();
    
20. 
    
21.         struct data *lb;
    
22. 
    
23.         lb = lianbiao();
    
24.         baochun(lb);
    
25.         puchu();
    
26. 
    
27. }
    
28. 
    
29. struct data *lianbiao()      //建立链表
    
30. {
    
31.         struct data *p,*x,*y;
    
32.        
    
33.         printf("输入工人数据以工号‘0’结束\n");
    
34. 
    
35.         p = x = y =(struct data *)malloc(ZJ);
    
36.        
    
37.         while(y->a)
    
38.         {
    
39.                 printf("输入工号 ");
    
40.                 scanf("%d",&y->a);
    
41.                 if(y->a != 0)
    
42.                 {
    
43.                         printf("输入名字 ");
    
44.                         scanf("%s",&y->m);
    
45.                         printf("输入工资 ");
    
46.                         scanf("%d",&y->g);
    
47. 
    
48.                         x = y;
    
49.                         y = (struct data *)malloc(ZJ);
    
50.                         x -> e = y;
    
51.                 }
    
52.                 else x -> e = NULL;
    
53.         }
    
54.         free(y);
    
55.         return p;
    
56. }  //
    
57. 
    
58. void baochun(struct data *p)   //将建立的链表保存到文件 （只保存工号工资）
    
59. {
    
60.         char a[20];
    
61.         FILE *fp;
    
62. 
    
63.         printf("输入创建文件的目录和名字\n");
    
64.         scanf("%s",a);
    
65. 
    
66.         if((fp = fopen(a,"wb")) == NULL)
    
67.         {
    
68.                 printf("文件建立失败\n");
    
69.                 exit(0);
    
70.         }
    
71. 
    
72.         printf("保存工号和工资\n");
    
73. 
    
74.         while(p)
    
75.         {
    
76.                 fprintf(fp,"%d,%d",p->a,p->g);
    
77.                 printf("%d,%d\n",p->a,p->g);
    
78.                 p = p -> e;
    
79.         }
    
80.         free(p);
    
81.         fclose(fp);
    
82. }
    
83. 
    
84. void puchu()  //取出工号工资进行删除操作
    
85. {
    
86.         char a[20];
    
87.         FILE *fp;
    
88.         struct data *p,*y,*x;
    
89. 
    
90.         printf("输入读取文件的目录和名字\n");
    
91.         scanf("%s",a);
    
92. 
    
93.         if((fp = fopen(a,"rb+")) == NULL)
    
94.         {
    
95.                 printf("打开文件失败\n");
    
96.                 exit(0);
    
97.         }
    
98. 
    
99.         p = y = x = (struct data *)malloc(ZJ);
    
100. 
     
101.         while(feof(fp)==0)                         /*取出的数据是乱码，且可能无限循环*/
     
102.         {
     
103.                 fscanf(fp,"%d%d",&y->a,&y->g);
     
104.                 printf("%d,%d\n",y->a,y->g);
     
105.                 x = y;
     
106.                 y = (struct data *)malloc(ZJ);
     
107.                 x -> e = y;
     
108.         }
     
109.         x -> e = NULL;
     
110.         free(y);
     
111. }

复制代码

我是师兄

                fprintf(fp,"%d,%d",p->a,p->g);
                printf("%d,%d\n",p->a,p->g);
楼主 你的代码
假设 有数据
12 34
56 78
你这样写入文件 就会是
12,3456,78.......

printf其实是 fprintf(stdout....)
你的printf 里面记得加\n
fprintf就得加 \n

                fscanf(fp,"%d%d",&y->a,&y->g);
                printf("%d,%d\n",y->a,y->g);
你文件中的内容 两个数据中间用逗号分开了
那么 读取的时候 2个数据中间就得加逗号
fscanf(fp, "%d,%d"......)

熊文杰

fscanf遇见空格就返回了。这里可以考虑用fread fseek 的方式。保存文件的时候 加一些特殊的间隔符号方便读取时候识别

熊文杰

另外，我还发现一个问题，就是你这个代码，开始输入0的时候就会崩溃，保护判断没有做

熊文杰

等下帮你整一下把。就当是复习，好久没搞c语言了。

vfcv

我是师兄 发表于 2013-4-23 14:30

                               
登录/注册后可看大图

fprintf(fp,"%d,%d",p->a,p->g);
                printf("%d,%d\n",p->a,p->g);
楼主 你 ...

谢谢这个我还真悲催了一个下午
你指点一下果然ok
还有楼上的不提醒我，我都忘记了写代码要容错；

熊文杰

大哥，你的代码逻辑有点惨不忍睹，链表函数里面，逻辑有点混乱。那里我帮你小改一下，没有大改，另外，我把读写改成了fwrite和fwrite了。代码如下

1. #define _CRT_SECURE_NO_WARNINGS
   
2. #include<stdio.h>
   
3. #include<stdlib.h>
   
4. 
   
5. #define ZJ sizeof(data)
   
6. 
   
7. struct data *lianbiao();
   
8. void baochun(struct data *p);
   
9. void puchu();
   
10. 
    
11. 
    
12. struct data
    
13. {
    
14.         int a;
    
15.         char m[20];
    
16.         int g;
    
17.         struct data *e;
    
18. };
    
19. 
    
20. 
    
21. 
    
22. struct data *lianbiao()      //建立链表
    
23. {
    
24.         struct data *p,*x,*y;
    
25. 
    
26.         printf("输入工人数据以工号‘0’结束\n");
    
27. 
    
28.         p = x = y =(struct data *)malloc(ZJ);
    
29. 
    
30.         while(y->a)
    
31.         {
    
32.                 printf("输入工号 ");
    
33.                 scanf("%d",&y->a);
    
34.                 if(y->a != 0)
    
35.                 {
    
36.                         printf("输入名字 ");
    
37.                         scanf("%s",&y->m);
    
38.                         printf("输入工资 ");
    
39.                         scanf("%d",&y->g);
    
40. 
    
41.                         x = y;
    
42.                         y = (struct data *)malloc(ZJ);
    
43.                         x -> e = y;
    
44.                 }
    
45.                 else x -> e = NULL;
    
46.         }
    
47.         if (y == p)
    
48.         {
    
49.                 p = NULL;
    
50.                 free(y);
    
51.                 y = NULL;
    
52.         }
    
53.        
    
54.        
    
55.         return p;
    
56. }  //
    
57. 
    
58. void baochun(struct data *p)   //将建立的链表保存到文件 （只保存工号工资）
    
59. {
    
60.         if (!p)
    
61.         {
    
62.                 return;
    
63.         }
    
64.        
    
65.         char a[20];
    
66.         FILE *fp = NULL;
    
67. 
    
68.         printf("输入创建文件的目录和名字\n");
    
69.         scanf("%s",a);
    
70. 
    
71.         if((fp = fopen(a,"wb")) == NULL)
    
72.         {
    
73.                 printf("文件建立失败\n");
    
74.                 exit(0);
    
75.         }
    
76. 
    
77.         printf("保存工号和工资\n");
    
78. 
    
79.         if (!p->a)
    
80.         {
    
81.                 free(p);
    
82.                 fclose(fp);
    
83.         }
    
84.        
    
85.         while(p)
    
86.         {
    
87.                 //fprintf(fp,"%d,%d",p->a,p->g);
    
88.                 int nCount =  fwrite(&(p->a),sizeof(int),1,fp);
    
89.                 nCount = fwrite(&(p->g),sizeof(int),1,fp);
    
90. 
    
91.                
    
92.                 printf("%d,%d\n",p->a,p->g);
    
93.                 p = p -> e;
    
94.         }
    
95.         free(p);
    
96.         fclose(fp);
    
97. }
    
98. 
    
99. void puchu()  //取出工号工资进行删除操作
    
100. {
     
101.         char a[20];
     
102.         FILE *fp;
     
103.         struct data *p,*y,*x;
     
104. 
     
105.         printf("输入读取文件的目录和名字\n");
     
106.         scanf("%s",a);
     
107. 
     
108.         if((fp = fopen(a,"rb+")) == NULL)
     
109.         {
     
110.                 printf("打开文件失败\n");
     
111.                 exit(0);
     
112.         }
     
113. 
     
114.         p = y = x = (struct data *)malloc(ZJ);
     
115. 
     
116.         int nCount = fread(&(y->a),sizeof(int),1,fp);
     
117.         nCount = fread(&(y->g),sizeof(int),1,fp);
     
118. 
     
119. 
     
120.         while(!feof(fp))                         /*取出的数据是乱码，且可能无限循环*/
     
121.         {
     
122.                 //读完最后一个字符后，fp->flag仍然没有被置为_IOEOF，因而feof()仍然没有探测到文件结尾。直到再次调用fgetc()执行读操作，feof()才能探测到文件结尾。这样就多输出了一个
     
123.                 printf("%d,%d\n",y->a,y->g);
     
124.                 int nCount = fread(&(y->a),sizeof(int),1,fp);
     
125.                 nCount = fread(&(y->g),sizeof(int),1,fp);
     
126. 
     
127.                
     
128.                 x = y;
     
129.                 y = (struct data *)malloc(ZJ);
     
130.                 x -> e = y;
     
131. 
     
132.         }
     
133.         x -> e = NULL;
     
134.         free(y);
     
135. }
     
136. 
     
137. int main(int argc, _TCHAR* argv[])
     
138. {
     
139.         struct data *lb;
     
140. 
     
141.         lb = lianbiao();
     
142.         baochun(lb);
     
143.         puchu();
     
144. 
     
145.         while (1)
     
146.         {
     
147.                 int d = 0;
     
148.                 scanf("%d",&d);
     
149. 
     
150.                 if (1 == d)
     
151.                 {
     
152.                         break;
     
153.                 }
     
154. 
     
155.         }
     
156.         return 0;
     
157. }

复制代码

建议把头指针和其他的指针区分开来。单独写createHead 和addNode函数

vfcv

熊文杰 发表于 2013-4-23 16:07

                               
登录/注册后可看大图

大哥，你的代码逻辑有点惨不忍睹，链表函数里面，逻辑有点混乱。那里我帮你小改一下，没有大改，另外，我把 ...

学习了果然高手，求教，求升级。求入门进阶教程

熊文杰

vfcv 发表于 2013-4-23 16:44

                               
登录/注册后可看大图

学习了果然高手，求教，求升级。求入门进阶教程

fscanf，和fprintf不建议用，效率不高，写的时候将2进制转换成ascii，读的时候又要讲ascii转换成2进制，效率很低的，常规做法是用fread和fwrite fseek配合进行读写。代码如果需要优化的话，单独写链表文件，不要写一起。不知道小甲鱼的零基础c语言里面是怎么说的，没看他的c语言教程。学习c语言的话，写个栈的例子，链表的例子感觉就差不多了。如果觉得还不够 可以写一些数据结构，比如复杂点的红黑树。这些东西如果能用c熟练的编写 就没什么问题了。c语言的书 我没看几本 一本谭浩强的入门c程序设计，一本c陷阱与缺陷 一本数据结构。然后再就是c++ primer 第三版。

252013680

学习了{:1_1:}{:1_1:}

我是师兄

熊文杰 发表于 2013-4-23 19:02

                               
登录/注册后可看大图

fscanf，和fprintf不建议用，效率不高，写的时候将2进制转换成ascii，读的时候又要讲ascii转换成2进制，效 ...

这个看具体情况了
效率未必是第一考虑

这里 如果需要写出来的文件
能直接打开查看内容 那么
用fscanf fprintf 是一个不错的选择