Malloc un puntero a un puntero a una matriz de estructura por referencia

Question

El siguiente código se compila, pero se bloquea inmediatamente por razones obvias para otros, pero no para mí. Parece que no puedo hacerlo bien, ¿alguien puede decirme cómo solucionarlo?

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

Se bloquea en esa 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

El

operador [] tiene mayor prioridad que el operador * unario. Por lo tanto, esto:

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

en realidad significa:

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

Los tipos son correctos aquí, por eso se compila. Pero de su código está claro que su intención en realidad era:

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

Así que solo usa paréntesis.

Answer 2

Usted asignó espacio para los punteros array_ref, pero no para lo que apuntan.

Answer 3

Intenta cambiar lo siguiente en setName ()

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

a

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

Esto funciona.

Answer 4

array [1] - > name es su problema. Esto debería ser (* array) [1] .name . Observe cómo los dos no son equivalentes. Todos los usos similares tienen el mismo problema, excepto [0] , que accidentalmente hace lo correcto.

Recuerde que array , el parámetro de función, no es su matriz, es un puntero a su matriz.

Answer 5

En funciones como esta, prefiero códigos 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;
}

Tenga en cuenta que esto atrapa tanto array [1] - > name como lo señala Roger Pate, y * array_ref [2] = array [0] como (casi ) señalado por Pavel, cuya solución (* array_ref) [2] = array [0] asigna de una person * array [1] no asignada, ambas difíciles de identificar aviso con la desreferencia adicional.

Por supuesto, principalmente hago esto porque uso C ++, y esto aumenta la seguridad de excepción;).