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','\0'};

// 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分配一个错误的poitner

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

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

在这种情况下，*不会取消引用pointe，你正在使用它声明一个poitner和pointeing到char，然后当你尝试访问指针时，你的程序试图读取导致未定义行为的无效内存地址。

然后你在

做相反的事情

ptr->data = data;

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

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

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

然后在TreeCreate()函数中，首先分配空格，然后使用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] = {'\0'};

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

    // 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数组元素的指针

3 个答案: