如何在C中释放二叉树中分配的内存

Question

我的 C 程序中的某个函数出现问题。该计划的目的是：

• 将二进制文件中的整数读取到数组中
• 使用二叉树对这些数字进行排序
• 做一些其他的事情
• 释放您使用 malloc() 分配的所有内存；

除了能够释放我的二叉树之外，我一切正常。这是我的树和节点（又名叶子）的结构。

typedef struct Node *NodeP;
typedef struct Tree *TreeP;

// Definition of a tree
    struct Tree
    {
        NodeP root;
    }Tree;

    // Definition of a node
    struct Node
    {
        int data;
        NodeP L, R;
    }Node;

在我的程序中，我使用 malloc 为我的树和每个单独的节点分配内存。所以我调用了一个函数来释放树及其所有节点。

/*
 * Frees the memory allocated for
 * each node
 */
void freeNode(NodeP p)
{
    if(p == NULL) return;
    freeNode(p->L);
    freeNode(p->R);
    free(p);
}

/*
 * Frees the memory allocated
 * for the tree
 */
TreeP freeTree(TreeP tree)
{
    if(tree->root == NULL)
        free(tree);
    else
    {
        freeNode(tree->root);
        free(tree);
    }

    return NULL;
}

当我运行这个程序时，我的调试器给了我这个错误。

EXC_BAD_ACCESS (code=EXC_I386_GPFLT)

我已经尝试过每次递归迭代，但我找不到它为什么给我一个错误。我在想它是在边缘情况下从树的边缘掉下来的？我不知道。非常感谢任何帮助！

编辑：

这里是下载完整程序的链接。我包含了一个自述文件和我正在使用的二进制文件。一个只有 10 个整数长度，另一个是 20000 长度。感谢您迄今为止的帮助！

https://copy.com/902v0bMv8DtIMUrc

Answer 1

这里有点混乱：

// Definition of a tree
struct Tree
{
    NodeP root;
}Tree;

// Definition of a node
struct Node
{
    int data;
    NodeP L, R;
}Node;

上面的定义定义了变量，而不是类型！

这里有个bug：

TreeP newTree()
{
    TreeP tree = (TreeP) malloc(sizeof(TreeP));
    tree->root = NULL;
    return tree;
}

它应该是TreeP tree = malloc(sizeof(struct Tree));。一个很好的例子说明为什么 typedef 结构指针是一个坏主意。由于 Tree 只保存一个指针，这不会造成任何问题，但应该修复它。

这里是 THE 错误：

/*
 * Creates a new node in the tree
 */
void insertTreeNode(TreeP tree, int data)
{
    NodeP p = tree->root;
    Boolean b = FALSE;

    /*  Creates a node and tests for errors  */
    NodeP node = (NodeP) malloc(sizeof(Node));
    if(node == NULL)
        fprintf(stderr, "Memory allocation failed! \n");
    else
    {
        node->data = data;
        node->L = NULL;
        node->R = NULL;

        /*  Inserts in empty tree case  */
        if(tree->root == NULL)
            tree->root = node;
        else
        {
            /*  Inserts in any other case  */
            while(p != NULL && b != TRUE)
            {
                if(node->data >= p->data)
                {
                    if(p->R == NULL)
                    {
                        p->R = node;
                        b = TRUE;
                    }
                    else p = p->R;
                }
                if(node->data <= p->data) // replace this line with else
                {
                    if(p->L == NULL)
                    {
                        p->L = node;
                        b = TRUE;
                    }
                    else p = p->L;
                }
            }
        }
    }
}

如果p->data == data，则必须将新节点链接到节点 p 的左侧或右侧，但不能在两个子树中。

可能还有其他错误，但这个会咬人！