你在delete中确实把结点给删掉了，可是别忘了结点a（根）还在指着他呢，你应该把结点a的右子树置为NULL。 不应该 观前不顾后

非要让我亲自动手，好吧
换成我这个就可以了
void Delete(BiTree &T,ElemType x)
{
    if(T)
    {
        Delete(T->lchild,x);
        Delete(T->rchild,x);
        if(T->data==x)
        {
            FREE(T);
            T=NULL;
        }
    }
}

或者你不用C++，也就不能用引用。此时你可以用二级指针来代替
别忘了在main函数中，加上 &

void Delete(BiTree *T,ElemType x)
{
    if(*T)
    {
        Delete(&(*T)->lchild,x);
        Delete(&(*T)->rchild,x);
        if((*T)->data==x)
        {
            FREE(*T);
            *T=NULL;
        }
    }
}

你自己换上试一试不就知道了吗。难道你编程从不调试，你完全可以设个断点，自己看看 a的右子树是否为NULL，你的程序肯定不是NULL。

你认为设成NULL，其实你没改，你改的是T这个局部变量。而且，我记得以前说过函数调用会产生临时对象（这里的函数参数就是）。
另外，还告诫过你，不要直接传对象，要传引用（当然也可以是指针）。
值传递和地址传递，你应该明白吧！在这里使用二级指针，是因为你要修改的是一个地址，所以要用指向地址的指针。

去年才学的忘的差不多了