私はこのコードを持っています:
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エラーが発生します。
このエラーに関する他の投稿を見ましたが、助けにはなりません。
これを試して:
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");
}
}
自分が所有していないメモリに書き込んでいます:
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
( 配列初期化の議論...)