【C++】配列を利用したデータ表示をしたいが、アクセス違反が発生する

Question

【C++】配列を利用したデータ表示をしたいが、アクセス違反が発生する

プログラムで処理したいことは以下の通りです。

• 各生徒の成績を、４つの課題の評点に基づきつけ、表示する
• 評点の最高、最低、平均を求め、表示する

以下のプログラムで処理していることは、以下の通りです。

• int型配列に、各生徒の４つの課題の評点を宣言
• 各生徒の評点、評点の合計を表示する

↓プログラム↓

#include<stdio.h>

#define HYOUKALINE_COUNT 4
#define STUDENT_COUNT 5
#define TASK_COUNT 4

int main() {
	int hyouten[STUDENT_COUNT][TASK_COUNT] = {
		{23,14,19,25},
		{13,16,14,17},
		{4,7,11,8},
		{12,15,13,17},
		{25,25,21,23}
	};
	int hyouten_goukei[STUDENT_COUNT] = {
		{0},
		{0},
		{0},
		{0},
		{0}
	};
	int hyoukaline[4] = { 80,50,20 };
	for (int x = 0; x < STUDENT_COUNT; x++) {
		for (int y = 0; y < TASK_COUNT; y++) {
			hyouten_goukei[x-1] += hyouten[x-1][y-1];//C6201, C6385
		}
	}
	printf("評点一覧\n　課題１　課題２　課題３　課題４　合計点\n");
	for (int x = 0; x < STUDENT_COUNT; x++) {
		for (int y = 0; x < TASK_COUNT; y++) {
			printf("　　 %3d",hyouten[x-1][y-1]); //ハンドルされない例外が 0x001A5449 (seiseki.exe) で発生しました: 0xC0000005: 場所 0x01100000 の読み取り中にアクセス違反が発生しました。

		}
		printf("　 %3d",hyouten_goukei[x-1]);
		printf("\n");
	}
	getchar();
}

↓出力↓

評点一覧
　課題１　課題２　課題３　課題４　合計点
　　  39　　  48　　   0　　 -858993460　　 -858993460　　  23　　  14　　  19　　  25　　  13　　  16　　  14　　  17　　   4　　   7　　  11　　   8　　  12　　  15　　  13　　  17　　  25　　  25　　  21　　  23　　 -858993460　　 -2094467618　　 17825220　　 1712915　　   1　　 21362848　　 21371072　　   1　　 21362848　　 21371072　　 17825312　　 1712487　　 -2094467494　　 1708862　　 1708862　　 15126528　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　 1746292　　 1746304　　   0　　 17825228　　   0　　 17825420　　 1720224　　 -2109896854　　   0　　 17825320　　 1712125　　 17825328　　 1713048　　 17825344　　 1990157145　　 15126528　　 1990157120　　 17825436　　 2007137316　　 15126528　　 540115870　　   0　　   0　　 15126528　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　 17825356　　   0　　 17825444　　 2007212096　　 1452416682　　   0　　 17825452　　 2007137268　　  -1　　 2007273438　　   0　　   0　　 1708862　　 15126528　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0　　   0

足りない情報などあれば返信ください！

Answer 1

このプログラムの問題点は3つあって、

1. (他の回答者が指摘済み)配列のインデックスの計算時になぜか-1しているので、インデックスがマイナスになってアクセス違反が出る問題
2. intの一次元配列hyouten_goukeiの初期値が2次元配列で書いてある問題
3. 結果の表示を行なうfor文のうち、内側のfor文の継続条件が間違っている(yと書くべきところにxと書いてある)問題

これらを解決すれば、評点一覧が表示されます。

Comments

[_____________]

無事正常に動作しました！
↓出力↓
評点一覧
　課題１　課題２　課題３　課題４　合計点
　　 23　　 14　　 19　　 25　 81
　　 13　　 16　　 14　　 17　 60
　　 4　　 7　　 11　　 8　 30
　　 12　　 15　　 13　　 17　 57
　　 25　　 25　　 21　　 23　 94

Answer 2

hyouten_goukei[x-1] += hyouten[x-1][y-1];//C6201, C6385

x あるいはyがゼロのときは、この配列のインデックスはマイナスとなってしまいます
文字通り、アクセス違反となります

Comments

[_____________]

x-1,y-1の「-1」を取り除けば良いということですか？

[Daregada]

むしろ、なぜ「x-1」や「y-1」にする必要があると思ったのですか?

xやyが1から始まるならともかく、0から始まるのですから、-1する必要はどこにもないのですけど。

[y_waiwai]

あなたの組んだそのコードに不要なものなら、取り除くべきです。

そうでないなら、単純に取り除いたらまずいでしょうし、そうであるなら、アクセス違反にならないように修正するってことになります

[_____________]

直してみたところ、31行目のprintf("　　 %3d",hyouten[x][y]);でもアクセス違反が発生してしまいました。
ハンドルされない例外が 0x00515442 (seiseki.exe) で発生しました: 0xC0000005: 場所 0x01102000 の読み取り中にアクセス違反が発生しました。