查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;