文章出處

今天在sitepoint上看到一篇文章,其中有一段是介紹如何改進以下的這行php代碼。這段代碼經常被他們用來測試應聘者。
  CODE:
  <?
  echo("<p>Search results for query: " .$_GET['query'] . ".</p>");
  ?>
  我覺得這段代碼非常適合用來測試一個php開發人員是否合格。因為這段代碼并沒有要求太多記憶上的東西(例如函數的記憶),但是對安全,性能,兼容性上都做了考察。
  大家也想想看如何改進代碼。我在后面給出修改后的代碼。
  CODE:
  <?php
  if (isset($_GET['query']))
  {
  echo '<p>Search results for query: ',
  htmlspecialchars($_GET['query'], ENT_QUOTES), '.</p>';
  }
  ?>
  改進的地方有 :
  把php短標簽(<?)換成了長標簽(<?php),因為并非所有php環境都支持短標簽
  增加判斷該全局變量是否存在,防止在某些環境下出現notice錯誤,而且在當前的例子里如果query無則沒必要輸出
  把echo的”(”去掉,echo不是一個函數,沒必要使用“(”
  把字符串兩邊的雙引號換成了單引號,提高了性能。因為雙引號里的內容會被嘗試進行變量和轉義字符的解析。
  對query這個get變量進行html過濾。防止輸出html,這可能會帶來包括xss在內的一些危險
  echo后面的字符串之間的間隔使用逗號而不是句號。因為句號之間的字符串會先被連接起來然后再輸出,而,號只是按順序輸出。在這個例子里顯然不需要先連接字符串。(這一點我平時倒沒太注意)

歡迎轉載:http://www.kanwencang.com/bangong/20161104/41061.html

文章列表


不含病毒。www.avast.com
arrow
arrow
    全站熱搜
    創作者介紹
    創作者 大師兄 的頭像
    大師兄

    IT工程師數位筆記本

    大師兄 發表在 痞客邦 留言(0) 人氣()