c - C でのこの宣言の何が問題になっていますか?

Question

構造を定義しました。

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

これは有効であるべきではありませんか？

score 3 · Accepted Answer

問題は、文字ポインタにメモリを割り当てておらず、keyそれに文字列をコピーしようとしていることです。未定義の動作が発生します。
keyは単なる初期化されていない文字ポインターであり、ランダムなアドレスを指すだけです。任意のポインターを使用できるようにするには、次の 2 つの条件があります。

あなたは両方に違反しています。

score 1 · Accepted Answer

文字列を格納するためのスペースを割り当てる必要があります。現在、生のポインターしかありません。このようなもの：

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

理想的には、これをstruct

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

2 に答える 2