わかりません！！

質問

関数についての質問です。
以下で大文字なのはオブジェクト形式のマクロで宣言してあります。
void goukaku( int a)
{
int succs[NUMBER] = {0};
int snum = 0;

for ( i = 0 ; i < num ; i ++ )
if ( tensu{a][i] >= GOUKAKU) succs[snum++] = i;
for ( i = 0 ; i < snum ; i ++ );
printf("%2d番　（％３ｄ点）￥ｎ” , succs[i] + i , tensu[a][succs[i]]);
}

中略
for ( i = 0 ; i < sub ; i ++ )
goukaku(i);

subは教科数、numは人数、配列　tensu[SUB][NUMBER]
sub , numにそれぞれ１を入れると無限ループになるのですが、どうしたらよいでしょうか？？

回答No.4

nagare · ベストアンサー

予測１の場合
for ( i = 0 ; i < num ; i ++ )
{
if ( tensu{a][i] >= GOUKAKU)
{
succs[snum++] = i;
}
for ( i = 0 ; i < snum ; i ++ )←iを違う変数にする
{
printf("%2d番　（％３ｄ点）￥ｎ” , succs[i] + i , tensu[a][succs[i]]);
}
}

予測２の場合
for ( i = 0 ; i < num ; i ++ )
{
if ( tensu{a][i] >= GOUKAKU)
{
succs[snum++] = i;
for ( i = 0 ; i < snum ; i ++ )←iを違う変数にする
{
printf("%2d番　（％３ｄ点）￥ｎ” , succs[i] + i , tensu[a][succs[i]]);
}
}
｝

お礼

わかりにくい質問に回答していただきありがとうございました。回答していただいた方法で直りました。
ありがとうございました。

回答No.3

ngsvx

質問するなら、ソースは正確に書きましょうね。

１． tensu{a][i]
２．for ( i = 0 ; i < snum ; i ++ );
３．printf("%2d番　（％３ｄ点）￥ｎ”

１，３は明らかにコンパイルエラーになります。
２も対応する括弧の数が合わなくなりエラーになりそうです。

もう一度、正確なソースを示してください。

それと、ループが複数ありますが、どのループで無限ループになっているのかも示してください。
あちこちに、表示用関数をしこんでおけばわかることです。
その結果、「示したソース以外の場所で無限ループしてた」ということもありますから。

回答No.2

noname#30871

すみません、寝ぼけて大間違いを書いてしまいました。No.1は忘れてください。

回答No.1

noname#30871

for ( i = 0 ; i < snum ; i ++ );

　このfor文の意図がよくわかりませんが、「i<snum」が続く限り無限にループするのはおわかりですね。つまり、snumがゼロのときです。

　その前でsnumに値を代入しているのは、

if ( tensu{a][i] >= GOUKAKU) succs[snum++] = i;

です。（tensu[a][i]の間違いですね）

このif文条件が成立しない限り、

succs[snum++] = i;

は実行されません。つまり、snum++されません。

すなわち、

for ( i = 0 ; i < num ; i ++ )

のループの中で、

if ( tensu[i] >= GOUKAKU)

が一度も成立しない場合に無限ループになります。