/* addtree函数:在p位置或p的下方增加一个w节点 */
struct tnode *addtree(struct tnode *p, char *w)
{
int cond=0;
if (!p) { /* 该单词是一个新单词 */
p = talloc(); /* 创建一个新节点 */
p->word = strdup(w);
p->count = 1;
p->left = p->right = NULL;
} else if ((cond=strcmp(w, p->word)) == 0)
p ->count++; /* 新单词与节点中的单词匹配 */
else if (cond
p->left = addtree(p->left, w);
else /* 若大于该节点中的单词,则进入右子树 */
p->right = addtree(p->right, w);
//cond = strcmp(w, p->word);
return p;
}
一眼看过去,好像这个代码没有啥子问题,但是我今天就遇到了一个奇葩问题。
注意:
if (!p) { /* 该单词是一个新单词 */
p = talloc(); /* 创建一个新节点 */
p->word = strdup(w);
p->count = 1;
p->left = p->right = NULL;
} else if ((cond=strcmp(w, p->word)) == 0)
最后一行代码建议不应该这样写,原因在于
你会误认为 else if 哪行代码的内容属于 if 语句内。调试的时候看不清楚 else if 里面的条件到底是属于那个 代码块。
建议吧else if 哪行代码 换行写, 这样 else if 里面的条件语句看的要清楚些。今天老子 调试了一个 大上午。原因就在于这个代码格式上。晕死。