本篇案例来自,KDE4源码。
<强>错误代码:强>
空白LDAPProtocol::德尔(const KUrl和_url bool) { … 如果((id=mOp.del (usrc.dn ())==1)) { LDAPErr (); 返回; } ret=拖把。waitForResult (id, 1); … }
<强>说明:强>
一些程序员竭尽全力想把更多的代码压缩到一行。他们尤其热衷于“如果”条件语句,把赋值和比较一次性完成。
一个典型的错误模式是使用<代码>如果(=Foo()==错误)代码>这样的表达式。上面我们正在处理的实例代码正是这种错误。
比较操作的优先级高于赋值操作。这就是为什么”<代码> mOp.del (usrc.dn())==1> 代码”比较会先执行,然后“真实的”(1)或“假”(0)值被赋给变量id。
如果mOp.del()返回“1”,函数会终止,否则,函数会持续运行,“id”变量会被赋予一个错误的值。它的结果总为0。
<强>正确代码:强>
id=mOp.del (usrc.dn ()); 如果(id==1) {
<>强建议:强>
不要懒得去写多余的代码:毕竟复杂的表达式很难读懂。首先进行赋值,然后再比较。这样以后维护你代码的程序员也会轻松很多,也会减少出错的可能。
今天的这个小技巧看起来很微不足道,不过我希望它会帮到你,牢牢记住并迫使自己写干净整洁且正确的代码,而不是“看我有多专业!”的风格
<强>总结强>
以上就是这篇文章的全部内容了,希望本文的内容对大家的学习或者工作具有一定的参考学习价值,谢谢大家对的支持。如果你想了解更多相关内容请查看下面相关链接