又發(fā)現(xiàn)了自己之前一個很不好的編程習(xí)慣……就是if條件判斷喜歡用&&把各個條件合并到一起。其實更健康的做法是寫兩層if。

拿《我的來世是個包裹》的代碼來舉例。有一段代碼是這樣的：

? ? ? ? ? ? ? ? ? ? if (isStoryModeOn && !GameSlot.IsMsgBoxOn)

? ? ? ? ? ? ? ? ? ? {

? ? ? ? ? ? ? ? ? ? ? ? UpdateShowStorySpeak();

? ? ? ? ? ? ? ? ? ? ? ? CheckMissionComplete();

? ? ? ? ? ? ? ? ? ? ? ? CheckItemInAndOut();

? ? ? ? ? ? ? ? ? ? ? ? CheckExitStoryParagraph();

? ? ? ? ? ? ? ? ? ? }

? ? ? ? ? ? ? ? ? ? else

? ? ? ? ? ? ? ? ? ? {

? ? ? ? ? ? ? ? ? ? ? ? AutoSave();

? ? ? ? ? ? ? ? ? ? }

意思是，非故事模式（調(diào)試模式下）調(diào)用自動存檔判斷方法（存檔時間間隔在AutoSave方法里處理了，不寫在外面），故事模式下且消息框不彈出的時候執(zhí)行各個邏輯檢測流程。

我很驚訝之前放著邏輯漏洞那么大的代碼無視。明顯不能這么寫。這樣會導(dǎo)致故事模式下只要一彈出對話框就會自動保存，我腦子真的有坑。正確的寫法是：

? ? ? ? ? ? ? ? ? ? if (isStoryModeOn)

? ? ? ? ? ? ? ? ? ? {

? ? ? ? ? ? ? ? ? ? ? ? if (!GameSlot.IsMsgBoxOn)

? ? ? ? ? ? ? ? ? ? ? ? {

? ? ? ? ? ? ? ? ? ? ? ? ? ? UpdateShowStorySpeak();

? ? ? ? ? ? ? ? ? ? ? ? ? ? CheckMissionComplete();

? ? ? ? ? ? ? ? ? ? ? ? ? ? CheckItemInAndOut();

? ? ? ? ? ? ? ? ? ? ? ? ? ? CheckExitStoryParagraph();

? ? ? ? ? ? ? ? ? ? ? ? }

? ? ? ? ? ? ? ? ? ? }

? ? ? ? ? ? ? ? ? ? else

? ? ? ? ? ? ? ? ? ? {

? ? ? ? ? ? ? ? ? ? ? ? AutoSave();

? ? ? ? ? ? ? ? ? ? }

沒錯，
?if (條件1?&&?條件2)? {語句1；語句2；……；語句n}
和
?if (條件1)?{ if(條件2){語句1；語句2；……；語句n}}
是等價的。
但是
?if (條件1?&&?條件2)? {語句1；語句2；……；語句n；}else{語句m；}
和
?if (條件1)?{ if(條件2){語句1；語句2；……；語句n}}else{語句m；}
完全不等價！

看了下面這張圖你們就明白了：

基本功不扎實的下場……