C++小知識:不要節(jié)約代碼行數(shù)
本篇案例來自 KDE4 源碼。
錯(cuò)誤代碼:
void LDAPProtocol::del( const KUrl &_url, bool ) { .... if ( (id = mOp.del( usrc.dn() ) == -1) ) { LDAPErr(); return; } ret = mOp.waitForResult( id, -1 ); .... }
說明:
一些程序員竭盡全力想把更多的代碼壓縮到一行。他們尤其熱衷于“if”條件語句,把賦值和比較一次性完成。
一個(gè)典型的錯(cuò)誤模式是使用if (A = Foo() == Error)
這樣的表達(dá)式。上面我們正在處理的實(shí)例代碼正是這種錯(cuò)誤。
比較操作的優(yōu)先級高于賦值操作。這就是為什么“mOp.del( usrc.dn() ) == -1
″ 比較會先執(zhí)行,然后“true”(1) 或“false”(0) 值被賦給變量 id。
如果mOp.del() 返回“-1”,函數(shù)會終止;否則,函數(shù)會持續(xù)運(yùn)行,“id”變量會被賦予一個(gè)錯(cuò)誤的值。它的結(jié)果總為0。
正確代碼:
id = mOp.del(usrc.dn()); if ( id == -1 ) {
建議:
不要懶得去寫多余的代碼:畢竟復(fù)雜的表達(dá)式很難讀懂。首先進(jìn)行賦值,然后再比較。這樣以后維護(hù)你代碼的程序員也會輕松很多,也會減少出錯(cuò)的可能。
今天的這個(gè)小技巧看起來很微不足道,不過我希望它會幫到你,牢牢記住并迫使自己寫干凈整潔且正確的代碼,而不是“看我有多專業(yè)!”的風(fēng)格
總結(jié)
以上就是這篇文章的全部內(nèi)容了,希望本文的內(nèi)容對大家的學(xué)習(xí)或者工作具有一定的參考學(xué)習(xí)價(jià)值,謝謝大家對腳本之家的支持。如果你想了解更多相關(guān)內(nèi)容請查看下面相關(guān)鏈接
- C++實(shí)踐分?jǐn)?shù)類中運(yùn)算符重載的方法參考
- C++實(shí)踐排序函數(shù)模板項(xiàng)目的參考方法
- C++二維數(shù)組中數(shù)組元素存儲地址的計(jì)算疑問講解
- 劍指offer之C++語言實(shí)現(xiàn)鏈表(兩種刪除節(jié)點(diǎn)方式)
- Android Java調(diào)用自己C++類庫的實(shí)例講解
- Python使用ctypes調(diào)用C/C++的方法
- C++小知識:用合適的工具來分析你的代碼
- C++小知識:不要去做編譯器的工作
- C++小知識:C/C++中不要按值傳遞數(shù)組
- C++實(shí)踐Time類中的運(yùn)算符重載參考方法
相關(guān)文章
C語言實(shí)現(xiàn)兩個(gè)遞減數(shù)列中尋找某一個(gè)數(shù)
這篇文章主要介紹了C語言實(shí)現(xiàn)兩個(gè)遞減數(shù)列中尋找某一個(gè)數(shù),是一類經(jīng)典的數(shù)組操作算法,需要的朋友可以參考下2014-09-09深入探討POJ 2312 Battle City 優(yōu)先隊(duì)列+BFS
本篇文章是對優(yōu)先隊(duì)列+BFS進(jìn)行了詳細(xì)的分析介紹,需要的朋友參考下2013-05-05