Malloc указатель на указатель на массив структуры по ссылке - PullRequest
Новая
       13

Malloc указатель на указатель на массив структуры по ссылке

4 голосов
/ 30 ноября 2009

Код ниже компилируется, но сразу вылетает по причинам, очевидным для других, но не для меня. Кажется, я не могу понять это правильно, может кто-нибудь сказать мне, как это исправить. 

*array_ref[2] = array[0];
*array_ref[3] = array[1];

Он падает на эту часть каждый раз. 

typedef struct test {
    char *name;
    char *last_name;
} person;



int setName(person ** array, person ***array_ref) {

    *array = malloc (5 * sizeof(person));
    *array_ref= malloc(5 * sizeof(person*));

   array[0]->name = strdup("Bob");
   array[1]->name = strdup("Joseph");
   array[0]->last_name = strdup("Robert");
   array[1]->last_name = strdup("Clark");


*array_ref[2] = array[0];
*array_ref[3] = array[1];


    return 1;
}



int main()
{
    person *array;
    person **array_r;

   setName(&array,&array_r);

    printf("First name is %s %s\n", array[0].name, array[0].last_name);
    printf("Second name is %s %s\n", array_r[3]->name, array_r[3]->last_name);

     while(1) {}
    return 0;
}

Ответы [ 5 ]

6 голосов
/ 30 ноября 2009

Оператор [] имеет более высокий приоритет, чем унарный operator*. Следовательно, это: 

*array_ref[2] = array[0];
*array_ref[3] = array[1];

фактически означает: 

*(array_ref[2]) = array[0];
*(array_ref[3]) = array[1];

Типы здесь верны, поэтому он компилируется. Но из вашего кода ясно, что ваше намерение было: 

(*array_ref)[2] = array[0];
(*array_ref)[3] = array[1];

Так что просто используйте скобки.

1 голос
/ 30 ноября 2009

В таких функциях я предпочитаю код типа: 

int setName(person ** out_array, person ***out_array_ref) {
    person* array = malloc(5 * sizeof(person));
    person** array_ref = malloc(5 * sizeof(person*));
    array[0].name = strdup("Bob");
    array[1].name = strdup("Joseph");
    array[0].last_name = strdup("Robert");
    array[1].last_name = strdup("Clark");
    // I'm guessing this was your intent for array_ref, here:
    array_ref[2] = &array[0];
    array_ref[3] = &array[1];

    *out_array = out_array;
    *out_array_ref = array_ref;
    return 1;
}

Обратите внимание, что это ловит как array[1]->name как отмечено Роджером Пэйтом, так и *array_ref[2] = array[0] как (почти) отмечено Павлом - чье решение (* array_ref) [2] = array [0] присваивает из нераспределенного person* array[1] - и то, и другое трудно заметить с дополнительным разыменованием.

Конечно, я в основном это делаю, потому что я использую C ++, и это повышает безопасность исключений;).

1 голос
/ 30 ноября 2009

Попробуйте изменить следующее в setName () 

 *array_ref[2] = array[0];
 *array_ref[3] = array[1];

до 

*(*array_ref+2) = array[0];
*(*array_ref+3) = array[1];

Это работает.

1 голос
/ 30 ноября 2009

array[1]->name ваша проблема. Это должно быть (*array)[1].name. Обратите внимание, что эти два не эквивалентны. Все аналогичные применения имеют ту же проблему, за исключением [0], который случайно делает правильные вещи.

Помните, что array, параметр функции, не ваш массив, это указатель на ваш массив.

1 голос
/ 30 ноября 2009

Вы выделили пространство для указателей array_ref, но не для того, на что они указывают.

...