Question

代码用于创建商店购买，库存维护系统 我在使用fscanf（fp，............）函数将数据从txt文件输入到链接列表时出现问题;

下面的代码在注释部分写了问题但是很简单，当我在turbo c中运行代码并在运行时输入数据时，如果我没有读取旧内容，内容将正确进入文件。每次打开程序时，文件都会被覆盖。 当我用fscanf（）阅读内容时，垃圾值开始被添加到文件中我不知道为什么。可能是因为我使用指针而不是对象，但我不知道另一种方式。

我知道我的程序效率低下但仍然想知道代码有什么问题以及如何解决它我使用了很多变量，其中一些可能永远不会被使用，请原谅我。代码中的问题可以在create function中找到：

#include<alloc.h>
#include<stdio.h>
#include<conio.h>
#include<string.h>
#include<graphics.h>
#define size 20
struct shop{
char name[size];
int quantity;
int price;
struct shop *next;
}*start;
typedef struct shop sh;

char u_name[30];
char u_pass[30];

int i,user,password;
int k=0,l=0;
char add_more;
char c,choice,c1,more;




void create()
{ FILE *fc,*fp;

struct shop *ptr,*temp,*g,*l,*m,*t,*i,*d;
char ch,v[20];
int r,z,w,flag=0;
//the code ***************from here******************
fc=fopen("storedata.txt","r");
d=(sh*)malloc (sizeof(sh));
d->next=NULL  ;
i=d;
m=d;


while(fscanf(fc,"%s\t%d\t%d",m->name,m->quantity,m->price)!=EOF)
{
d=(sh*)malloc (sizeof(sh));
m->next=d;
m=m->next;

}
m->next=NULL;
fclose(fc);

t=i;

clrscr();
printf("NAME\t\t\tQUANTITY\t\t\t\tPRICE(RS)");

do
{
printf("\n%s ",t->name);
printf("\t\t\t%-20d",t->quantity);
printf("\t\t\t%-40d",t->price);
t=t->next;
}while(t!=NULL);
 getch();
 getch();
//*************till here********the smaller code part above is the code to read in the file which doesnt work correctly
start=i;}  // when i remove this line all the values entered in the file are correct but file is overridden every time i run it

感谢

Answer 1

就像scanf一样，fscanf需要位置/地址，它将存储format参数表示的信息。以下用法不正确：

fscanf(fc,"%s\t%d\t%d",m->name,m->quantity,m->price);

由于m->quantity和m->price都不是有效地址，因此您应使用&运算符：

fscanf(fc,"%s\t%d\t%d",m->name,&m->quantity,&m->price);

Answer 2

fscanf()的返回值处理已关闭。

它不返回指针，它返回int，see the documentation。整数是成功转换的次数;您应该将值与格式字符串中%说明符的nunber匹配。

它需要指向数据存储位置的指针;像%d这样的整数转换需要int的地址，即代码中的&m->quantity。

在C.中也是don't cast the return value of malloc()。实际上，重写

d=(sh*)malloc (sizeof(sh));

为：

d = malloc(sizeof *d);

提高清晰度，减少重复次数和简洁性。

从txt文件读取数据到链表

2 个答案: