你中間寫的有點問題，
我改了一下，可以運行了，也可以計算出樹的高度，
你想做的是一個二叉排序樹，不允許數字重復，
以－1作為輸入結束標志，
不過現在是遇到相同輸入時就結束，而且你的樹做的好像也不對。



因為這一句啊：
if(p==null) h=0;

你的遞歸結束條件是p==NULL，
所以即使一個結點的左右子樹是空指針你還是要調用一次，返回高度0。



#include<stdio.h>
#include<malloc.h>
#define null 0
int counter=0;
int h,hl,hr;
typedef struct btreenode
{
int data;
struct btreenode  *lchild;
struct btreenode  *rchild;
}bnode;

bnode  *create(int x,bnode *lbt,bnode *rbt)
{
bnode *p;
p=(bnode*)malloc(sizeof(bnode));
p->data=x;
p->lchild=lbt;
p->rchild=rbt;
return(p);
}
void ins_lchild(bnode *p,int x)
{bnode *q;
if(p==null)
printf("Illegal insert.");
else
{
q=(bnode*)malloc(sizeof(bnode));
q->data=x;
q->lchild=null;
q->rchild=null;
p->lchild=q;
}
}

void ins_rchild(bnode *p,int x)
{bnode *q;
if(p==null)
printf("Illegal insert");
else
{q=(bnode*)malloc(sizeof(bnode));
q->data=x;
q->lchild=null;
q->rchild=null;
p->rchild=q;
}
}

int max(int a,int b)
{
if(a>b)
return(a);
else
{
if(a<b)
return(b);
else
return(a);
}
}

void high(bnode *p,int& h)
{

if(p==null)
h=0;
else
{
if(p->lchild!=null)
  high(p->lchild,hl);
if(p->rchild!=null)
  high(p->rchild,hr);
h=max(hl,hr)+1;
}

}

void main()
{
bnode *bt,*p,*q;
int x;
printf("Input root:");
scanf("%d",&x);
p=create(x,null,null);
bt=p;
scanf("%d",&x);
while(x!=-1)
{
p=bt;
q=p;
while(x!=p->data&&q!=null)

{
p=q;
if(x<p->data)
q=p->lchild;
else
q=p->rchild;
}
if(x==p->data)
{
printf("The number is always egxit.");
break;
}


else
if(x<p->data)
ins_lchild(p,x);
else
ins_rchild(p,x);
scanf("%d",&x);
}
p=bt;


high(p,h);
printf("\n");

  printf("h=%d",h); 

}