Bug of day: ret = ret && foo(); 错误
时间：2009-06-27 来源：slimzhao
发现我自己已经第二次犯这样的错误， 比如在容器中的对象， 一段代码想检查是否有任何一个对象非法， 同时检查函数有付作用，
bool ret = true; for(vector<MyObj>::const_iterator it = x.begin(); it != x.end(); ++it) { ret = ret && it->is_valid(); }
if( !ret ) { fprintf(stderr, "some object is invalid\n"); }
这段代码只会报告一个结果， 有没有任何一个对象非法。
＆＆短路作用决定了 ret && it->is_valid() 不能如期工作， 因为如果第一个对象检查失败之后， 后面的对象就不会再次被检查了。
应改为 it->is_valid() && ret
bool ret = true; for(vector<MyObj>::const_iterator it = x.begin(); it != x.end(); ++it) { ret = ret && it->is_valid(); }
if( !ret ) { fprintf(stderr, "some object is invalid\n"); }
这段代码只会报告一个结果， 有没有任何一个对象非法。
＆＆短路作用决定了 ret && it->is_valid() 不能如期工作， 因为如果第一个对象检查失败之后， 后面的对象就不会再次被检查了。
应改为 it->is_valid() && ret
相关阅读 更多 +
