插入新节点时的AVL树Malloc函数

Question

我最近在C。。然后我尝试创建一个AVL。。这样做的第一步。就是在每个节点中添加一个额外的组件bf（平衡因子）。。我是这样做的。

    struct tnode{
        int info;
        struct tnode*left;
        struct tnode*right;
        int bf;//new field for avl tree..
                };

每次malloc在插入时为新节点分配地址。。。我将其信息部分分配给用户输入的值。。将左右指针设置为NULL。除此之外，它还将新节点的bf分配给0。。在插入第一个节点（即根节点）后，程序下次在malloc部分甚至无法为新节点分配内存。。。只要我去掉写着newnode的部分-

    int main(){
    int choice,val;
    deci:
    printf("\n1.Insert");
    printf("\n2.Display");
    printf("\n3.Search.");
    printf("\n4.Delete");
    printf("\n5.Naa..");
    scanf("%d",&choice);
    switch(choice){
            case 1:printf("\nEnter element to add:");
                    scanf("%d",&val);
                    add(val);

                    //display();
                    break;
            case 2:display();
                    break;
            case 3:printf("\nEnter element to search for:");
                    scanf("%d",&val);
                    search(val);
                    break;

            case 4:printf("\nEnter element to Delete: ");
                    scanf("%d",&val);
                    deletei(val);
                    display();
                    break;
            case 5:return 0;

            }
    goto deci;
    return 0;
    }


    void add(int val){
    struct tnode * newnode=(struct tnode * )malloc(sizeof(struct tnode *));
    newnode->info=val;
    newnode->left=newnode->right=NULL;
    newnode->bf=0;//problem making,problem solved removing this statement
    if(root==NULL){
            root=newnode;

                    }                       
    else{
        struct tnode*tree=root;
        struct tnode*ptree=NULL;
    while(tree!=NULL){
                    if(val<tree->info){
                                        ptree=tree;
                                        tree=tree->left;

                    }
                    else{
                        ptree=tree;
                        tree=tree->right;
                    }                       


        }   
    if(val<ptree->info){

                ptree->left=newnode;
                tree=ptree->left;


        }
else{
ptree->right=newnode;   
tree=ptree->right;

    }
    }


    }   

Answer 1

您的第一个主要错误是您仅为add（）中的newnode地址分配了一个空间。您应该将其转换为：

struct tnode * newnode=(struct tnode * )malloc(sizeof(struct tnode));

在这种情况下，您为所需节点分配整个空间。

附言。也请放弃“去”指令，因为使用它是一种不好的做法。

Answer 2

你的问题在这一行：

struct tnode * newnode = (struct tnode *) malloc(sizeof(struct tnode *))

从malloc的C参考

分配未初始化存储的大小字节

如果分配成功，则返回指向所分配内存块中最低（第一个）字节的指针

因此malloc分配一个内存块并返回一个指向该块开头的指针。malloc的参数是您要分配的内存块的大小。

在您的例子中，您传递的是sizeof（struct tnode*）。因此，您正在为指向struct tnode类型值的指针分配足够的内存。如果您想为struct tnode本身分配内存，请将sizeof更改为sizeof（struct tnode）。那么固定行应该如下所示

struct tnode * newnode = (struct tnode *) malloc(sizeof(struct tnode))