Q&A
解決済
総合スコア1
投稿2025/09/19 12:01
編集2025/09/20 12:29テストの点数の分布図を作成したいです。
例えば、6人の生徒の点数が30点・50点・55点・59点・75点・79点だったとすれば、
* * * * * *
0 10 20 30 40 50 60 70 80 90 100
このような分布図を作りたいです。
入力する生徒の数が11までは問題なく分布図を作れるのですが、12のときに分布図の上に改行文字が出力され、13を超えると空白を無限に出力する無限ループが起こってしまいます。
error
1エラーメッセージはなしです。
c言語
1#include <stdio.h> 2 3int main(void) 4{ 5 int no; 6 7 printf("生徒の数は？："); scanf("%d", &no); 8 9 int a[no]; // データ全体の数 10 int bunpu[11] = {0}; // 各点数のデータの数 0～9・10～19・…・90～99・100で分ける 11 12 for (int i = 0; i < no; i++) { // データの読込 13 do { 14 printf("%d人目の点数：", i + 1); scanf("%d", &a[i]); 15 if (a[i] < 0 || a[i] > 100) printf("\a正しく点数を入力してください。\n"); 16 } while (a[i] < 0 || a[i] > 100); 17 bunpu[a[i] / 10]++; // 各点数のデータの数を増やす 18 } 19 20 int max = bunpu[0]; 21 22 for (int i = 1; i < no; i++) { // 各点数のデータの数のなかで最大のものを求める 23 if (max < bunpu[i]) max = bunpu[i]; 24 } 25 26 int max2 = max; // for文の制御式に用いるmaxをmax2とする 27 28 for (int i = 1; i <= max2; i++) { // 分布図の行の繰り返し 29 for (int q = 0; q < 11; q++) { // 分布図の列の繰り返し *を打つための繰り返し 30 if (bunpu[q] >= max) printf(" * "); 31 else printf(" "); 32 } 33 putchar('\n'); 34 max--; 35 } 36 37 printf("---------------------------------\n"); // 分布図の目盛の部分 38 printf(" 0 10 20 30 40 50 60 70 80 90 100\n"); 39 40 return 0; 41} 42
for文が意図せず無限ループしてしまう理由はネット上で見つけられませんでした。
特になし
回答3件
0
for文が意図せず無限ループしてしまう理由はネット上で見つけられませんでした。
どうしてそこで「ネット上」を探すのかちょっと疑問に思ったり。探した結果に「プログラムが間違っているから」というのはなかったのかしら。
答えが分かった段階でこんな話しても面白くないかもしれないけれど、いわゆるprintfデバッグで容易に原因を突き止められたのでは、と思います。
forループが無限ループになると思ったのなら、ループに関係する変数(特に継続条件に関わるもの)を確認しましょう。
for (int i = 1; i <= max2; i++) { // 分布図の行の繰り返し
の次の行に、
printf("i:%d max2:%d", i, max2);
を入れてみます。「無限」状態ではmax2が変な値になっていることが容易に見えたでしょう。(無限ではなく、iがmax2に至るまで待てれば、ループは止まったのでしょう)
max2は、
int max2 = max; // for文の制御式に用いるmaxをmax2とする
で値が決まっていますから、どうやらmaxが変な値になっているっぽいですね。
ではmaxはどうやって決まっていますか？
int max = bunpu[0]; for (int i = 1; i < no; i++) { // 各点数のデータの数のなかで最大のものを求める if (max < bunpu[i]) max = bunpu[i]; }
おかしいのはmaxだということなので、maxの値を追跡します。ループ中、最後がいいかな
printf("i:%d max:%d", i, max);
を入れてみましょう。
なお、この後の表示とかは(画面が流れてしまうので)止めてしまってもかまわないでしょう。for文が終わったところでreturnしてしまえばとりあえずいいですね。
この段階で、iがbunpuの要素数を超えて、maxが不定の大きな数になったりすることに気がつくと思います。
なんで？ループの継続条件にi < noと書かれている(あなたが書いた)からですね。
某知恵袋あたりでこんなこと書くと「ゴタクはいいから間違っている場所だけ教えろ」と怒られたりしそうですが。
あるいは、そのソースコードをまるっとAIに投げれば「ここがおかしい」と指摘してくれます。
でも、この程度のミスは自分で見つけられるようにしておいた方がいいですね。
Cで値が吹っ飛ぶとかそういう場合、配列の添え字か、ポインタの計算の間違いであることは非常に多いです。いきなりfor文が無限ループになる話をネットで探す前に、その辺を確認してみましょう。
投稿2025/09/19 23:30
総合スコア7770
0
プログラムを勉強中と思われますので、ご提示されているコードについてこうしたらどうかというのを勝手に書いてみます。(個人の嗜好・志向が多少なりとも反映されています。)
変数名は英単語にしましょう
プログラムの文は基本英単語で構成されるため、変数名等も英単語を基本とするのが慣習です。
配列 a は配列である必要はありません
配列 a は i 番目の要素が入力の受けと分布データ配列の指定に使われるだけです。
複数のデータの入力には配列を使うことが多いですが、それは複数の入力を後からまた使うのが前提ですので、"ループ1回内で使って終わり" のような入力なら、ただの変数で受けて十分です。
同じ式を複数個所に書かないほうが良いです
入力した値が正常値/異常値かを判断する式が2か所(ifとwhile)にあります。
このようにすると正常値/異常値の式が変わった場合に両方を修正しないといけなくなり間違えの元になりますので、1つに纏まるように書くようにすると良いと思います。
"異常値ならメッセージ表示"と"異常値ならループ継続"の2つになっている所を、無限ループ内で"正常値ならループを抜ける" "ループを抜けなかったらメッセージ表示"とすることで条件判断は1つになります。
最大値を求めるだけのループは(必ずしも)必要ありません
全ての値を入力しデータを集めてから改めてループして最大値を求めていますが、件ではデータの更新は一箇所だけですので、データを集めながら最大値を求めることも出来ます。
プログラムの規模が大きくなると分かり易い様に処理を分けるようになりますが、この規模では単独の処理とするよりデータ収集の一環として纏めてしまったほうが良さそうに思います。
(文字通り)逆転の発想で max2 を不要にできます。
max2 は、ループ終了条件が変わらないようにするためだけに使われています。
が、ループの向きを 0→max で無く max→0 にすれば、max2 は(ついでにループカウンタも)不要になります。
同じ部分を少し違えて表示するのなら、 3項演算子 というのが使えます
" * " と " " を条件で切り替えて表示するなら、真ん中の '*'/' ' の部分だけを 条件?'*':' ' という 式 で表すことが出来ます。式なので printf 内に埋め込めます。
コメントに頼らないようにしましょう
コメントの書き方や在り方については様々ありますが、個人的にはコメントに頼るとプログラム自体の理解が進み難いと思います。
例えるなら、英語の映画で日本語字幕を追っていては英語学習の役には立ち難いような感じでしょうか。
ということで、以下のようにしてみました。コメントはほぼそのまま+追加です。
c
1#include <stdio.h> 2 3int main(void) { 4 int num; 5 printf("生徒の数は？："); 6 scanf("%d", &num); 7 8 int datas[11] = {0}; // 各点数のデータの数 0～9・10～19・…・90～99・100で分ける 9 int max = 0; //datas 内の最大値 10 11 for(int i=0; i<num; ++i) { // データの読込 12 int a; 13 while(1) { //無限ループ 14 printf("%d人目の点数：", i+1); 15 scanf("%d", &a); 16 if(0 <= a && a <= 100) break; //正常範囲ならループを抜ける 17 printf("\a正しく点数を入力してください。\n"); //ココにくるということは正常範囲外なのでエラー表示 18 } 19 int v = ++datas[a/10]; // 各点数のデータの数を増やし、その値を v に代入する 20 if(max < v) max = v; // 各点数のデータの数のなかで最大のものを求める 21 } 22 23 for( ; max>0; --max) { // 分布図の行の繰り返し 24 for(int j=0; j<11; ++j) { // 分布図の列の繰り返し *を打つための繰り返し 25 printf(" %c ", datas[j]>=max?'*':' '); 26 } 27 putchar('\n'); 28 } 29 printf("---------------------------------\n"); // 分布図の目盛の部分 30 printf(" 0 10 20 30 40 50 60 70 80 90 100\n"); 31 32 return 0; 33}
* * * * * * --------------------------------- 0 10 20 30 40 50 60 70 80 90 100
投稿2025/09/20 04:44
総合スコア13429
0
ベストアンサー
サッと見て下記のところは動作が怪しいと思いました。
for (int i = 1; i < no; i++) { // 各点数のデータの数のなかで最大のものを求める if (max < bunpu[i]) max = bunpu[i]; }
bunpuは0～10の要素を持つ配列ですが、人数であるnoまでループを回してbunpuの要素を参照しています。
人数が12のとき iの最大値は11となりますが、bunpuは[11]の要素を持っていないため配列外の領域を参照してしまうのではないでしょうか。
投稿2025/09/19 13:25
総合スコア319
あなたの回答
tips
プレビュー
