MALLOC um apontador para um apontador para uma matriz de estrutura pela referência

Question

O código a seguir compila, mas imediatamente cai por razões óbvias para os outros, mas não para mim. Eu não consigo fazer a coisa certa, alguém pode me dizer como corrigir isso.

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

Ele trava em que parte cada vez.

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;
}

Answer 1

Operador [] tem precedência maior do que operator* unário. Assim, este:

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

realmente significa:

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

Tipos estão corretas aqui, é por isso que ele compila. Mas do seu código é claro que a sua intenção realmente era:

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

Então, basta usar parênteses.

Answer 2

Você alocado espaço para os ponteiros array_ref, mas não para o que eles apontam.

Answer 3

Tente alterar o número de seguidores na setName ()

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

para

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

Isso funciona.

Answer 4

array[1]->name é o seu problema. Este deve ser (*array)[1].name. Observe como os dois não são equivalentes. Todos os usos semelhantes têm o mesmo problema, exceto para [0], que faz acidentalmente a coisa certa.

Lembre-se que array, o parâmetro de função, não sua matriz, é um ponteiro para a matriz.

Answer 5

Em funções como este eu prefiro código como:

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;
}

Note-se que esta capturas tanto array[1]->name como observado por Roger Pate, e *array_ref[2] = array[0] como (quase) observado por Pavel - cuja solução (* array_ref) [2] = array [0] cessionários de um person* array[1] não atribuído - ambos os quais são difícil de aviso com o dereference extra.

É claro, eu principalmente fazer isso porque eu uso C ++, e essa segurança aumenta exceção;).