正确的内存分配

Question

我有以下结构：

typedef struct bucket {
    char *key;
    ENTRY *data;
    struct bucket *next;
} bucket;

typedef struct {
    size_t size;
    bucket **table;
} hash_table;

但我不知道如何为此分配内存。 我尝试过：

hash_table* ht = malloc(sizeof(hash_table)*101);

为了创建 101 个条目的哈希表，但它不起作用！ 谁能帮我？ 我真的很感激！

Answer 1

预先分配所有 101 个（或任意数量）存储桶是没有意义的，通常在向表中插入新数据时一次分配一个存储桶。

预分配哈希数组确实有意义，该数组具有固定大小，但这是一个存储桶指针数组，而不是存储桶数组，因此有些的答案是错误的。

你会有这样的事情，创建一个空的哈希表，带有固定大小的存储桶数组：

hash_table * hash_table_new(size_t capacity)
{
  size_t i;

  hash_table *t = malloc(sizeof *t);
  t->size = capacity;
  t->bucket = malloc(t->size * sizeof *t->bucket);
  for(i = 0; i < t->size; i++)
    t->bucket[i] = NULL;
  return t;
}

此代码：

• 分配一个 hash_table 结构来保存该表
• 用指定的容量初始化它的大小
• 分配适当的存储桶指针数组length
• 确保每个存储桶指针为 NULL（这不能用 memset() 正确完成，因为假设“所有位为零”是 NULL 在内存中的查找方式是不安全的）
• 尽可能使用 sizeof ，但不在类型上，因此没有括号
• 不转换 malloc() 的返回值，因为这在 C 中从来都不是一个好主意
• 不检查 malloc() 的返回值，当然，您应该在实际代码中执行此

操作需要第二个函数来执行实际的哈希插入，然后需要分配一个新的存储桶，根据键计算哈希值，在哈希表的数组中选择正确的位置，然后在那里插入新条目。

Answer 2

不完全的。 假设这是 C，您可能想要创建一个函数：

 hash_table* init_table(size_t size) {
     size_t i;
     hash_table* ht = (hash_table*)malloc(sizeof(hash_table));
     if (ht == NULL) return NULL;
     ht->size = size;
     ht->table = (bucket**)malloc(sizeof(bucket*)*size);
     if (ht->table == NULL) {
         free(ht);
         return NULL;
     }
     for (i = 0; i < size; ++i) {
         ht->table[i] = NULL;
     }
     return ht;
 }

您可能需要该结构中的一些其他字段。

如果你想变得狡猾，并且从不重新分配存储桶，你可以这样做：

 hash_table* init_table(size_t size) {
     hash_table* ht = (hash_table*)malloc(sizeof(hash_table)+sizeof(bucket)*size);
     if (ht == NULL) return NULL;
     ht->size = size;
     ht->table = (bucket**)(ht+1);
     for (i = 0; i < size; ++i) {
         ht->table[i] = NULL;
     }
     return ht;
 }

编辑：我将我的存储桶*表固定为存储桶**

编辑2：我已经摆脱了memsets并添加了malloc的错误检查。

Answer 3

hash_table 始终只有 sizeof(hash_table) 字节大。 table 元素是一个指向 bucket 元素的指针数组的指针。 所以你需要这样的东西：

hash_table* ht = malloc(sizeof(hash_table));
ht->size = 101;
ht->table = malloc(sizeof(bucket*)*ht->size);

但我怀疑可能有一些随之而来的初始化方法，然后你可以做这样的事情：

hash_table* ht = alloc_hash_table(101);

不管怎样，我对 C 有点生疏，所以把这个与谷物一起考虑盐。

Answer 4

你的 typedef 有一些问题。 假设您使用 MSVC。

声明此处类型的简单方法如下：

此 typedef 包括 _type {} 类型，*ptype； 格式同时声明类型和指向自定义类型的指针。 如果您在 hash_table 中看到，您可以使用 pbucket *table，它消除了代码中的额外 ***，并且可以在进行动态分配时提供帮助（帮助您清楚分配的内容等。 ）。 你原来的 typedef，如果你看起来有 typedef struct bucket {} bucket;，你至少需要修改你指定 typedef 时的两个“bucket”名称之一。

如果您使用 C++ 构建设置，您还需要进行强制转换，如果使用纯 C，则可能不需要强制转换，因此您的 malloc 行将是（我进行了以下 typedef 更改）；

hash_table* ht = (phash_table) malloc(sizeof(hash_table)*101);

不管怎样，这个片段应该适合你；

typedef struct _bucket {    
    char *key;    
    void *data;    
    _bucket *next;
} bucket, *pbucket;

typedef struct _hash_table {    
    size_t size;    
    pbucket *table;
}hash_table, *phash_table;