Question

我正在尝试使用malloc制作X [N] [M]数组，并且出现此错误。有人可以帮我吗？

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

int main(int argc, char argv[])
{
  int N,M;  // N=megethos rows , M=megethos columns
  int **X;  // pinakas
  int size,i,j;


  printf("Give me the rows of the table: ");
  scanf("%d",&N);
  printf("Give me the columns: ");
  scanf("%d",&M);

  size = N*sizeof(int*);
  X = malloc(size);

  size = M * sizeof(int*);
  for(i=0; i<N;i++){
    X =malloc(size);
  }

  for(i=0;i<N;i++){
    for(j=0;j<M;j++){
      printf("Eisagwgh toy %d - %d stoixeioy: ",i+1,j+1);
      scanf("%d",&X[i][j]);
    }
  }

  for(i=0;i<N;i++){
    printf("\n");
    for(j=0;j<M;j++){
      printf("%d",X[i][j]);
    }
  }
  free(X);
}

我已经尝试更改此行： size = N*sizeof(int*); 为此：size = N*sizeof(int);，但仍然出现相同的错误。

Answer 1

请参见下面的代码，我对您的代码做了很小的改动，必须看到注释。

#include <stdio.h>
#include <stdlib.h>
int main(int argc, char argv[])
{
  int N,M;  // N=megethos rows , M=megethos columns
  int **X;  // pinakas
  int size,i,j;


  printf("Give me the rows of the table: ");
  scanf("%d",&N);
  printf("Give me the columns: ");
  scanf("%d",&M);

  size = N*sizeof(int*);
  X = malloc(size);

  size = M * sizeof(int*);
  for(i=0; i<N;i++){
      //Here it intialize each row with size
    X[i] =malloc(size);
  }

  for(i=0;i<N;i++){
    for(j=0;j<M;j++){
      printf("Eisagwgh toy %d - %d stoixeioy: ",i+1,j+1);
      scanf("%d",&X[i][j]);
    }
  }

  for(i=0;i<N;i++){
    printf("\n");
    for(j=0;j<M;j++){
      printf("%d",X[i][j]);
    }
  }
  free(X);
}

Answer 2

我建议采用这种分配内存的方式：

    X = malloc(N * sizeof(int*));
    for (int i = 0; i < N; i++)
        X[i] = (int*)malloc(M * sizeof(int));

代替这种错误的分配：

  size = N*sizeof(int*);
  X = malloc(size);

  size = M * sizeof(int*);
  for(i=0; i<N;i++){
    X =malloc(size);
  }

分段错误（核心已转储）C

2 个答案: