c言語

質問

int kexnsakusuu(int n[30], int c[30],int g){
int y = 0;
int h = 0;
int flag = 0;
int flag_1;
int i=1;
int u;
while(c[i]>0){
h++;
u = 0;
flag_1=0;
while(n[u]>0 && flag_1==0){
if(abs(n[u] - c[i]) <= g){y++;flag_1++;};
u++;
}
i++;
}
if(y == h)flag = 1;
return flag;
}
n[30]に１２　１８　３　３　０　０　０　０　以下０
c[30]に３０００　１２　３　０　０　０　０ 以下０
ｃ[１] c[２]とn[]の絶対値が２以下（ｇは２）で調べたいのですが、上記だとf‘lagは１を返したいです。
９０００件のn[]を調べると誤動作があるみたいです。

回答No.1

AsarKingChang

ざっとしか、見てないんですが。
>９０００件のn[]を調べると誤動作があるみたいです。
n[]が30個しか配列がないので、最大でも30でBreak
かけたほうがいいですよね。
9000など可変長に受け取りたいなら、*nで受け取ったほうが
まだましです。
ただ、ざっと見た感じなので、まだ何がしたいのかまでは
読み取れてませんが。
とりあえず、
iとuが30（を含む）以上になったら、停止したほうがいいですよ。
i++;の後に
if (i>30) break;
など