復制代碼 代碼如下:
<?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
}
?>
粗看起來代碼已經無懈可擊，我們親愛的 小馬 還是發現了問題的存在。就是在 Javascript 中的那個判斷是永遠為 true
復制代碼 代碼如下:
if ('undefined' == typeof document.cookie['cookieName']) {
// ...
}
因為這段代碼是在 PHP 端有個前提，就是
if (!haveCookie('cookieName'))的時候，才會在客戶端顯示。那麼，當不滿足這一條件，這段代碼自然就不會扔給客戶端。這樣說似乎有點籠統，那麼先撇開 Javascript 代碼，我們就單純使用 PHP 代碼表述一下
復制代碼 代碼如下:
<?php
header("Content-type: text/javascript");
if (!haveCookie('cookieName')) {
if (!haveCookie('cookieName')) {
setCookie('cookieName');
}
}
?>
這樣就顯得清晰了很多，並很容易就能發現問題所在 -- 我們在不經意間就多做了一次判斷，雖然這是 Javascript 在客戶端執行的。
總結下，本人從這段代碼想到的些廢話：
代碼越長，不見得效率就越高
在不影響邏輯和流程的情況下，盡量將多個判斷寫在一起
盡量將低復雜度的函數放前判斷
過多的判斷容易造成程序效率降低，在判斷中使用高時間復雜度的函數時尤其要注意
如果發現 if 嵌套得太多，就得重新考慮流程和算法
健壯的代碼不是靠過分的判斷保證而成的
將代碼簡化後，會發現很多還未發現的問題
過多的判斷另個角度理解，是缺乏對代碼的信心
最後，再次感謝 小馬 同志。
