查BUG笔记-一个令人尴尬的低级错误-if-else的分支

业务有bug,需要在同事的代码上修改问题,引出了这个bug;

问题1.

原来的代码是这个样子的,程序中有三种情况,现在需要在CalculateMode::SUB分支中增加一个条件,因为下面代码是在循环中,我就将条件&&到了CalculateMode::SUB == mode条件之后,没看到可能会走到else分支。出现了CalculateMode::SUB == mode,但结果执行到了else分支;else分支对应的是CalculateMode::SUB == DIV

int mode = 0;
if (CalculateMode::ADD == mode) {
	//code 1;
}
else if (CalculateMode::SUB == mode) {
	//code 2;
}
else {
	//code 3;
}

代码改成了下面这个样子:

int mode = 0;
if (CalculateMode::ADD == mode) {
	//code 1;
}
else if (CalculateMode::SUB == mode) {
	if (new condition){
		//code 2;
	}	
}
else if (CalculateMode::DIV == mode){
	//code 3;
}

在想是这种方式好呢,还是之前那种if-else的方式好呢,下面这段代码,起码不会出现之前的bug了;

问题2.

之前自己碰到过,别人也碰到过
下面这段代码中,不管condition是否为true,都会输出"condition ";
是因为()后使用了;,虽然代码块有缩进,但是这已经是两个语句了;

if(condition);
	cout<<"condition "<<endl;
  • 0
    点赞
  • 0
    收藏
  • 打赏
    打赏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
©️2022 CSDN 皮肤主题:深蓝海洋 设计师:CSDN官方博客 返回首页
评论

打赏作者

黑山老妖的博客

你的鼓励将是我创作的最大动力

¥2 ¥4 ¥6 ¥10 ¥20
输入1-500的整数
余额支付 (余额:-- )
扫码支付
扫码支付:¥2
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

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

余额充值