條件語(yǔ)句體應(yīng)該總是被大括號(hào)包圍徘层。盡管有時(shí)候你可以不使用大括號(hào)(比如组砚,條件語(yǔ)句體只有一行內(nèi)容)吻商,但是這樣做會(huì)帶來(lái)問(wèn)題隱患。比如糟红,增加一行代碼時(shí)艾帐,你可能會(huì)誤以為它是 if 語(yǔ)句體里面的。此外盆偿,更危險(xiǎn)的是柒爸,如果把 if 后面的那行代碼注釋掉,之后的一行代碼會(huì)成為 if 語(yǔ)句里的代碼塘雳。
推薦:
if (!error) {
return success;
}
不推薦:
if (!error)
return success;
和
if (!error) return success;
在 2014年2月 蘋果的 SSL/TLS 實(shí)現(xiàn)里面發(fā)現(xiàn)了知名的 goto fail 錯(cuò)誤腔寡。
代碼在這里:
static OSStatus
SSLVerifySignedServerKeyExchange(SSLContext *ctx, bool isRsa, SSLBuffer signedParams, uint8_t *signature, UInt16 signatureLen)
{
OSStatus err;
...
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0)
goto fail;
if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0)
goto fail;
goto fail;
if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0)
goto fail;
...
fail:
SSLFreeBuffer(&signedHashes);
SSLFreeBuffer(&hashCtx);
return err;
}
顯而易見表箭,這里有沒有括號(hào)包圍的2行連續(xù)的 goto fail; 。我們當(dāng)然不希望寫出上面的代碼導(dǎo)致錯(cuò)誤今野。
此外,在其他條件語(yǔ)句里面也應(yīng)該按照這種風(fēng)格統(tǒng)一谈撒,這樣更便于檢查腥泥。