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

【问题标题】：making a pointer that stores an element of a char array创建一个存储 char 数组元素的指针
【发布时间】：2015-03-02 23:47:55
【问题描述】：

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

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

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

【问题讨论】：

第一个Don't cast the result of malloc()。然后，请张贴string的声明。
typedef'ing structs 总是一个坏主意，应该避免
在 C 中，不应强制转换来自 malloc（和家族）的返回值

标签： c arrays pointers tree linked-list

【解决方案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()函数中，先分配空间，然后像这样使用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;
}

【讨论】：

另外，它在代码的顶部是相反的。他正在分配一个指向 char 的指针（在 ptr->data = data; // WARNING 行中，因为 ptr->data 是一个 char。
@russianfool 是的，我认为 OP 将 char poitners 与 char's 混合在一起

【解决方案2】：

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

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

【讨论】：

是的，不知道为什么我们的教授让我们为 Node *TreeCreate 使用指针输入，但也许它在未来更具有相关性，我才刚刚开始搞砸这个作业。

【解决方案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;
}

【讨论】：