Бесплатный 2d массив.Статический указатель - PullRequest
Новая
       29

Бесплатный 2d массив.Статический указатель

1 голос
/ 15 января 2012

У меня есть статический указатель 

   static int **2dArr;

, а затем я выделяю память для 2d массива.Как я могу освободить эту память и заменить ее другим массивом? 

   //////////////////////////////////////////////////////////////////////////
  void func(int **arr) {
    int i,j,k,tmp;
    int **destMatrix = NULL;
destMatrix = (int**) malloc(N * sizeof(int *));

if ((destMatrix == NULL)) {
    fprintf(stderr, "out of memory\n");
    exit(2);
}

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

    if (destMatrix[i] == NULL) {
        fprintf(stderr, "out of memory\n");
        exit(2);
    }
}



for(i = 0; i < N; ++i) {
    free(arr[i]);
}
free(arr);
arr = NULL;
arr = destMatrix;

}
  //////////////////////////////////////////////////////////////////////////
  int main() {
   2dArr = (int**) malloc(N * sizeof(int *));
   if (2dArr== NULL) {
    fprintf(stderr, "out of memory\n");
    exit(2);
     }


for (i = 0; i < N; i++) {
    2dArr[i] = (int*) malloc(N * sizeof(int));

    if (2dArr[i] == NULL) {
            fprintf(stderr, "out of memory\n");
            exit(2);
    }
}
    func(2dArr);
    // try to print new array, access violation
 }

Я освобождаю память и пытаюсь заменить указатель.Но тогда у меня есть нарушение прав доступа.Как я могу сделать это правильно?

Ответы [ 3 ]

3 голосов
/ 15 января 2012

Пара вещей:

  • Вам необходимо передать аргумент по ссылке в func: void func(int ***arr) {
  • Затем изменить способ освобождения вещей: free((*arr)[i]); и free(*arr);
  • и, наконец, измените способ переназначения arr: *arr = destMatrix;
  • также не называйте переменные, начинающиеся с цифры - это не работает.
  • (такжеу вас было ++i вместо i++ в цикле for в конце func) это на самом деле не имеет значения - см. комментарий ниже

Я отладил эти проблемы с помощью Valgrind , посмотрите.

Рабочий код ниже: 

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

#define N 10 
//////////////////////////////////////////////////////////////////////////
void func(int ***arr) {
  int **destMatrix = NULL;
  destMatrix = malloc(N * sizeof(int *));

  if ((destMatrix == NULL)) {
    fprintf(stderr, "out of memory\n");
    exit(2);
  }

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

    if (destMatrix[i] == NULL) {
      fprintf(stderr, "out of memory\n");
      exit(2);
    }
  }

  for(i = 0; i < N; i++) {
    free((*arr)[i]);
  }
  free(*arr);
  *arr = destMatrix;

}
//////////////////////////////////////////////////////////////////////////
int main() {
  int **tdArr = malloc(N * sizeof(int *));
  if (tdArr== NULL) {
    fprintf(stderr, "out of memory\n");
    exit(2);
  }

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

    if (tdArr[i] == NULL) {
      fprintf(stderr, "out of memory\n");
      exit(2);
    }
  }

  func(&tdArr);
  // try to print new array, does not get access violation

  //assign numbers 0-99
  int j;
  for (i=0; i<N; i++){
    for (j=0; j<N; j++){
      tdArr[i][j] = i*10+j;
    }
  }
  //print numbers
  for (i=0; i<N; i++){
    for (j=0; j<N; j++){
      printf("%d\n", tdArr[i][j]);
    }
  }

  //free internal arrays
  for (i=0; i<N; i++){
    free(tdArr[i]);
  }
  //free outer array
  free(tdArr);

  return 0;
}
3 голосов
/ 15 января 2012

Я не совсем уверен, что вы пытаетесь сделать здесь, но имейте в виду, что утверждение 

arr = destMatrix;

в func устанавливает только локальную копию вашего указателя на destMatrix. Вернувшись в main (и глобально), 2dArr (или как вы его на самом деле называли, поскольку, как заметил @nm, это неверный идентификатор), все еще имеет свое первоначальное значение malloc ed, даже если вы успешно освободили все указатели в func. Если вы попытаетесь использовать или освободить этот указатель (или старые указатели, на которые он все еще [вероятно] указывает)) в другом месте, вы можете получить различные ошибки. (В этом случае, будьте рады, что вы получаете ошибки - вместо этого вы могли бы молча уничтожать память, от которой зависит какой-то другой процесс.) Если вы намереваетесь изменить 2dArr в func, вам нужно объявить 

void func(int ***arr);

и позвоните по номеру 

func(&2dArr);

, чтобы вы могли обращаться к исходным значениям и изменять их с помощью разыменования, например, 

free((*arr)[i]);
1 голос
/ 15 января 2012

В func вы освобождаете память для статической переменной 2darr и похоже, что вы намереваетесь установить ее на destMatrix, но вы только устанавливаете копию переменной входного указателя на func.

Если вы измените на void func(int **& arr), тогда вы можете установить 2darr, это будет передавать int ** по ссылке вместо значения.

...