快捷導(dǎo)航

由php if 想到的些問題

更新時間：2008年03月22日 14:34:11 作者：

在編寫一段并不復(fù)雜的腳本的時候，發(fā)現(xiàn)了一個問題。先說說代碼，它的主要功能是用 PHP 判斷是否生成一段 Javascript，并使用 Cookie 記錄狀態(tài)。

<?php  
/* PHP code */  
header("Content-type: text/javascript");  
if (!haveCookie('cookieName')) {  
   // ... do something  
?>  
/* Javascript code */  
if ('undefined' == typeof document.cookie['cookieName']) {  
    setCookie('cookieName', 3600);  
}  

   // ... do something with Javascript  
<?php  
}  
?>

粗看起來代碼已經(jīng)無懈可擊，我們親愛的小馬還是發(fā)現(xiàn)了問題的存在。就是在 Javascript 中的那個判斷是永遠為 true

復(fù)制代碼代碼如下:

if ('undefined' == typeof document.cookie['cookieName']) {  
    // ...  
}

因為這段代碼是在 PHP 端有個前提，就是

if (!haveCookie('cookieName'))的時候，才會在客戶端顯示。那么，當(dāng)不滿足這一條件，這段代碼自然就不會扔給客戶端。這樣說似乎有點籠統(tǒng)，那么先撇開 Javascript 代碼，我們就單純使用 PHP 代碼表述一下

復(fù)制代碼代碼如下:

<?php  
header("Content-type: text/javascript");  
if (!haveCookie('cookieName')) {  
   if (!haveCookie('cookieName')) {  
       setCookie('cookieName');  
   }  
}  
?>

這樣就顯得清晰了很多，并很容易就能發(fā)現(xiàn)問題所在 -- 我們在不經(jīng)意間就多做了一次判斷，雖然這是 Javascript 在客戶端執(zhí)行的。

總結(jié)下，本人從這段代碼想到的些廢話：

代碼越長，不見得效率就越高
在不影響邏輯和流程的情況下，盡量將多個判斷寫在一起
盡量將低復(fù)雜度的函數(shù)放前判斷
過多的判斷容易造成程序效率降低，在判斷中使用高時間復(fù)雜度的函數(shù)時尤其要注意
如果發(fā)現(xiàn) if 嵌套得太多，就得重新考慮流程和算法
健壯的代碼不是靠過分的判斷保證而成的
將代碼簡化后，會發(fā)現(xiàn)很多還未發(fā)現(xiàn)的問題
過多的判斷另個角度理解，是缺乏對代碼的信心
最后，再次感謝小馬同志。