在堆栈中使用 strcpy() 失败

Question

这是一个关于从堆栈中推送字符串和弹出字符串的代码，字符串名称必须是固定的（不是用户输入的）。但是代码无法运行（它无法显示任何内容）。关键是因为strcpy(temp->id,val);，如果我去掉strcpy(temp->id,val);，那么代码可以正常运行（但是printf 不可读的字符）。我可以知道我的strcpy 有什么问题吗？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct LLNode
{
    char id[4];
    struct LLNode *next;
};

struct LLNode * createNode(char val[])
{
    struct LLNode *temp;
    temp=(struct LLNode *)malloc(sizeof(struct LLNode));
    strcpy(temp->id,val);
    temp-> id[4] = val;
    temp-> next = NULL;
    return (temp);
};

char push(char val[], struct LLNode *head)
{
    struct LLNode *temp;
    temp = createNode(val);
    temp->next = head->next;
    head->next = temp;
};

char pop(struct LLNode *head)
{
    struct LLNode *temp;
    char val;
    val = head->next->id;
    temp = head->next;
    head->next = head->next->next;
    free(temp);
    return(val);
};

int main()
{
    char value;
    struct LLNode *head = NULL;
    struct LLNode *tail = NULL;
    struct LLNode *curr;

    head = createNode('\0');
    tail = createNode('\0');

    char id[6][4] = {"A123","B234","C345","D456","E567","F678"};
    int n;

    for (int i=0;i<6;i++)
    {
        push(id[i],&head->id);
        printf("%s\n",&head->id);
    }


};

Answer 1

char id[4];

char id[6][4] = {"A123","B234","C345","D456","E567","F678"};

字符串是一个以 nul 结尾的非 nul 元素序列包括终止符。

你总是忘记那个小而重要的终结者。

虽然在使用字符串文字初始化具有显式长度的数组时效果很好，因为您可能会以这种方式删除隐式终止符，但请记住此后您没有字符串。

无论如何，您的节点数组没有利用该规则，而且太短了。

---

struct LLNode * createNode(char val[]);

head = createNode('\0');

不应为期望字符串的函数提供单个字符。更糟糕的是，'\0' 是一个合法的空指针文字。即使那不是你想要的。

---

temp-> id[4] = val;

分配一个指向字符数组末尾的指针是不可能的。您是否要求编译器发出警告并遵守特定标准？对于 gcc / clang，我建议至少 -Wextra -std=c99 -pedantic.