如何在C中初始化指向指针结构的指针？

Question

我有一个结构，它是一个节点，另一个结构是这些节点的列表。在列表结构中，它是一个节点数组，但它不是一个数组，而是一个指向具有大小整数的指针的指针：

typedef struct node {
    struct node *next;
    MyDef *entry;
} Node;


typedef struct list {
    Node **table;
    int size;
} List;

List *initialize(void)
{
    List *l;
    Node **n;

    if ((l = (List *)malloc(sizeof(List))) == NULL)
        return NULL;
    l->size = 11;

    /* I think this is correctly allocating the memory for this 'array' of nodes */
    if ((n = (Node **)malloc(l->size * sizeof(Node))) == NULL)
        return NULL;

    /* Now, how do I set MyDef *entry and Node *next to NULL for each of the 'array'? */

    l->table = n;

    return l;
}

How do I set MyDef *entry and Node *next to NULL for every 'array'?

Answer 1

(Node **) 是指向 Node 的 [array of] 指针，因此您分配的数组不会有任何结构成员。

您应该使用 (Node *)，然后您将拥有 Node 结构的指向数组，或者单独分配每个 Node，然后将指向它们的指针放入数组中。

标准 C 库中存在适合您情况的函数 calloc() ：它用 0 初始化分配的区域（对应于 (char/short/int/long)0、0.0 和 NULL）。

还有内存泄漏。

/* I think this is correctly allocating the memory for this 'array' of nodes */
if (... == NULL)
    return NULL;

当数组分配失败时，您不会释放 List，但会丢失指向它的指针。将其重写为：

/* I think this is correctly allocating the memory for this 'array' of nodes */
if ((n = (Node **)malloc(l->size * sizeof(Node))) == NULL) {
    free(l);
    return NULL;
}

因此，从我的角度来看，正确的代码将是：

typedef struct node {
    struct node *next;
    MyDef *entry;
} Node;


typedef struct list {
    Node *table; /* (!) single asterisk */
    int size;
} List;

List *initialize(void)
{
    List *l;
    Node **n;

    if ((l = (MList *)malloc(sizeof(List))) == NULL)
        return NULL;
    l->size = 11;

    /* I think this is correctly allocating the memory for this 'array' of nodes */
    if ((n = (Node *)calloc(l->size, sizeof(Node))) == NULL)
    {
        free(l);
        return NULL;
    }

    /* Now, how do I set MyDef *entry and Node *next to NULL for each of the 'array'? */

    l->table = n;

    return l;
}

此外，C99 允许您创建可变大小的结构，因此您可以像这样初始化结构

typedef struct list {
    int size;
    Node table[0]
} List;

并在表中分配尽可能多的节点
malloc(sizeof(List) + sizeof(Node)*n);

Answer 2

首先，在我看来，您在分配数组的代码中存在错误：它应该说 sizeof(Node*) 而不是 sizeof(Node) ，因为您想要分配一个指向 Node 的指针数组而不是 Node 对象数组。

然后您可以遍历数组列表：

for ( unsigned i = 0; i < l->size; ++i )
{
    Node* node = l->table[ i ];
    node->entry = NULL;
    node->next = NULL;
}

另一个提示：您确实应该检查初始化函数是否存在内存泄漏的可能性。