我对C&基础知识的编程很陌生。 C ++。我一直在尝试在我用于图像分析的.dat文件上运行代码。
基本上这个.dat文件有x& y簇的位置及其半径。使用此代码,我试图将我的图像(在缩放像素后尺寸为200mm x 300mm)划分为5mm x 5mm的网格,并找到半径= 1的点数和半径= 100的点数。
这是我的代码:
#include<stdio.h>
int main()
{
int i=0,g,h,ng[350][250],ns[350][250],ntot[350][250];
float a[6001],b[6001],c[6001];
FILE *fres1, *fres2;
for(g=5;g<=350;g=g+5)
{
for(h=5;h<=250;h=h+5)
{
ng[g][h]=0;
ns[g][h]=0;
ntot[g][h]=0;
}
}
fres1 = fopen("trial_final_1.dat","r+");
fres2 = fopen("result_boxes.dat","w");
if(fres1 == NULL)
{
printf("Error!! Cannot open file \n" );
return 1;
}
else
{
printf("File opened successfully, kindly check your folder for result_boxes file :) \n");
fprintf(fres2,"x(mm) y(mm) no. of glass beads no. of stain-less steel balls total no. of balls \n");
while(!feof(fres1))
{
fscanf(fres1,"%f %f %f",&a[i],&b[i],&c[i]);
g = ((a[i]/5) + 1)*5;
h = ((b[i]/5) + 1)*5;
if(c[i] == 100)
{
ng[g][h] = ng[g][h]+1;
}
else
{
ns[g][h] = ns[g][h] + 1;
}
ntot[g][h] = ntot[g][h] + 1;
++i;
}
g = 5;
h = 5;
for(g=5;g<=350;g=g+5)
{
for(h=5;h<=250; h=h+5)
{
fprintf(fres2,"%d %d %d %d %d\n",g,h,ng[g][h],ns[g][h],ntot[g][h]);
}
}
}
fclose(fres1);
fclose(fres2);
return 0;
}
我的输入.dat文件(即'trial_final_1.dat')包含以下数据:
150.505951 0.797619 100.000000
172.327438 5.976130 100.000000
137.538651 11.089217 100.000000
151.304276 10.139803 100.000000
137.008926 13.175000 100.000000
120.400734 13.943015 1.000000
136.759262 14.199074 100.000000
在运行我的代码时,最初我得到输出“文件已成功打开,请检查您的文件夹中的result_boxes文件:)”,但在此之后我收到一条消息,指出file.exe已停止工作。请帮我解决这个问题。我也尝试使用while(fscanf(fres1,"%f %f %f",&a[i],&b[i],&c[i])!=EOF )
代替while(!feof(fres1))
,但两种情况下的输出都保持不变。
答案 0 :(得分:1)
您没有对阵列进行任何边界检查。快速浏览一下代码,可以看出你用这个循环超越了ng
,ns
和ntot
的范围:
for(g=5;g<=350;g=g+5)
{
for(h=5;h<=250; h=h+5)
{
...
}
}
上面的问题是您使用的是<=
而不是<
,或者您的数组维度是一个太小的元素。
在写入之前,您也不会检查fres2
是否是有效的文件句柄,也不会检查您对fscanf
的调用是否成功。例如:
fscanf(fres1,"%f %f %f",&a[i],&b[i],&c[i]);
g = ((a[i]/5) + 1)*5;
h = ((b[i]/5) + 1)*5;
如果上述调用失败,您可能正在使用a[i]
或b[i]
的未初始化值,然后继续将其用作数组索引而不进行任何边界检查。这是一个灾难的秘诀,它可能总是会发生,因为你的循环在尝试阅读之前检查EOF 。
至少你应该这样做:
if( 3 != fscanf(fres1,"%f %f %f",&a[i],&b[i],&c[i]) ) break;
还有一件事......当i
达到6001的值时,您认为会发生什么?