由php if 想到的些问题

yipeiwu_com6年前PHP代码库
复制代码 代码如下:

<?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 嵌套得太多,就得重新考虑流程和算法
健壮的代码不是靠过分的判断保证而成的
将代码简化后,会发现很多还未发现的问题
过多的判断另个角度理解,是缺乏对代码的信心
最后,再次感谢 小马 同志。

相关文章

让PHP开发者事半功倍的十大技巧小结

如果你使用一面大镜子作为冲浪板会发生什么?或许你会在较短的时间内征服海浪,但是你肯定从内心深处明白,这不是冲浪的正确选择。同样的道理也适用于PHP编程,尽管这样的类比听起来有一些古怪。我...

PHP简单计算两个时间差的方法示例

本文实例讲述了PHP简单计算两个时间差的方法。分享给大家供大家参考,具体如下: <?php //PHP计算两个时间差的方法 $startdate="2010-12-1...

修复ShopNC使用QQ 互联时提示100010 错误

修复ShopNC使用QQ 互联时提示100010 错误

QQ 互联不允许 URL 有 Hash 存在,而 ShopNC 默认下是 ?act=toqq&op=g 这样的链接回调的,所以会导致设置失败,或者 100010 错误。 1. 建...

FastCGI 进程意外退出造成500错误

FastCGI 进程意外退出造成500错误

在一台新服务器上,安装新网站,之前只放至了一个网站。是服务器商配置好的,非集成环境。 添加了一个新站,路径都制定好了,但是在访问时出现了500错误。提示貌似是php的问题,但是之前的网站...

解析Linux下Varnish缓存的配置优化

Varnish是一款高性能的开源HTTP加速器,挪威最大的在线报纸 Verdens Gang 使用3台Varnish代替了原来的12台Squid,性能比以前更好。但与老牌的squid相比...