C++小知识:不要节约代码行数

时间:2021-05-20

本篇案例来自KDE4 源码。

错误代码:

void LDAPProtocol::del( const KUrl &_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 ) {

建议:

不要懒得去写多余的代码:毕竟复杂的表达式很难读懂。首先进行赋值,然后再比较。这样以后维护你代码的程序员也会轻松很多,也会减少出错的可能。

今天的这个小技巧看起来很微不足道,不过我希望它会帮到你,牢牢记住并迫使自己写干净整洁且正确的代码,而不是“看我有多专业!”的风格

总结

以上就是这篇文章的全部内容了,希望本文的内容对大家的学习或者工作具有一定的参考学习价值,谢谢大家对的支持。如果你想了解更多相关内容请查看下面相关链接

声明:本页内容来源网络,仅供用户参考;我单位不保证亦不表示资料全面及准确无误,也不保证亦不表示这些资料为最新信息,如因任何原因,本网内容或者用户因倚赖本网内容造成任何损失或损害,我单位将不会负任何法律责任。如涉及版权问题,请提交至online#300.cn邮箱联系删除。

相关文章