今天小編就爲大家分享一篇關於C++小知識:不要節約代碼行數，小編覺得內容挺不錯的，現在分享給大家，具有很好的參考價值，需要的朋友一起跟隨小編來看看吧
本篇案例來自 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 ) {
建議：
不要懶得去寫多餘的代碼：畢竟複雜的表達式很難讀懂。首先進行賦值，然後再比較。這樣以後維護你代碼的程序員也會輕鬆很多，也會減少出錯的可能。
今天的這個小技巧看起來很微不足道，不過我希望它會幫到你，牢牢記住並迫使自己寫乾淨整潔且正確的代碼，而不是“看我有多專業！”的風格
總結
以上就是這篇文章的全部內容了，希望本文的內容對大家的學習或者工作具有一定的參考學習價值，謝謝大家對神馬文庫的支持。如果你想了解更多相關內容請查看下面相關鏈接