配列の数が519051を超えるときにも正常に動作させたい

前提・実現したいこと

引数で与えられたint値の回数だけ 0: 9 の乱数を生成し，各数字の出現頻度を表示するプログラムを作りました。

発生している問題・エラーメッセージ

519051を超えると正常に表示されなくなります。
ある値以上では下のようになり
520000付近では下のようになります。

該当のソースコード

c言語
1```c言語
2#include <stdio.h>
3#include <stdlib.h>
4#include <time.h>
5void hist(int n){
6    int i, k = 0, a[n], b[10] = {0};
7    srand(time(NULL));
8    for (i = 0; i < n;i++){
9        a[i] = rand() % 10;
10        b[a[i]]++;
11    }
12    printf("n=%7d:", n);
13    while(k<=9){
14        printf("  %5.1f", (double)b[k] * 100 / n);
15        k++;
16    }
17    putchar('\n');
18}
19int main(void) {
20    hist(100);
21    hist(1000);
22    hist(10000);
23    hist(100000);
24    hist(519051);/*これを超えるとバグる*/
25    return 0;
26}


### 試したこと
簡単なプログラムを書いたところ配列の数が519051を超えると表示されなくなります。
![イメージ説明](7173d9b52809e6ec739d90f51adb8eb9.png)


### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

hoshi-takanori

2021/05/19 19:27

スタックが溢れてるのでは。 https://teratail.com/questions/15198

takasima20

2021/05/19 21:14 編集

malloc 使ったら?

行動規範の内容に同意します

回答3件

c++で配列のサイズが大きく多くて実行時にエラーが起こる場合の対応

投稿2021/05/20 00:28

jbpb0

総合スコア7651

ベストアンサー

ローカル変数が使える領域はそんなに大きくないんですわ。
なもんだから、大きな領域はmallocで確保(使い終わったらfreeで開放)

C
1void hist(int n) {
2//  int i, k = 0, a[n], b[10] = { 0 };
3    int i, k = 0, b[10] = { 0 };
4    int* a = (int*)malloc(sizeof(int)*n); // 領域を動的に確保する
5    if ( a == NULL ) { // 確保できんかったらギブアップ
6      printf("no more memories, give up\n");
7      return;
8    }
9    srand(time(NULL));
10    for (i = 0; i < n; i++) {
11        a[i] = rand() % 10;
12        b[a[i]]++;
13    }
14    printf("n=%7d:", n);
15    while (k <= 9) {
16        printf("  %5.1f", (double)b[k] * 100 / n);
17        k++;
18    }
19    putchar('\n');
20    free(a); // 領域開放
21}
22...