この回答ではどうしてできないのか分かりません。（C, 配列）

Question

###前提・実現したいこと
あるC言語の入門書における、配列の分野での問題です。
0~10点の整数で点数をつけるテストで、20人分の点数を入力する。
入力された数値を元に、以下のような形で集計して表示させる。

1点 ○人
2点 ×人
・
・
10点 □人

###発生している問題・エラーメッセージ
エラーメッセージは出ませんが、上記における○や×といった数値が848993459のようにおかしくなります。他は大丈夫です。
解答があり、そこには自分よりもスマートな回答が載っていましたが、自分の回答ではどうしてできないのか分かりません。

###該当のソースコード

#include <stdio.h>

int main() {
	int data[20];
	int score[11];
	int loop,loop2 ;

	//1-20番目までの人のそれぞれの点数を入力

	for (loop = 0; loop < 20; loop++) {
		do {

			printf("%d番目の人の点数を入力してください: ", loop + 1);
			scanf("%d", &data[loop]);
		} while (data[loop]<0 ||  10<data[loop]);
	}
	//0-10の点数について、1-20番までの人を毎回照合。点数が一致する度にscore[点数]を１増やす。

	for (loop2 = 0; loop2 < 11; loop2++) {
		for (loop = 0; loop < 20; loop++) {
			if (data[loop] == loop2) {
				score[loop2]++;
			}
		}
	}
	
	//それぞれの点数について増やしたscore[点数]を表示
	for (loop2 = 0; loop2 < 11; loop2++) {
		printf("%d点は%d人です。\n", loop2 , score[loop2]);
	}
  
	return 0;
}

###試したこと

###補足情報(言語/FW/ツール等のバージョンなど)
Visual Studio 2017 RC Community
C言語
Windows7

Answer 1

int score[11];

ここで変数宣言するだけでは、score配列の「中身」は「不定」の状態であり、メモリ上の数値はめちゃくちゃなものが入っています。なので、そこに++で1を足しても、0からのカウントアップではなくめちゃくちゃな数字に1足されるだけです。

int score[11] = {};

Cではこう書くだけで配列の0初期化ができるので、これで0からのカウントアップになるはずです。

訂正

すみません、上のコードはC++ではいけるのですが、純粋なCではコンパイルエラーになります。こう書いてください。

int score[11] = {0};

なんでこうなるのかという説明は、以下のページに詳しいことが書いてあります。
https://cpplover.blogspot.jp/2010/09/blog-post_18.html

Comments

[gotutiyan]

初期化をすることで解決しました。それとともに、０の初期化の方法がそのような方法でできるとは初めて知りました。貴重なアドバイスありがとうございました。

[masaya_ohashi]

すみません、ベストアンサーに選んでもらった後なんですが訂正を足しておきました。実際コーディング刷る際は気をつけてください。

[gotutiyan]

確認しました。ありがとうございます。

Answer 2

score[]が初期化されていないままscore[loop2]++;としているので結果が不定になるんだと思います。
最初にscoreの配列の内容を0で初期化するコードを追加すればいいと思います。

Comments

[gotutiyan]

初期化をすることで解決しました。ありがとうございました。

Answer 3

配列score[]の要素が初期化されていないためこれを++でインクリメントしても得たい値は得られません。
初期化されていないとは「0かも知れないし848993450かもしれない-5000かも知れないそれ以外かもしれない」値が設定されています。

以下のようにloop2のfor文の処理の最初に0を代入してください。

    for (loop2 = 0; loop2 < 11; loop2++) {
        score[loop2] = 0; // このように0で初期化してください。
        for (loop = 0; loop < 20; loop++) {
            if (data[loop] == loop2) {
                score[loop2]++;
            }
        }
    }

Comments

[gotutiyan]

初期化をすることで解決しました。ありがとうございました。

Answer 4

score[n]を初期化しないと処理系によってはうまくいきません。
初期値が0というのは処理系によってはあるかもしれませんが、そうでない前提で書いたほうがいいです。

Comments

[gotutiyan]

初期化をすることで解決しました。ありがとうございました。

Answer 5

変数scoreを初期化していないだけではないでしょうか。
変数は明示的に初期化しないと不定値が入っています。
なので最初に0で初期化する処理を加えれば良いと思いますよ。

Comments

[gotutiyan]

初期化をすることで解決しました。ありがとうございました。

Answer 6

int score[11];と宣言しただけでは、値は不定です。そのまま使うと、まともに動きません。

最初にscore[]の各要素を0クリアしておきましょう。

Comments

[gotutiyan]

初期化をすることで解決しました。ありがとうございました。

Answer 7

score配列の状態が不定なためです。最初に全要素を０で初期化しましょう。

Comments

[gotutiyan]

初期化をすることで解決しました。ありがとうございました。