C 中的这个声明有什么问题？答案

【问题标题】：What is wrong with this declaration in C?C 中的这个声明有什么问题？
【发布时间】：2013-10-26 06:19:01
【问题描述】：

我定义了一个结构。

typedef struct bucket {
    char *key;
    void *value;
    struct bucket *next;
} Bucket;

此函数在名为“htable.h”的文件中定义。

int put(Table *table, const char *key, void *value) {

    Bucket insertBucket;
    strcpy(insertBucket.key, key);

    return 0;
}

这不应该有效吗？

【问题讨论】：

标签： c hashtable

【解决方案1】：

问题是您没有为字符指针key 分配任何内存，而您正试图将字符串复制到它。它给你未定义的行为。
key 只是一个未初始化的字符指针，它只是指向一个随机地址。要能够使用任何指针，有两个条件：

你都违反了。

【讨论】：

-1，抱歉吹毛求疵，但字符指针不够大，无法容纳一个字符。您可以进行邪恶的强制转换并存储多达四个，但这是糟糕的代码。
@self.: 很公平，已修改。
它是一个未初始化的指针，所以它没有指向任何有用的地方。
@JonathanLeffler - 你是说在strcpy(insertBucket.key, key); 行之前，做insertBucket.key=malloc(strlen(key)+1); 这样的事情不是很好吗？如果不是，为什么不呢？（还假设包括free() 在内的其他代码也将在同一块中完成）
@ryyker：不，他不是那个意思，他只是说明了指针使用的基本规则。如何以及在哪里（从堆栈或从堆）为指针分配内存取决于使用情况.如果可以，最好避免动态内存分配，并且仅在需要时使用它。

【解决方案2】：

您需要分配一些空间来存储您的字符串。现在你只有一个原始指针。像这样的：

char dest[80]; // arbitrary amount of space for example
strcpy(dest, key);

理想情况下，您希望在 struct 中执行此操作

typedef struct bucket {
  char key[80];
  // .. other members ..
} Bucket;

【讨论】：