C语言代码格式

/* 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 里面的条件语句看的要清楚些。今天老子 调试了一个 大上午。原因就在于这个代码格式上。晕死。