我有一个档案results.csv
,我需要阅读此档案的第一行并在output.txt
上打印出来。不知怎的,它在所有内容之后打印随机字符,我无法弄清楚出了什么问题。
命令:a.c results.csv
第一行:
date,home_team,away_team,home_score,away_score,tournament,city,country,neutral
output.txt:date,home_team,away_team,home_score,away_score,tournament,city,country,neutral,(!£,(!£,(!£,(!£,(!£,@,£,(!£,(!£
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <string.h>
typedef struct
{
char *line1;
char *line1a;
char *line1b;
char *team1;
char *team2;
char *reason;
char *city;
char *country;
char *neutral_field;
}data;
void open_input(char *argv[], FILE **input)
{
if((*input=fopen(argv[1], "r")) == NULL)
{
printf("%s not found\n", argv[1]);
exit(1);
}
}
void open_output(char *string, FILE **output)
{
if((*output=fopen(string, "w")) == NULL)
{
printf("%s not found\n", string);
exit(1);
}
}
void alloc_data(data *d, int size)
{
d->line1 = (char*)malloc(4*sizeof(char));
d->team1 = (char*)malloc(9*sizeof(char));
d->team2 = (char*)malloc(9*sizeof(char));
d->line1a = (char*)malloc(10*sizeof(char));
d->line1b = (char*)malloc(10*sizeof(char));
d->reason = (char*)malloc(10*sizeof(char));
d->city = (char*)malloc(4*sizeof(char));
d->country = (char*)malloc(7*sizeof(char));
d->neutral_field = (char*)malloc(7*sizeof(char));
}
void store(data *d, FILE *input, FILE **output)
{
fscanf(input, "%s,%s,%s,%s,%s,%s,%s,%s,%s", d[0].line1, d[0].team1, d[0].team2, d[0].line1a, d[0].line1b, d[0].reason, d[0].city, d[0].country, d[0].neutral_field );
fprintf(*output, "%s,%s,%s,%s,%s,%s,%s,%s,%s\n", d[0].line1, d[0].team1, d[0].team2, d[0].line1a, d[0].line1b, d[0].reason, d[0].city, d[0].country, d[0].neutral_field );
}
int main(int argc, char *argv[])
{
FILE *input;
FILE *output;
char *string = "output.txt";
int size = 1000;
open_input(argv, &input);
open_output(string, &output);
data *d;
d = (data*)malloc(size*sizeof(data));
alloc_data(d, size);
store(d, input, &output);
free(d);
return 0;
}
答案 0 :(得分:1)
您的缓冲区不足以容纳终止NUL字节。 scanf
存储NUL字节(超出缓冲区),但真正拥有该字节的对象可能会覆盖它,所以当printf
查找NUL时它直到内存后才会找到它
缓冲区溢出是一个比你看到的更大的问题,谁知道你没有留出空间的那些NUL字节是什么对象粉碎?当您读取具有略微不同的标题拼写的数据文件时会发生什么?突然之间,您的硬编码分配大小将比现在更加错误。
答案 1 :(得分:1)
fscanf(input, "%s,%s,%s,%s,%s,%s,%s,%s,%s", d[0].line1, d[0].team1,...
上面的代码试图将整行读入d[0].line1
,导致缓冲区溢出。 team1
,其余的将包含未初始化的数据。
您必须按如下方式更改fscanf
:
fscanf(input, "%3[^ ,\n\t],%9[^ ,\n\t],...
其中3是4 - 1,而4是d[0].line1
或者,您可以使用strtok
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void store(FILE *input, FILE *output)
{
char buf[500];
while(fgets(buf, sizeof(buf), input))
{
//strip end-of-line from `buf`
if(strlen(buf))
if(buf[strlen(buf) - 1] == '\n')
buf[strlen(buf) - 1] = 0;
//tokenize with strtok
char *token = strtok(buf, ",");
while(token)
{
fprintf(output, "%s", token);
token = strtok(NULL, ",");
}
fprintf(output, "\n");
}
}
int main(int argc, char *argv[])
{
FILE *input = fopen("input.txt", "r");
FILE *output = fopen("output.txt", "w");
store(input, output);
return 0;
}
使用上面的代码,您不需要额外的结构。
<小时/> 如果您确实使用数据结构,则必须更加小心。您似乎正在尝试创建一个1000data
的数组,但以下只创建一个超大指针,而不是data
的数组
int size = 1000;
data *d;
d = (data*)malloc(size*sizeof(data));
alloc_data(d, size);
此外,对于每个malloc
,应该有相应的free
。