请各位达人看看运行下面的程序后怎么会出现debug assertion failed

jover0 2009-07-06 09:36:44
#include<stdio.h>
#include<process.h>
#include<iostream.h>
#include<fstream.h>
#include<math.h>

float a[512][512]={0};
#define N 512
typedef long int NatureNumber;
double sum[N/2]={0};
float plus=0;


read()
{
FILE* fp;
int tmp;


fp=fopen("06111-5.dat","r");
if(fp=NULL)
exit(-1);
for (int i=0; i<512; ++i )
{
for (int j=0; j<512; ++j )
{
fscanf(fp,"%d",&tmp);
a[i][j] = (float)tmp*25.1/65535;
plus+=a[i][j];
}
}
plus=plus/512/512;
return 0;
}

int main()
{
read();
NatureNumber i, j;
char file4[] = "Plasma12.dat";
ofstream fs(file4);
for (i = 0; i <N; i++)
{
for (j =0; j <N; j++)
{
fs<<a[i][j]-plus<<" ";
}
fs<<"\n";
}
}



...全文
27 3 打赏 收藏 转发到动态 举报
写回复
用AI写文章
3 条回复
切换为时间正序
请发表友善的回复…
发表回复
spring203 2009-07-06
  • 打赏
  • 举报
回复
楼主,个人意见:
1.代码漏洞 read中FILE* fp,有fopen确没有fclose,ofstream fs(file4)也是。
2.算法漏洞 两个for循环里面有fscan,如果文件中没有512*512数目怎么办,正确的应该是从文件中读出,有多少个保存多少个。
3.代码优化 乘法与除法,可以考虑位操作,这样效率高点。
4.代码风格 变量记得初始化,如int tmp,这个变量不初始化会带来很多麻烦的。
其实还有很多,诚心告诉楼主,如果水平一般可以考虑,写一些简单点的程序与算法,还有每个函数或者类(结构体)的用法要研究透彻点才能使用它。
说法可能有失偏颇,决无恶意,见谅!

bitxinhai 2009-07-06
  • 打赏
  • 举报
回复
fp=fopen("06111-5.dat","r");
if(fp=NULL) //错误,应为if(fp==NULL)
exit(-1);
Ryanwen 2009-07-06
  • 打赏
  • 举报
回复
1、将文件路径替换为绝对路径
2、if(fp=NULL) //这句明显的错误不会没发现吧
exit(-1);

16,472

社区成员

发帖
与我相关
我的任务
社区描述
VC/MFC相关问题讨论
社区管理员
  • 基础类社区
  • Web++
  • encoderlee
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告

        VC/MFC社区版块或许是CSDN最“古老”的版块了,记忆之中,与CSDN的年龄几乎差不多。随着时间的推移,MFC技术渐渐的偏离了开发主流,若干年之后的今天,当我们面对着微软的这个经典之笔,内心充满着敬意,那些曾经的记忆,可以说代表着二十年前曾经的辉煌……
        向经典致敬,或许是老一代程序员内心里面难以释怀的感受。互联网大行其道的今天,我们期待着MFC技术能够恢复其曾经的辉煌,或许这个期待会永远成为一种“梦想”,或许一切皆有可能……
        我们希望这个版块可以很好的适配Web时代,期待更好的互联网技术能够使得MFC技术框架得以重现活力,……

试试用AI创作助手写篇文章吧