Malloc un puntero a un puntero a una matriz de estructura por referencia
-
08-07-2019 - |
Pregunta
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;
}
Solución
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.
Otros consejos
Usted asignó espacio para los punteros array_ref, pero no para lo que apuntan.
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.
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.
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;).