Chinaunix首页 | 论坛 | 博客
  • 博客访问: 61780
  • 博文数量: 12
  • 博客积分: 1400
  • 博客等级: 上尉
  • 技术积分: 150
  • 用 户 组: 普通用户
  • 注册时间: 2008-05-02 11:54
文章分类

全部博文(12)

文章存档

2011年(1)

2010年(2)

2009年(1)

2008年(8)

我的朋友

分类:

2009-03-16 18:07:41

增加这三行php代码的安全,性能和兼容性

今天在sitepoint上看到一篇文章,其中有一段是介绍如何改进以下的这行php代码。这段代码经常被他们用来测试应聘者。
CODE:
echo("

search results for query: " .$_GET['query'] . ".

");
?>

我觉得这段代码非常适合用来测试一个php开发人员是否合格。因为这段代码并没有要求太多记忆上的东西(例如函数的记忆),但是对安全,性能,兼容性上都做了考察。

大家也想想看如何改进代码。我在后面给出修改后的代码。

 

CODE:
if (isset($_GET['query']))
{
   echo '

search results for query: ',
      htmlspecialchars($_GET['query'], ENT_QUOTES), '.

';
}
?>

改进的地方有 :

把php短标签(

增加判断该全局变量是否存在,防止在某些环境下出现notice错误,而且在当前的例子里如果query无则没必要输出

把echo的”(”去掉,echo不是一个函数,没必要使用“(”

把字符串两边的双引号换成了单引号,提高了性能。因为双引号里的内容会被尝试进行变量和转义字符的解析。

对query这个get变量进行html过滤。防止输出html,这可能会带来包括xss在内的一些危险

echo后面的字符串之间的间隔使用逗号而不是句号。因为句号之间的字符串会先被连接起来然后再输出,而,号只是按顺序输出。在这个例子里显然不需要先连接字符串。(这一点我平时倒没太注意)

阅读(677) | 评论(0) | 转发(0) |
给主人留下些什么吧!~~