Segmentation Fault (core dumped)

Question

#include <stdio.h>
#define row 5
#define column 5
int main() {
int a;
int i,j;
int mat[row][column] = {{1,1,1,1,1},{0,1,1,1,1},{0,0,1,1,1},{0,0,0,1,1,},{0,0,0,0,1}};

printf("反転する方向を選んでください(縦：０、横：１)");
scanf("%d",&a);
if(a == 0) {
for(i = 4; i >= 0; i--) {
for(j = 0; j < column; i++) {
printf("%d ",mat[i][j]);
}
printf("\n");
}
}

if(a == 1) {
for(i = 4; i >= 0; i--) {
for(j = 4; j >= 0; j--) {
printf("%d",mat[j][i]);
}
printf("\n");
}
}
else {
printf("不適切な数値が入力されました");
}
}

これを実行するとSegmentation Fault (core dumped)と出るのですがどこを直せばいいですか？

Comments

[kazto]

コンパイル環境はどこでしょうか。

Answer 1

for(j = 0; j < column; i++)

がiとjでした

Answer 2

Segmentation Faultが出たときは配列範囲外にアクセスしていないか重点的に確かめましょう。

for(i = 4; i >= 0; i--) {
for(j = 0; j < column; i++) {
...
}
...
}

内側のループの更新文、i++じゃなくてj++にするべきですね。

コードの書き方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。

Comments

[LouiS0616]

ところで。
全く同じ課題に見えますが、お二人は同じ講義を受けているのでしょうか。
https://teratail.com/questions/135949

[y_waiwai]

しかし、いまどき、初心者にCの講義って、難易度高杉だとおもうんだが。。

[LouiS0616]

未だに結構あるみたいですよ。私の大学もそうでしたし。

聞いた中で酷い話だと、『ポインタと構造体は難しいから触れない』って方針の講義もあるとか。
それこそもうCをやる意味が全然ないですよね。

[y_waiwai]

う、ううむ。。いまどきならC#をやればいいのに、とおもうんだけどねー

[LouiS0616]

悲しいことにCしか知らない (実際にはCもろくに書けない) 教員がいるみたいです。
彼らはC以外を全く知らないので、全てCで解決できると思っているし、実際にしようとするのです。

---
友人が卒研で微分方程式の解の可視化をしていたのですが、それにすらCを使っていました。
既存のライブラリもあるだろうに、積分もfor文でぐるぐる回して自前で計算していて。

特に酷かったのが、数式の各パラメータを調整する際、毎回手で何か所も書き直してコンパイルしているのです。
設定ファイルを用意しろと。せめてコマンドライン引数を使えと。どんなに説得しても無駄でした。

これ、教授が用意したコードなんだそうです。

[LouiS0616]

『日本刀を使って料理するのをやめろ』って、何度も忠告したんですけどね。馬耳東風でした。教授の言うことだから間違いないんだそうです。虚しい。