2次元配列をランダムに塗りつぶします

Question

このコードはn <に適していますが、n = 40ではアクセス違反エラーが発生します： このコードはXとOをランダムに埋めます。

int i=0,j=0; 
int x=0,y=0; 
int n=40; 
for(i=0;i<n;i++) 
{ 
    for(j=0;j<n;j++) 
     arr[i][j]='O'; 
} 

srand(clock()); 
for(i=0;i<n*n;i++) 
{ 
    x = rand()%n; 
    y = rand()%n; 
    if(arr[x][y] == 'O') arr[x][y]='X'; 
} 

宣言：あなたができる

arr = (char**)malloc(n); 
for(i=0;i<n;i++) 
    arr[i] = (char*)malloc(n); 

Answer 1

変更

arr = (char**)malloc(n); 

arr = (char**)malloc(n*sizeof(char*)); 

Answer 2

： -

for(i=0;i<n;i++) 
    { 
     for(j=0;j<n;j++) 
      arr[i][j]= ((rand() % 2) == 0) ? 'O' : 'X'; 
    } 

とあなたの配列がnによってn個であることを確認してください。代わりの場所ですべてのメモリを割り当てます、これらの複数のmallocを、...

arr = (char**)malloc(n * n * sizeof(char)); 

Answer 3

for(i=0;i<n*n;i++) 
{ 
    x = rand()%n; 
    y = rand()%n; 
    if(arr[x][y] == 'O') arr[x][y]='X'; 
    ... 

n*nの？ arrはnの要素のみを持ち、arr[0...n-1]はそれぞれnの要素しか持っていません。 xまたはyが>= nの場合、配列の末尾にある要素にアクセスして、未定義の動作が発生します。この場合は、アクセス違反の原因となるため、あなたの幸運です。

これとarr = (char**)malloc(n);は、arr = (char**)malloc(n * sizeof(char*));である必要があります。

Answer 4

nが一定である、またはあなたのコンパイラは、その後、1-Dとして、それをinitalizeへのポインタの配列にキャストすることができ、C99をサポートしている場合：ちょうどランダム `に値を設定しないのはなぜ

int i; 
char (*arr)[n] = malloc(n*n); 
char *vals = (char*)arr; 

for(i=0; i<n*n; i++) 
    vals[i] = (rand()%2)?'X':'O'; 

//now, use arr[y][x] as usual