<解決しました>全ての要素が1である10×10行列の要素をある条件に従って0にし、要素の合計を表示するc言語のプログラム

[po_po]

入力は、初めにいくつの命令があるか、次に命令の種類、その次に行列の要素を0にする範囲を指定しています

入力が1,1,3,2,6のときは95
2,1,3,2,6,1,3,4,7なら94
4,1,3,2,6,2,6,3,8,1,8,2,6,2,2,3,6なら82を表示すれば正しいのですが、違った値が表示されます。
プログラムのどこが間違っているのか教えてください。

コード:

#include <stdio.h>


// 配列にサイズを定義する。
#define MATRIX_SIZE (10)

void show_data(int state[MATRIX_SIZE][MATRIX_SIZE]) {
  int i, j;
  printf("------\n");
  for (i = 0; i < MATRIX_SIZE; i++ ) {
    for (j = 0; j < MATRIX_SIZE; j++) {
      printf("%d ", state[i][j]);
    }
    printf("\n");
  }
  printf("------\n");
}



void order(int state[][], int n,int c, int s, int e){// nが1ならc列のs行からe行に0を代入し、2ならc行のs列からe列に0を代入する
	int i ;
	if(n == 1){
	for(i = s; i <= e ; i++){
		state[i][c] = 0;
	}
}
    else if(n == 2){
    int i;
	for(i = s; i <= e ; i++){
		state[c][i] = 0;
	}
    }
    
}
	
void execute(int state[][], int a){// aの値の数だけorderを実行する
	int i;
	int n, c, s, e;
	
	for(i = 1; i <= a; i++){
		scanf("%d%d%d%d", &n, &c, &s, &e);
		printf("%d %d %d %d \n",n ,c ,s ,e );
		order(state , n, c, s, e);
	}
}
	
main(){
	int a;
	int state[10][10];
    int i;
    int j;
    int sum = 0;
    for(i = 0; i < 10; i++ ){
    	for( j = 0; j < 10; j++){
    		state[i][j] = 1;
    	}
    }
    
	scanf("%d", &a );
	printf("%d\n",a);
	execute(state, a);
	for(i = 0; i < 10; i++ ){
    	for( j = 0; j < 10; j++){
    		sum = sum + state[i][j];
    	}
    }
    show_data(state);
	printf("%d",sum);
}

[へにっくす]

とりあえずそのままでは、VC++に通すことはできなかったので、以下のように修正しました。

コード:

#include <stdio.h>


// 配列にサイズを定義する。
#define MATRIX_SIZE (10)

void show_data(int state[MATRIX_SIZE][MATRIX_SIZE]) {
    int i, j;
    printf("------\n");
    for (i = 0; i < MATRIX_SIZE; i++ ) {
        for (j = 0; j < MATRIX_SIZE; j++) {
            printf("%d ", state[i][j]);
        }
        printf("\n");
    }
    printf("------\n");
}



void order(int state[MATRIX_SIZE][MATRIX_SIZE], int n,int c, int s, int e) { // nが1ならc列のs行からe行に0を代入し、2ならc行のs列からe列に0を代入する
    int i ;
    if(n == 1) {
        for(i = s; i <= e ; i++) {
            state[i][c] = 0;
        }
    }
    else if(n == 2) {
        int i;
        for(i = s; i <= e ; i++) {
            state[c][i] = 0;
        }
    }

}

void execute(int state[MATRIX_SIZE][MATRIX_SIZE], int a) { // aの値の数だけorderを実行する
    int i;
    int n, c, s, e;

    for(i = 1; i <= a; i++) {
        scanf("%d%d%d%d", &n, &c, &s, &e);
        printf("%d %d %d %d \n",n ,c ,s ,e );
        order(state , n, c, s, e);
    }
}

void main() {
    int a;
    int state[10][10];
    int i;
    int j;
    int sum = 0;
    for(i = 0; i < 10; i++ ) {
        for( j = 0; j < 10; j++) {
            state[i][j] = 1;
        }
    }

    scanf("%d", &a );
    printf("%d\n",a);
    execute(state, a);
    for(i = 0; i < 10; i++ ) {
        for( j = 0; j < 10; j++) {
            sum = sum + state[i][j];
        }
    }
    show_data(state);
    printf("%d",sum);
}

実行結果

コード:

C:\Users\MIX-C\Documents>1
1
1
1 3 2 6
1 3 2 6
------
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
------
95
C:\Users\MIX-C\Documents>1
2
2
1 3 2 6
1 3 2 6
1 3 4 7
1 3 4 7
------
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 0 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
------
94
C:\Users\MIX-C\Documents>1
4
4
1 3 2 6
1 3 2 6
2 6 3 8
2 6 3 8
1 8 2 6
1 8 2 6
2 2 3 6
2 2 3 6
------
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 0 0 0 0 1 0 1
1 1 1 0 1 1 1 1 0 1
1 1 1 0 1 1 1 1 0 1
1 1 1 0 1 1 1 1 0 1
1 1 1 0 0 0 0 0 0 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
------
83
C:\Users\MIX-C\Documents>

実行結果を見る限りだと、最後の値だけ83と違いますが、このことを言っていますか？
もしかして最後のは2,2,3,6でなく2,2,3,8では？

コード:

C:\Users\MIX-C\Documents>1
4
4
1 3 2 6
1 3 2 6
2 6 3 8
2 6 3 8
1 8 2 6
1 8 2 6
2 2 3 8
2 2 3 8
------
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 0 0 0 0 0 0 1
1 1 1 0 1 1 1 1 0 1
1 1 1 0 1 1 1 1 0 1
1 1 1 0 1 1 1 1 0 1
1 1 1 0 0 0 0 0 0 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
1 1 1 1 1 1 1 1 1 1
------
82
C:\Users\MIX-C\Documents>

[box]

入力は、初めにいくつの命令があるか、

まあこれはわからなくはないですが、

次に命令の種類、その次に行列の要素を0にする範囲を指定しています

このあたりに来ると「？？？」となってしまいます。
命令の種類って何でしょうか？
要素を0にする範囲の指定方法は何でしょうか？

[po_po]

解決しました。ありがとうございました。
二次元配列の関数への渡し方がよくわかっていなかったのが原因でした。
あと最後の入力は6じゃなく8でした。

[みけCAT]

解決したら、解決チェックをお願いします。
タイトルの変更を含めた不要な編集はおやめください。

[naohiro19]

タイトルを変更しただけでは解決したとみなせません。
タイトルを変更せず解決にチェックをするようお願いします。

また、解決した時は、「解決しました」とだけ言って去らず、ソースコードや解決した方法を明記して下さい。
同じ事で困っている人の為に過去ログに有用な情報を残すようお願いします。

[po_po]

すいません・・・
解決チェックというのはどうすればできるのでしょうか？

[Dixq (管理人)]

「返信」をクリックして出てくる返信を書く入力欄の下に

dome.png (32.26 KiB) 閲覧数: 4552 回

上記のような「解決」のチェックボックスがあると思います。

また、解決チェック時にはどのように解決したかを書いて頂けると
後に閲覧した人のためになるので、よければご協力お願いします。

[po_po]

へにっくす様
ありがとうございました。
配列を受け取る関数を作るとき、１次元目の要素数は省略できるけれどそれ以降は省略できないことを知らなかったので起きた間違いでした。
修正してもらったとおり、show_data以降の引数がstate[][]と２次元目に具体的な数が入っていなかったので、state[MATRIX_SIZE][MATRIX_SIZE]とすることで解決しました。
最後の入力はこちら側の間違いでした。

box様
すいません。
次からはより具体的にわかりやすく質問します。

みけCAT様、naohiro19様
すいません
利用の際のルールをよく把握していませんでした。以後気を付けます。

Dixq様
丁寧に教えてくれてありがとうございます。助かりました。