POJ 1577 Falling Leaves（二叉搜索树）

思路：当时学长讲了之后，似乎有点思路----------就是倒着建一个 二叉搜索树

代码1：超时

详见超时原因

#include<iostream>
#include<cstring>
#include<stdio.h>
using namespace std;

char c[100][100];
struct node
{
char c;
node *lchild;
node *rchild;
};

void f(node *p)//前序遍历输出，用递归
{
printf("%c",p->c);
if(p->lchild!=NULL)
f(p->lchild);
if(p->rchild!=NULL)
f(p->rchild);
}

int main()
{
int i;
int j;
int exit;
exit=0;
while(1)
{
memset(c,'\0',sizeof(c));
i=0;
while(1)
{
scanf("%s",c[i]);
if(c[i][0]=='*'||c[i][0]=='$')
{
if(c[i][0]=='$')
exit=1;
break;
}
i++;
}
i--;
node *p=new node;
p->c=c[i][0];
p->lchild=NULL;
p->rchild=NULL;
i--;
node *p2;
while(1)//遍历i
{
j=0;
while(1)//遍历j
{
p2=p;
node *p3=new node;//存储字母节点
p3->c=c[i][j];
p3->lchild=NULL;
p3->rchild=NULL;
while(1)//把字母挂到树中
{
if(p3->c < p2->c)
{
if(p2->lchild==NULL)
{
p2->lchild=p3;
break;
}
else if(p2->lchild!=NULL)//超时原因
p2=p2->lchild;
}
else if(p3->c > p2->c)//超时原因
{
if(p2->rchild==NULL)
{
p2->rchild=p3;
break;
}
else if(p2->rchild!=NULL)//超时原因，这三处的 else if() 判断多余，应直接写成else
p2=p2->rchild;
}
}
j++;
if(c[i][j]=='\0')
break;
}
i--;
if(i==-1)//修正超时原因后，此处会造成 runtime error，因为i为 -1时 也会 进入此while循环，数组越界
break;
}
f(p);
printf("\n");
if(exit==1)
break;
}
return 0;
}

修正代码：

1.

当时也是看了一下网上的一篇代码，看到他用的是一个结构体数组，我在考虑是不是动态开辟结构体浪费了时间，于是我改成用结构体数组，结果还是超时。

看来并非单单是数组这个原因造成我的代码超时( 当然我感觉，数组应该确实 比动态 开辟 省点时间)

2.

然后我继续想。。。

我突然间想到一个问题，那就是在我的代码中，有几句这样的 语句

if(p2->lchild==NULL)
{
p2->lchild=p3;
break;
}
else if(p2->lchild!=NULL)//超时原因
p2=p2->lchild;

看到没有？ 就是有一些没必要的判断，

在 if(p2->lchild==NULL) 这一句判断完之后，

下面的 else if(p2->lchild!=NULL)//超时原因 这一句纯属多余

于是我就抱着试试看的心态改了一下。

改过来之后 ，发现超时原因的确如此。

之所以这么写，我还是感觉自己容易出错，有时候考虑不周，所以都加上了 if() 条件判断，当时是为了避免出错 ，才这么写的。

所以养成了一点点的习惯这么写。

没想到。。。。这么写后造成超时，看来还是得改正这种写法，没必要的判断别用 -----------直接用 else 语句

代码2：正确

#include<iostream>
#include<cstring>
#include<stdio.h>
using namespace std;

char c[100][100];
struct node
{
char c;
node *lchild;
node *rchild;
};

void f(node *p)//前序遍历输出，用递归
{
printf("%c",p->c);
if(p->lchild!=NULL)
f(p->lchild);
if(p->rchild!=NULL)
f(p->rchild);
}

int main()
{
int i;
int j;
int exit;
exit=0;
while(1)
{
memset(c,'\0',sizeof(c));
i=0;
while(1)
{
scanf("%s",c[i]);
if(c[i][0]=='*'||c[i][0]=='$')
{
if(c[i][0]=='$')
exit=1;
break;
}
i++;
}
i--;
node *p=new node;
p->c=c[i][0];
p->lchild=NULL;
p->rchild=NULL;
i--;
node *p2;
while(i>=0)//遍历i    --------此处为修改的上面代码 (  修改前出现了 runtime error )，应该 把 i > = 0 的 判断放在前面
{
j=0;
while(1)//遍历j
{
p2=p;
node *p3=new node;//存储字母节点
p3->c=c[i][j];
p3->lchild=NULL;
p3->rchild=NULL;
while(1)//把字母挂到树中
{
if(p3->c < p2->c)
{
if(p2->lchild==NULL)
{
p2->lchild=p3;
break;
}
else    //  修正 上面代码
p2=p2->lchild;
}
else   //  修正 上面代码
{
if(p2->rchild==NULL)
{
p2->rchild=p3;
break;
}
else   //  修正 上面代码
p2=p2->rchild;
}
}
j++;
if(c[i][j]=='\0')
break;
}
i--;
}
f(p);
printf("\n");
if(exit==1)
break;
}
return 0;
}

ps：再看这段代码，其实超时原因不是因为else if

而是p3->c > p2->c应该为p3->c >= p2->c

。。。