malloc 通过引用指向结构数组的指针答案

【问题标题】：Malloc a pointer to a pointer to a structure array by referencemalloc 通过引用指向结构数组的指针
【发布时间】：2010-12-21 11:55:26
【问题描述】：

下面的代码可以编译，但由于其他人明显的原因而立即崩溃，但对我来说不是。我似乎无法做到这一点，谁能告诉我如何解决这个问题。

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

【问题讨论】：

在代码有 bug 的情况下，特别是当 bug 是实际崩溃并且每次都发生时，您通常应该做的是尝试将其缩小到最少的代码量仍然让它发生。这将使其他人在查看代码时更容易找到错误，更重要的是，您更容易理解导致崩溃的确切原因。
@Edan，我不会说在这种情况下这是一个问题。这里代码不多，问题还算清楚。
+1 伊丹。缩小范围以删除 array_ref 可以澄清问题，并且肯定会帮助 ZPS 理解它。

标签： c arrays data-structures pointers malloc

【解决方案1】：

在这样的函数中，我更喜欢这样的代码：

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

请注意，这会捕获 Roger Pate 所指出的 array[1]->name 和 Pavel （几乎）所指出的 *array_ref[2] = array[0] - 其解决方案 (*array_ref)[2] = array[0] 从未分配的 person* 分配array[1] - 通过额外的取消引用很难注意到这两者。

当然，我这样做主要是因为我使用 C++，这增加了异常安全性；）。

【讨论】：

【解决方案2】：

运算符 [] 的优先级高于一元 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];

所以只需使用括号。

【讨论】：

记住array[1] 没有分配:)。

【解决方案3】：

array[1]->name 是你的问题。这应该是(*array)[1].name。注意两者是不等价的。所有类似的用法都有相同的问题，除了[0]，它不小心做了正确的事情。

记住array，函数参数，不是你的数组，它是一个指向你的数组的指针。

【讨论】：

【解决方案4】：

尝试在 setName() 中更改以下内容

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

到

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

这行得通。

【讨论】：

【解决方案5】：

您为 array_ref 指针分配了空间，但没有为它们指向的内容分配空间。

【讨论】：

他将它们指向已分配的对象，这不是问题。