参照による構造体配列へのポインターへのポインターの割り当て

Question

以下のコードはコンパイルされますが、他の人には明らかな理由ですぐにクラッシュしますが、私にはわかりません。うまくいかないようです。これを修正する方法を教えてください。

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

Answer 1

演算子 [] は、単項の 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];

したがって、括弧を使用してください。

Answer 2

array_refポインターにスペースを割り当てましたが、それらが指すものには割り当てていません。

Answer 3

setName（）で以下を変更してみてください

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

to

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

これは動作します。

Answer 4

array [1]-＆gt; name が問題です。これは（* array）[1] .name でなければなりません。 2つが同等ではないことに注意してください。誤って正しいことを行う [0] を除き、同様の使用法にはすべて同じ問題があります。

array 、関数パラメーター、は配列ではなく、配列へのポインターであることを思い出してください。

Answer 5

このような関数では、次のようなコードを好みます：

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]-＆gt; name と、（ほぼ） * array_ref [2] = array [0] の両方をキャッチすることに注意してください）Pavelが指摘-そのソリューション（* array_ref）[2] = array [0]は未割り当ての person * array [1] から割り当てます-どちらも難しい追加の逆参照で注意してください。

もちろん、C ++を使用しているため、ほとんどこれを行います。これにより例外の安全性が向上します;）。