C++小知识:不要节约代码行数
本篇案例来自 KDE4源码。
错误代码：
voidLDAPProtocol::del(constKUrl&_url,bool)
{
....
if((id=mOp.del(usrc.dn())==-1)){
LDAPErr();
return;
}
ret=mOp.waitForResult(id,-1);
....
}
说明：
一些程序员竭尽全力想把更多的代码压缩到一行。他们尤其热衷于“if”条件语句，把赋值和比较一次性完成。
一个典型的错误模式是使用if(A=Foo()==Error)这样的表达式。上面我们正在处理的实例代码正是这种错误。
比较操作的优先级高于赋值操作。这就是为什么“mOp.del(usrc.dn())==-1″比较会先执行，然后“true”(1)或“false”(0)值被赋给变量id。
如果mOp.del()返回“-1”，函数会终止；否则，函数会持续运行，“id”变量会被赋予一个错误的值。它的结果总为0。
正确代码：
id=mOp.del(usrc.dn());
if(id==-1){
建议：
不要懒得去写多余的代码：毕竟复杂的表达式很难读懂。首先进行赋值，然后再比较。这样以后维护你代码的程序员也会轻松很多，也会减少出错的可能。
今天的这个小技巧看起来很微不足道，不过我希望它会帮到你，牢牢记住并迫使自己写干净整洁且正确的代码，而不是“看我有多专业！”的风格
总结
以上就是这篇文章的全部内容了，希望本文的内容对大家的学习或者工作具有一定的参考学习价值，谢谢大家对毛票票的支持。如果你想了解更多相关内容请查看下面相关链接
声明：本文内容来源于网络，版权归原作者所有，内容由互联网用户自发贡献自行上传，本网站不拥有所有权，未作人工编辑处理，也不承担相关法律责任。如果您发现有涉嫌版权的内容，欢迎发送邮件至：czq8825#qq.com（发邮件时，请将#更换为@）进行举报，并提供相关证据，一经查实，本站将立刻删除涉嫌侵权内容。