web-dev-qa-db-ja.com

Cでトラップ6エラーを中止する

私はこのコードを持っています：

void drawInitialNim(int num1, int num2, int num3)
{
    int board[2][50]; //make an array with 3 columns
    int i; // i, j, k are loop counters
    int j;
    int k;

    for(i=0;i<num1+1;i++)      //fill the array with rocks, or 'O'
        board[0][i] = 'O';     //for example, if num1 is 5, fill the first row with 5 rocks
    for (i=0; i<num2+1; i++)
        board[1][i] = 'O';
    for (i=0; i<num3+1; i++)
        board[2][i] = 'O';

    for (j=0; j<2;j++) {       //print the array
      for (k=0; k<50;k++) {
         printf("%d",board[j][k]);
      }
    }
   return;
}

int main()
{
    int numRock1,numRock2,numRock3;
    numRock1 = 0;
    numRock2 = 0;
    numRock3 = 0; 
    printf("Welcome to Nim!\n");
    printf("Enter the number of rocks in each row: ");
    scanf("%d %d %d", &numRock1, &numRock2, &numRock3);
    drawInitialNim(numRock1, numRock2, numRock3); //call the function

    return 0;
}

これをgccでコンパイルすると、問題ありません。ファイルを実行すると、値を入力した後に中止トラップ6エラーが発生します。

このエラーに関する他の投稿を見ましたが、助けにはなりません。

carraysloopsabort
18
user1753491

これを試して：

void drawInitialNim(int num1, int num2, int num3){
    int board[3][50] = {0}; // This is a local variable. It is not possible to use it after returning from this function. 

    int i, j, k;

    for(i=0; i<num1; i++)
        board[0][i] = 'O';
    for(i=0; i<num2; i++)
        board[1][i] = 'O';
    for(i=0; i<num3; i++)
        board[2][i] = 'O';

    for (j=0; j<3;j++) {
        for (k=0; k<50; k++) {
            if(board[j][k] != 0)
                printf("%c", board[j][k]);
        }
        printf("\n");
    }
}
0
BLUEPIXY

自分が所有していないメモリに書き込んでいます：

int board[2][50]; //make an array with 3 columns  (wrong)
                  //(actually makes an array with only two 'columns')
...
for (i=0; i<num3+1; i++)
    board[2][i] = 'O';
          ^

この行を変更します。

int board[2][50]; //array with 2 columns (legal indices [0-1][0-49])
          ^

に：

int board[3][50]; //array with 3 columns (legal indices [0-2][0-49])
          ^

配列を作成するとき、[3]の初期化に使用される値は配列サイズを示します。
既存の配列にアクセスする場合、インデックス値はゼロベースです。

作成された配列の場合：int board[3][50];
法定インデックスはboard [0] [0] ... board [2] [49]です

[〜＃〜] edit [〜＃〜]悪い出力コメントと初期化コメントに対処するには

出力をフォーマットするための追加の「\ n」を追加します。

変化する：

  ...
  for (k=0; k<50;k++) {
     printf("%d",board[j][k]);
  }
 }

       ...

に：

  ...
  for (k=0; k<50;k++) {
     printf("%d",board[j][k]);
  }
  printf("\n");//at the end of every row, print a new line
}
...

Initializeボード変数：

int board[3][50] = {0};//initialize all elements to zero

（ 配列初期化の議論...）

27
ryyker