3

4次元配列を動的に割り当てるために、この単純なコードを作成しました。

#include <stdlib.h>
#include <stdio.h>

int**** alloc() {

    int i,j,k;
    int ****matrix;
    int x,y,z,n_pairs;

    x= 62;
    y= 45;
    z= 28;
    n_pairs = 4;

    matrix = (int ****) malloc(x*sizeof(int***));

    for (i=0; i<x; i++) {
        matrix[i] = (int ***) malloc(y*sizeof(int**));

        if(matrix[i]==NULL)
            return NULL;

        for (j=0; j<y; j++) {
            matrix[i][j] = (int **) malloc(z*sizeof(int*));

            if (matrix[i][j] == NULL)
                return NULL;

            for (k=0; k<n_pairs; k++) {
                matrix[i][j][k] = (int *)calloc(n_pairs,sizeof(int));

                if (matrix[i][j][k] == NULL)
                    return NULL;
            }
        }
    }

    return matrix;
}


void freeMatrix(int ****m) {

    int i,j,k;
    int x,y,z;

    x= 62;
    y= 45;
    z= 28;


    for(i=0; i<x; i++) {
        for(j=0; j<y; j++) {
            for(k=0; k<z; k++) 
                free(m[i][j][k]);

            free(m[i][j]);
        }

        free(m[i]);
    }

    free(m);
}

int main() {

    int i,j,k,h;
    int ****m = NULL;

    m = alloc();

    for(i=0;i<62;i++)
        for(j=0;j<45;j++)
            for(k=0;k<28;k++)
                for(h=0;h<4;h++)
                    printf("%d\t",m[i][j][k][h]);

    system("pause");

    return 0;
}

問題は、このコードAccess Violationを実行しようとすると、が発生することです。多次元配列を割り当て/解放する正しい方法ではありませんか？はいの場合、問題は何ですか？

4

2 に答える 2

4

ここに 1 つの問題があります。

        for (k=0; k<n_pairs; k++) { //<----- This should read `k<z'
            matrix[i][j][k] = (int *)calloc(n_pairs,sizeof(int));

おそらくz、 ではなく にループするつもりでしたn_pairs。

于 2012-05-28T16:00:04.860 に答える
0

マトリックスを作成する場合、正しい方法は、必要なすべてのメモリを割り当ててから、それをダイビングすることです。

2.dim の場合

#include <stdlib.h>
#include <stdio.h>

int main(void)
{
  int i;
  int ** matrix;
  int rows = 10;
  int colums =10;


  matrix = (int**)malloc(rows*sizeof(int*));
  matrix[0] = (int*)malloc(rows*colums*sizeof(int));

  for(i =1; i<rows;i++)
    matrix[i] = &matrix[i-1][colums];

  free(matrix[0]);
  free(matrix);
  return 0;
}

これは、行列用の連続したメモリ領域を持つためです。これはより速くすることができます

于 2012-05-28T16:17:22.563 に答える