业务有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;