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