请教这段代码运行时间怎么这么长？

[sssfanatcsdn]

for(i=1; i<=30; i++) //执行完该循环要竟用七、八分钟之多!!这是怎么回事？
{
mark:
for(j=1; j<=100; j++) x[j]=GenerateRandomValue();//这里调用函数生成随机数。
if(check(x)==0) goto mark;
for(j=1; j<=N; j++) A[i][j]=x[j];
}

其中check()如下：
static int check(double x[])
{
for(i = 0; i < N; i++)
if((x[i]<=0)||(x[i]>=C) return 0;
double sum = 0;
for(i = 0; i < N; i++)
sum += array[i]*x[i];
if(sum != 0) return 0;
return 1;
}

[younggle]

还有一处if(sum != 0) return 0;
这里的浮点数是不能直接与 0 比较的，浮点数和整数在计算机上是截然不同的。

至于怎样比较，楼主自己考虑吧。

[black325]

晕~~~~~~~~~~~~~~~~~~~~~~~~~~~~~迷~~~~~~~~~~~~~~~ing

[peterID]

代码不清楚，N , array, C的值可能有问题。
还有你check函数里面是 if(sum!= 0) return 0;
如果这样话，应该会经常返回0的

[sssfanatcsdn]

感谢大家的指点，最近写代码写得也是晕头转向，以致于表达的有点含糊。在向大家表示歉意的同时做如下说明：
1)生成的随机数在0与1之间；
2)N = 100；
3)array[100]是个已有初值的数组；
4)我把C取值为100；

请大家发表高见!!!

[Mybox]

把随即数去绝对值就可以if((x[i]<=0)||(x[i]>=C) return 0;少很都判断。sum!=0也是不对的啊

[sssfanatcsdn]

还没有解决呀!还是弄不明白问题出在哪，唉!

[qybao]

还有啊
for(j=1; j<=100; j++) x[j]=GenerateRandomValue();//j=0的情况没有。
而
static int check(double x[])
{
for(i = 0; i < N; i++)
if((x[i]<=0)||(x[i]>=C) return 0; //这里又是从0开始的，x[0]是什么？
double sum = 0;
for(i = 0; i < N; i++)
sum += array[i]*x[i];
if(sum != 0) return 0;
return 1;
}

for(j=1; j<=100; j++) x[j]=GenerateRandomValue();//为该x[j-1]=GenerateRandomValue();

[healer_kx]

估计就是随即出来的数字常常被if给推翻了呗，。

[jixingzhong]

你把随即数的范围做一个限定 ！

[jixingzhong]

晕忽忽 ！！
你的程序 ：数据范围 int 是 32位的，知道有多大？？if((x[i]<=0)||(x[i]>=C) return 0;函数有 N 的几率返回 0 ，于是循环又继续了 ！！
估计应该是把条件改成 与&&吧 : if((x[i]<=0)&&(x[i]>=C) return 0;

[wscft]

同意把随机数限定范围！

[jixingzhong]

晕 见笑了 ~
上面的错误 .... 表要笑我 呵呵，最进很糊涂 ！

不知道楼主的程序的逻辑是什么 ？
在32位编译器下 INT的范围很大，产生的数据 在((x[i]<=0)||(x[i]>=C) 的范围几率是很大的啊！
（字母 C 好象也就是67吧，也就是说只有 产生了 0-67 之间的数据，才可以 ！更糟糕的是，你的程序是一个循环判断：for(i = 0; i < N; i++) if((x[i]<=0)||(x[i]>=C) return 0; 也就是一旦有一个数据不在 0-67之间，就要重新获取所有的随即数 ... 这个几乎是不可能的 ！ 你的机器能不无限循环，就很不错了 ！！）

[bugebear3]

楼上说得对