Question

我最近在C中成功创建了一个bst。创建一个AVL。第一步。要添加一个额外的每个节点中都有bf（balance factor）组件。我按如下操作。

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

每次插入时，malloc都会向新节点分配一个地址...我将其信息部分分配给用户输入的值..将左右指针设置为NULL。除此之外，它还分配bf 将newnode的值设置为0。在插入第一个节点（即根节点）后，程序在malloc部分失败，下次甚至为newnode分配了一个内存...一旦我删除了说newnode-> bf = 0的部分，问题就消失了。为什么会发生这种情况。下面是我的主要函数和add（）函数，它将整数作为输入并将其插入到全局声明为根节点为“ root”的树中。

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

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

PS。也请退出“ goto”说明，因为使用它是一种不好的做法。

Answer 2

您的问题在这一行：

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

从C reference到malloc

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

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

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

您要传递的是sizeof(struct tnode *)。因此，您正在为指针指向struct tnode类型的值分配足够的内存。如果要为struct tnode本身分配内存，请将sizeof更改为sizeof(struct tnode)。然后固定线应该像

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

插入新节点时，AVL TREE Malloc函数

2 个答案: