c言語を実行すると無限ループしてしまいます。

Question

#include <stdio.h>
#define NUMBER 80　/*人数の上限*/
int main(void) {
  int num, i, tensu[NUMBER], bunpu[11], j;
printf("人数を入力してください:");
do {
  scanf("%d", &num);
  if(num<0 || num>NUMBER) {
printf("1〜%dで入力してください:", NUMBER);
  }
} while (num<0 || num>80);
printf("%d人の点数を入力してください\n", num);
for(i=0; i<num; i++) {
  printf("%d番:", i+1); 
  do {
    scanf("%d", &tensu[i]);
    if(tensu[i]<0 || tensu[i]>100) {
      printf("\a1〜100で入力してください:");
    }
  } while(tensu[i]<0 || tensu[i]>100);
  bunpu[tensu[i]/10]++;
}
printf("\n");
printf("---分布グラフ---\n");
for(i=0; i<=9; i++) {
  printf("%d〜%d:", i*10, i*10+9);
  for(j=0; j<bunpu[i]; j++) {
    putchar('*');
  }
  printf("\n");
}
if(bunpu[10]) {
  printf("    100:");
  for(j=1; j<=bunpu[10]; j++) {
    putchar('*');
  }
}
printf("\n");
  return 0;
}

①上記のコードを実行すると、分布グラフで『0~9:*****・・・・・』のように無限ループしてしまいます。理由を教えてください。
②tensu[NUMBER]の部分ですが、[]の中は配列の要素数であると認識しております。NUMBERは人数の上限であり、変数tensuの要素数ではないと思うのですが、どうしてここにNUMBERを入れるのですか？要素数ならもう一つの変数numを入れるのが適切だと思います。ちなみにnumを入れるとエラーが出たので、おそらく[]の中に変数を入れるのはダメなんですよね。

二つ質問してしまってすみません。回答お願いします。

Answer 1

①
bunpuが初期化されていません。

for(j=0; j<bunpu[i]; j++) {
    putchar('*');
}

が大量にループするというところからbunpuが怪しいな、という疑い方をします。

②

おそらく[]の中に変数を入れるのはダメなんですよね。

Cのバージョンによってはnumを要素数にしても通ります。

int main(void) {
    int num, i, bunpu[11], j;
    printf("人数を入力してください:");
    do {
        scanf("%d", &num);
        if(num<0 || num>NUMBER) {
            printf("1〜%dで入力してください:", NUMBER);
        }
    } while (num<0 || num>80);
    
    int tensu[num];
    

Cのバージョンによっては変数を要素数にできないので、
代わりに十分な領域を確保して、実際使うのはnum個としているだけです。

Comments

[K.M.PEANUTS]

モヤモヤが一気に解決しました、ありがとうございます。もう一つ質問になってしまうのですが、tensu[NUMBER]は初期化しなくていいのですか？読んでいる本ではbunpuだけしか初期化を行っていません。

[ozwk]

scanfで値を入れているので、使う分(num個)は初期化しています。

[K.M.PEANUTS]

なるほど！全部理解できました！本当にありがとうございます????‍♂️

Answer 2

bunpu配列を初期化していません。

C言語の場合、ローカル変数を初期化せずに使った場合、結果は未定義です（何が起きるか保証されません）。