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


转载于:https://my.oschina.net/searone/blog/391981

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值