Question

我编写了一个将整数值插入二叉搜索树的程序。它似乎工作正常，但当我修改相同接受字符数组而不是整数时，我得到意想不到的结果。这是我的完整代码：

struct Node{
char data[50];
struct Node* right;
struct Node* left;
};

typedef struct Node* NODE;

NODE createNode(char data[]){
    NODE newNode = (NODE) malloc (sizeof(struct Node));

if(!newNode){
    cout<<"Not enough memory"<<endl;
    exit(-1);
}
newNode->left = NULL;
newNode->right = NULL;
strcpy(newNode->data,data);
return (newNode);
}

void insertNode(NODE* head,char data[]){

    NODE newNode = createNode(data);
    NODE hold_the_head = *head;
    if(*head == NULL){
        *head = newNode;
        (*head)->right = NULL;
        (*head)->left = NULL;
        return;
    }

    while(1){
        if((newNode->data>(*head)->data)&&((*head)->right==       NULL)){
            (*head)->right = newNode;
            *head = hold_the_head;
            return;
        }
        else if( newNode->data > (*head)->data ){
            (*head) = (*head)->right;
        }

        else if( (newNode->data < (*head)->data) && ( (*head)->left ==   NULL ) ){
            (*head)->left = newNode;
            *head = hold_the_head;
            return;
        }
        else if( newNode->data < (*head)->data ){
            (*head) = (*head)->left;
        }
    }
}

void inOrderTraversal(NODE node){

    if(node == NULL)
       return;
    inOrderTraversal(node->left);
    cout<<node->data<<"\t";
    inOrderTraversal(node->right);
}

int main(){

    NODE head = NULL;
    insertNode(&head,"karan");
    insertNode(&head,"sameer");
    insertNode(&head,"palak");
    insertNode(&head,"jagdish");
    insertNode(&head,"naman");
    insertNode(&head,"umang");
    insertNode(&head,"chandu");

    inOrderTraversal(head);
    cout<<endl;
    return 0;
}

输出：

karan sameer palak jagdish naman umang chandu

预期：

chandu jagdish karan naman palak sameer umang

之前已经问过这样的问题，但是有一些编译错误。我的代码没有抛出任何错误，但似乎存在一些逻辑缺陷！

Answer 1

除了rachitmanit的回答，我觉得你用C语言写作，而不是C ++。

char data[50];

如果您使用C ++编写，我建议使用std::string。它可以方便地与==，<等进行比较。

NODE newNode = (NODE) malloc (sizeof(struct Node));

旧C＆＃39; malloc仅分配内存，并且不构造对象（即不调用构造函数）。它应该是：NODE newNode = new Node;

(*head)->right = NULL;
(*head)->left = NULL;
/*etc...*/

NULL通常是0，它是一个整数，而不是指针。我绝对建议使用nullptr。

void insertNode(NODE* head,char data[]){

通常，指针参数可能是nullptr，您应该检查它是否是void insertNode(NODE &head, std::string data){。我建议使用参考，您不必：cout<<endl;

std::cout<<std::endl

应为struct Node。

不要忘记释放记忆。您的程序分配内存并且不会释放它，这将导致内存泄漏。 struct Node { std::string data; Node *right; Node *left; ~Node() { delete right; delete left; } }; /* ... */ int main() { /* ... */ delete head; return 0; }应该在销毁时释放内存：

href

Answer 2

如果是整数，您的“数据”实际上是一个值。虽然这里“node-＆gt; data”是第一个data []数组块的地址。记住node-＆gt; data [0]是一个值。您在这里比较地址而不是实际的“价值”。

此外，您必须遵循：How to compare string in an character array?

这应该有所帮助。

c ++

2 个答案: