发现我自己已经第二次犯这样的错误， 比如在容器中的对象， 一段代码想检查是否有任何一个对象非法， 同时检查函数有付作用，
bool ret = true;
for(vector::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
阅读(727) | 评论(0) | 转发(0) |
