Malloc un puntatore a un puntatore a un array di strutture per riferimento
-
08-07-2019 - |
Domanda
Il codice seguente viene compilato, ma si blocca immediatamente per motivi ovvi agli altri, ma non a me. Non riesco a farlo bene, qualcuno può dirmi come risolvere questo problema.
*array_ref[2] = array[0];
*array_ref[3] = array[1];
Si blocca su quella parte ogni volta.
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;
}
Soluzione
[]
ha una precedenza maggiore rispetto all'operatore unario *
. Quindi, questo:
*array_ref[2] = array[0];
*array_ref[3] = array[1];
in realtà significa:
*(array_ref[2]) = array[0];
*(array_ref[3]) = array[1];
I tipi sono corretti qui, motivo per cui si compila. Ma dal tuo codice è chiaro che il tuo intento era in realtà:
(*array_ref)[2] = array[0];
(*array_ref)[3] = array[1];
Quindi usa solo le parentesi.
Altri suggerimenti
Hai allocato spazio per i puntatori array_ref, ma non per ciò a cui puntano.
Prova a modificare quanto segue in setName ()
*array_ref[2] = array[0];
*array_ref[3] = array[1];
a
*(*array_ref+2) = array[0];
*(*array_ref+3) = array[1];
Funziona.
array [1] - > name
è il tuo problema. Dovrebbe essere (* array) [1] .name
. Nota come i due non sono equivalenti. Tutti gli usi simili hanno lo stesso problema, ad eccezione di [0]
, che fa accidentalmente la cosa giusta.
Ricorda che array
, il parametro della funzione, non è il tuo array, è un puntatore al tuo array.
In funzioni come questa preferisco il codice come:
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;
}
Nota che questo cattura sia array [1] - > name
come notato da Roger Pate, sia * array_ref [2] = array [0]
come (quasi ) notato da Pavel - la cui soluzione (* array_ref) [2] = array [0] assegna da una persona *
array [1]
non allocata - entrambi difficili da notare con la dereference extra.
Certo, lo faccio principalmente perché uso C ++ e questo aumenta la sicurezza delle eccezioni;).