C语言多层结构体内存分配报错：无法访问users成员

咱们一步步梳理你的问题——你现在主要卡两个核心点：内存分配的大小计算错误，以及对嵌套指针结构体的访问逻辑理解偏差，先逐个解决：

1. 先解释你遇到的报错 #

'request for member users in something not a structure or a union' 这个错误的本质是：
HT->buckets 的类型是 bucket**，它是一个指向bucket指针的数组，不是单个bucket结构体实例。你不能直接用 HT->buckets->users 访问成员，必须通过索引定位到具体的某个bucket（比如 HT->buckets[i]），再访问它的users属性。

2. 内存分配的错误修正 #

你的代码里有3处关键的malloc尺寸错误：

• 错误1：分配HT->buckets时的尺寸
  你写的 HT->buckets = malloc(HTSize * sizeof(HT->buckets)); 是错的——sizeof(HT->buckets) 得到的是bucket**指针本身的大小（通常4/8字节），而我们需要的是数组中每个元素（bucket*）的大小。正确写法：

  // 更安全的写法：用指针解引用来推导类型，避免手动写类型出错
  HT->buckets = malloc(HTSize * sizeof(*HT->buckets));
  

• 错误2：提前分配users的逻辑
  你在初始化HT->buckets数组后立刻尝试分配HT->buckets->users，这时候数组里的每个bucket*还都是空指针（后面的循环才会给它们分配内存），这一步不仅语法错，逻辑上也是非法的。users的分配必须放在单个bucket实例被创建之后。

• 错误3：分配users数组的尺寸
  给bucket的users分配内存时，malloc(buckSize * sizeof(HT->buckets->users)) 同样错误——HT->buckets->users是user**类型，我们需要的是每个元素user*的大小。正确写法：

  HT->buckets[i]->users = malloc(buckSize * sizeof(*HT->buckets[i]->users));
  

3. 循环里的逻辑错误修正 #

• malloc后的判断顺序颠倒：你先给HT->buckets[i]->next = NULL，再判断HT->buckets[i] == NULL——如果malloc失败返回NULL，给空指针的next赋值会直接崩溃。应该先判断分配是否成功，再初始化成员。
• 用户分配后的判断对象错误：你写的if(HT->buckets[i]==NULL)，但这里要检查的是HT->buckets[i]->users[j]是否分配成功，不是HT->buckets[i]。

修正后的完整代码 #

// Creation and Initialization of HashTable
hashtable* createInit(unsigned int HTSize,unsigned int buckSize){
    hashtable* HT = malloc(sizeof(hashtable)); // C标准中不需要强制转换malloc返回值
    if(HT==NULL) {
        printf("Error in hashtable memory allocation... \n");
        return NULL;
    }
    HT->hashSize=HTSize;
    HT->bucketSize=buckSize;

    // 分配bucket指针数组
    HT->buckets = malloc(HTSize * sizeof(*HT->buckets));
    if(HT->buckets==NULL) {
        printf("Error in Buckets memory allocation... \n");
        free(HT); // 避免内存泄漏
        return NULL;
    }

    for(int i=0; i <HTSize; i++){
        // 分配单个bucket实例
        HT->buckets[i] = malloc(sizeof(bucket));
        if(HT->buckets[i]==NULL) {
            printf("Error in Bucket %d memory allocation... \n",i);
            // 释放已分配的资源，避免泄漏
            free(HT->buckets);
            free(HT);
            return NULL;
        }
        HT->buckets[i]->next = NULL;

        // 分配当前bucket的user指针数组
        HT->buckets[i]->users = malloc(buckSize * sizeof(*HT->buckets[i]->users));
        if(HT->buckets[i]->users==NULL) {
            printf("Error in Users array for Bucket %d memory allocation... \n",i);
            free(HT->buckets[i]);
            free(HT->buckets);
            free(HT);
            return NULL;
        }

        for(int j=0; j <buckSize; j++){
            // 分配单个user实例
            HT->buckets[i]->users[j] = malloc(sizeof(user));
            if(HT->buckets[i]->users[j]==NULL) {
                printf("Error in User %d of Bucket %d memory allocation... \n",j,i);
                // 释放当前bucket已分配的所有user
                for(int k=0; k<j; k++){
                    free(HT->buckets[i]->users[k]);
                }
                free(HT->buckets[i]->users);
                free(HT->buckets[i]);
                free(HT->buckets);
                free(HT);
                return NULL;
            }
            // 初始化user成员，避免野指针
            HT->buckets[i]->users[j]->userID = NULL;
            HT->buckets[i]->users[j]->wallet = 0;
            HT->buckets[i]->users[j]->userBC = NULL;
            HT->buckets[i]->users[j]->userSendList = NULL;
            HT->buckets[i]->users[j]->userReceiveList = NULL;
        }
    }
    return HT;
}

多层结构体操作的实用技巧 #

• 从顶到底逐层初始化：先创建顶层hashtable，再创建buckets数组，接着逐个创建bucket，最后创建users数组和user实例，绝对不要跳层操作。
• 用sizeof(*ptr)代替手动写类型：这样如果结构体类型后续修改，你不需要改动malloc的大小计算，大幅减少出错概率。
• 严格处理内存泄漏：一旦某个malloc失败，必须释放之前已经成功分配的所有内存（上面的代码加了基础的释放逻辑，复杂项目可以封装成专门的清理函数）。
• 时刻理清指针类型：写代码时多提醒自己：
  • hashtable* → 指向hashtable的指针
  • hashtable->buckets → bucket**（bucket指针数组）
  • hashtable->buckets[i] → bucket*（单个bucket的指针）
  • hashtable->buckets[i]->users → user**（user指针数组）
  • hashtable->buckets[i]->users[j] → user*（单个user的指针）

内容的提问来源于stack exchange，提问作者Κωνσταντίνος Δημουλιάς