创建一个指针来存储 char 数组的元素

Question

我一直在努力弄清楚为什么会收到以下警告：
初始化从整数生成指针而不进行转换

突出显示的警告是我在下面提到的地方。我目前使用的代码只是以链表方式创建元素树的开始。这段代码似乎工作正常，但我得到了警告的停靠点。

typedef struct Node {
        struct Node *leftChild;
        struct Node *rightChild;
        char data;
} Node;

Node *TreeCreate(int level, const char *data) {
    struct Node *ptr = (struct Node*) malloc(sizeof (Node));
    if (ptr == NULL) {
        // malloc failed
        return 0;
    }
    ptr->data = data;   // WARNING
    ptr->leftChild = NULL;
    ptr->rightChild = NULL;
    return ptr;
}
// TEST CODE IN MAIN
char list[6] = {'A', 'B', 'C','[=10=]'};

// Determines the element
const char *tree = list[0]; // WARNING
ptr = TreeCreate(1, tree);
if (ptr != NULL) {
    sprintf(string, "TreeData: %c\n", ptr->data);
    OledDrawString(string);
    OledUpdate();
}

Answer 1

你的根本错误是你将一个指针分配给一个 char 这是错误的

const char *tree = list[0]; // WARNING

这不会产生您期望的结果。

在这种情况下，* 没有取消引用指针，您正在声明一个指针并用它指向 char，然后当您尝试访问指针时，您的程序会尝试在无效的内存地址读取导致未定义的行为。

然后你在

中做相反的事情

ptr->data = data;

您应该启用编译器警告以避免这种错误。

要处理您显然想要处理的数据，首先您需要像这样重新定义结构

typedef struct Node {
    struct Node *leftChild;
    struct Node *rightChild;
    char *data;
    /*   ^ this should be a char pointer */
} Node;

然后在TreeCreate()函数中，通过先分配space然后使用memcpy()这样复制数据

Node *TreeCreate(int level, const char *data) {
    size_t       length;
    struct Node *ptr;

    ptr = malloc(sizeof (Node));
    if (ptr == NULL) {
        return NULL;
    }
    if (data != NULL)
    {
        length    = strlen(data);
        ptr->data = malloc(1 + length);
        if (ptr->data != NULL)
            memcpy(ptr->data, data, 1 + length);
    }
    else
        ptr->data = NULL;        

    ptr->leftChild  = NULL;
    ptr->rightChild = NULL;

    return ptr;
}

Answer 2

我想我明白了。以下修复了我的警告。感谢您的快速回复！

const char *tree = &list[0];
ptr->data = *data;

Answer 3

the following, a complete program, 
that cleanly compiles
and has the warnings fixed
and eliminates the clutter and unnecessary typedef statements.

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


struct Node 
{
        struct Node *leftChild;
        struct Node *rightChild;
        char data;
};

struct Node *TreeCreate(int level, const char *data) 
{
    struct Node *ptr = malloc(sizeof (struct Node));
    if (ptr == NULL) 
    {
        // malloc failed
        return NULL ;
    }

    // implied else, malloc successful

    ptr->data = *data;   // WARNING
    ptr->leftChild = NULL;
    ptr->rightChild = NULL;
    return ptr;
}

int main()
{
    struct Node *ptr = NULL;
    char  string[120] = {'[=10=]'};

    // TEST CODE IN MAIN
    char list[6] = {'A', 'B', 'C','[=10=]'};

    // Determines the element
    const char *tree = &list[0]; // WARNING

    ptr = TreeCreate(1, tree);

    if (ptr != NULL) 
    {
        sprintf(string, "TreeData: %c\n", ptr->data);
        //OledDrawString(string);
        //OledUpdate();
    }
    return 0;
}

创建一个指针来存储 char 数组的元素

making a pointer that stores an element of a char array

c

arrays

tree

pointers

linked-list