c言語のプログラムが思うように動きません

Question

全ての要素が1の10行10列の行列を命令によってその要素のいくつかを0にして、すべての要素の合計を数えるプログラムを作りたいのですが、間違った結果が出てしまいます。どこが間違っているのか教えてください。

コード#include <stdio.h>

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[c][i] = 0;
	}
}
    else if(n == 2){
    int i;
	for(i = s; i <= e ; i++){
		state[i][c] = 0;
	}
    }
    else{}
}
	
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);
		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 );
	execute(state, a);
	for(i = 0; i < 10; i++ ){
    	for( j = 0; j < 10; j++){
    		sum = sum + state[i][j];
    	}
    }
	printf("%d",sum);
}

Answer 1

１．ＯＳ、開発ツールは何を使っていますか？
２．デバッガの使い方は練習していませんか？
⇒エディタ、コンパイラ、デバッガを使いこなせる事が
最低限の、プログラミングの基本です。
それらが、一体になったのが統合開発環境（ＩＤＥ）です。
今では、マイコン用でも統合開発環境が、あたりまえで普通だったりします。
３．デバッガが使えない場合は、デバッグコードを仕込み、
各種必要な情報を、ダンプや、表示出来るように工夫します。
昔々の何も無い時代ではないのですから、御自分でも工夫をしましょう。

Answer 2

思うように動く前にコンパイルできませんが・・・
void order(int state[][], int n,int c, int s, int e)と
void execute(int state[][], int n, int c, int s, int e)でエラーになりませんでしたか？
コンパイルエラーの状態を質問するとき以外は、ちゃんとしたソースを提示しましょう。

Comments

[cateye]

エラー内容
~/test gcc tst91.c
tst91.c:3:16: error: array type has incomplete element type
void order(int state[][], int n, int c, int s, int e)
^
tst91.c:19:18: error: array type has incomplete element type
void execute(int state[][], int a)
^
tst91.c: In function 'execute':
tst91.c:26:9: error: type of formal parameter 1 is incomplete
order(state, n, c, s, e);
^
tst91.c: At top level:
tst91.c:30:1: warning: return type defaults to 'int' [-Wreturn-type]
main()
^
tst91.c: In function 'main':
tst91.c:44:10: error: type of formal parameter 1 is incomplete
execute(state, a);
^
tst91.c:51:1: warning: control reaches end of non-void function [-Wreturn-type]
}
^
~/test

Answer 3

まずは、２次元派配列を関数にどう渡すか？ を解決することが必要とおもいます。

参考：

• ２次元配列の引数渡し http://murakan.cocolog-nifty.com/blog/2011/08/cc-b731.html

質問にあるケースをベースに、配列の内容を表示する関数をつくるとしたら、１つの実装例としてはこんな感じになります。

#include <stdio.h>

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

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");
}

int main() {
  int state[MATRIX_SIZE][MATRIX_SIZE];
  int i;
  int j;
  for (i = 0; i < MATRIX_SIZE; i++ ) {
    for ( j = 0; j < MATRIX_SIZE; j++) {
      state[i][j] = 1;
    }
  }

  show_data(state);
  return(0);
}

実行例：

$ gcc m.c
$ ./a.out
------
1 1 1 1 
1 1 1 1 
1 1 1 1 
1 1 1 1 
------

このコードをベースにして、 0 の要素を設定する関数、 合計を計算する関数を追加していくとよいと思います。

Answer 4

scanfを使っているので、意図しない入力になっている可能性があります。
scanfの直後で、入力値をprintfしてみてください。

アドバイスとしては、
１．scanfは戻り値をチェックすることが必須(sscanf等も同様)
２．scanfでなく、fgetsとsscanfに書き換えた方が良い

Answer 5

試した数字を教えてください、プログラムは間違えないですね・・・。
ただ、配列オーバーした時の挙動は不定なので、制御を入れる方が良いです。

Comments

[退会済みユーザー]

入力１、１、３、２、６のとき95　２、１、３、２、６、１、３、４、７のとき94　　２、１、３、２、６、２、５、０、４のとき９１　　４、１，３，２，６、２，６，３，８、1，8，2，6、2，2，3，8もとき82がでればよいのですが結果は違います

[MasaakiIrie]

４、１，３，２，６、２，６，３，８、1，8，2，6、2，2，3，8のとき７８てでます？

[退会済みユーザー]

動作を停止してしまいます・・・

[MasaakiIrie]

見る限りでは、エラー終了するような処理には見えませんけど・・・
上記のものは、動作したものを張り付けていますか？
記述してるものは多分コンパイルが通らないとおもいますけど・・・。