/* 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 < 0) /* 若小于该节点中的单词,则进入左子树 */ 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 里面的条件语句看的要清楚些。今天老子 调试了一个 大上午。原因就在于这个代码格式上。晕死。