C言語 配列の和 要素指定の位置を入力した後から間違っていると思います。

Question

実行時に mと nとを入力し， m行 n列の要素を含む左下から右上がりライン全体にあるすべての要素の合計値sumを求めるプログラムを作成する．

自分の作成したプログラム↓-----------------
#include<stdio.h>
#define M 3
#define N 4
int main(void) {
int a[M][N]={
{1, 7, 1, 5},
{0, 2, 8, 1},
{4, 0, 2, 8}
};
int sum;
int m, n;
int i, j;

printf("配列a[M][N]：\n");

for(i=0; i<M; i++){
                   for(j=0; j<N; j++){
                                      printf("%d", a[i][j]);
									  }
					}	
printf("m?");
scanf("%d", &m);
printf("n?");
scanf("%d", &n);
sum=0;
for(i=m-1; i<M; i++){
                     for(j=n+1; j>=0; j--){
                                            sum=sum+a[i][j];
                                           }
                    }

printf("右上がりライン(%d %d)にある要素の合計 = %d\n", m, n, sum);

return 0;

}

結果が想定とはことなるため訂正箇所とアドバイスお願いします。

Answer 1

修正前の質問タイトル：『C言語 配列 魔法陣の和』に対する指摘
魔法陣ではなく魔方陣だと思いますが、どちらにせよ意味が通じません。

魔方陣（まほうじん、英：Magic square）とは、n×n 個の正方形の方陣に数字を配置し、縦・横・対角線のいずれの列についても、その列の数字の合計が同じになるもののことである。特に1から方陣のマスの総数 n2 までの数字を1つずつ過不足なく使ったものを言う。

引用元：魔方陣 - Wikipedia (太字は引用者)
ご提示のプログラムのどこに魔方陣があるのでしょうか。

訂正箇所とアドバイスお願いします。

強いて言うなら、行列を表示する際にはこうした方が見やすい、ということでしょうか。

printf("配列a[M][N]：\n");
for(int y=0; y<M; y++){
    for(int x=0; x<N; x++){
        printf("%d ", a[y][x]);
    }
    printf("\n");
}    

質問の追記／修正を受けて

とりあえずこんなコードを組んでみました。
『ある座標を打ち込まれたときに、可能な限り左下に突き進んだ座標を返す』

#include <stdio.h>
#include <stdlib.h>
#define X 4
#define Y 3

int main(void) {
    int mat[Y][X]={
        {1, 7, 1, 5}, 
        {0, 2, 8, 1}, 
        {4, 0, 2, 8}  
    };

    printf("配列mat[Y][X]：\n");
    for(int yi=0; yi<Y; yi++){
        for(int xi=0; xi<X; xi++){
            printf("%d ", mat[yi][xi]);
        }
        printf("\n");
    }    
    
    int x, y;
    printf("x?"); scanf("%d", &x);
    printf("y?"); scanf("%d", &y);

    if(!(0 <= x && x < X)) exit(1);
    if(!(0 <= y && y < Y)) exit(1);

    while(x > 0  && y < Y - 1) {
        x--; y++;
    }
    printf("x:%d y:%d\n", x, y);

    return 0;
}

実行例

配列mat[Y][X]：
1 7 1 5
0 2 8 1
4 0 2 8
x?1
y?1
x:0 y:2

次の三点を考えてみてください。

• なぜ上記のようなコードを例として示したのか？
• コードがところどころ(特に変数名が)書き換えられているが、真意はなにか？
• 『左下から右上へ』の一直線の和を考えるときに、二重ループが必要か？

どうしてもわからなければ、わからない点を自分なりに整理してコメントをください。

---

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

また、質問をする際には、上手くいかない点を明確にした方がよいです。

• コンパイルが出来ない
• 実行中に落ちる
• 特定の入力の下で動かない
• 結果が想定通りでない

今回はわりあい期待する結果が自明ですが、それでも期待する入出力は書いた方がよいです。