條件語(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)一,這樣更便于檢查。