10條PHP進階技巧[修正版]

1.使用一個SQL注射備忘單
一個基本的原則就是，永遠不要相信使用者提交的資料。
另一個規則就是，在你發送或者儲存資料時對它進行轉義（escape）。
可以總結為：filter input, escape output (FIEO).　輸入過濾，輸出轉義。
通常導致SQL注射漏洞的原因是沒有對輸入進行過濾，如下語句： 複製代碼 代碼如下:<?php
$query = "SELECT *
FROM users
WHERE name = '{$_GET['name']}'";

在這個例子中，$_GET['name']來自使用者提交的資料，既沒有進行轉義，也沒有進行過濾~~
對於轉義輸出，你要記住用於你程式外部的資料需要被轉義，否則，它可能被錯誤地解析。
相反，過濾輸入能確保資料在使用前是正確的.
對於過濾輸入，你要記住，在你程式外部的未經處理資料需要被過濾，因為它們是不可信任的。
如下例子示範了輸入過濾和輸出轉義： 複製代碼 代碼如下:<?php
// Initialize arrays for filtered and escaped data, respectively.
$clean = array();
$sql = array();
// Filter the name. (For simplicity, we require alphabetic names.)
if (ctype_alpha($_GET['name'])) {
$clean['name'] = $_GET['name'];
} else {
// The name is invalid. Do something here.
}
// Escape the name.
$sql['name'] = mysql_real_escape_string($clean['name']);
// Construct the query.
$query = "SELECT *
FROM users
WHERE name = '{$sql['name']}'";
?>

另一個有效防止SQL注射的方法是使用prepare 語句，如： 複製代碼 代碼如下:<?php
// Provide the query format.
$query = $db->prepare('SELECT *
FROM users
WHERE name = :name');
// Provide the query data and execute the query.
$query->execute(array('name' => $clean['name']));
?>

2.瞭解比較子之間的不同
例如，你使用strpos() 來檢測在一個字串中是否存在一個子串 (如果子串沒有找到，函數將返回 FALSE ), 結果可能會導致錯誤: 複製代碼 代碼如下:<?php
$authors = 'Chris & Sean';
if (strpos($authors, 'Chris')) {
echo 'Chris is an author.';
} else {
echo 'Chris is not an author.';
}
?>

上例中，由於子串處於最開始的位置，因此strpos() 函數正確地返回了0，表明子串處於字串中最開始的位置。然後，因為條件陳述式會把結果當成Boolean（布爾）類型的，因此　0　就被PHP給計算成了　FALSE，最終導致條件陳述式判斷失敗。
當然，這個BUG可以用嚴格的比較語句來修正： 複製代碼 代碼如下:<?php
if (strpos($authors, 'Chris') !== FALSE) {
echo 'Chris is an author.';
} else {
echo 'Chris is not an author.';
}
?>

3.減少else（Shortcut the else）
記住，在你使用變數前總是要先初始化它們。
考慮如下一個用來根據使用者名稱來檢測使用者是否是管理員的條件陳述式： 複製代碼 代碼如下:<?php
if (auth($username) == 'admin') {
$admin = TRUE;
} else {
$admin = FALSE;
}
?>

這個看起來似乎足夠安全，因為看一眼就很容易理解。想象一下有一個更複雜一點的例子，它為name和email同時設定變數，為方便起見： 複製代碼 代碼如下:<?php
if (auth($username) == 'admin') {
$name = 'Administrator';
$email = 'admin@example.org';
$admin = TRUE;
} else {
/* Get the name and email from the database. */
$query = $db->prepare('SELECT name, email
FROM users
WHERE username = :username');
$query->execute(array('username' => $clean['username']));
$result = $query->fetch(PDO::FETCH_ASSOC);
$name = $result['name'];
$email = $result['email'];
$admin = FALSE;
}
?>

因為　$admin 還是明確地被設定為TRUE or FALSE，似乎一切都完好。但是，如果另一個開發人員後來在代碼裡加了一個elseif語句，很可能他會忘記這回事： 複製代碼 代碼如下:<?php
if (auth($username) == 'admin') {
$name = 'Administrator';
$email = 'admin@example.org';
$admin = TRUE;
} elseif (auth($username) == 'mod') {
$name = 'Moderator';
$email = 'mod@example.org';
$moderator = TRUE;
} else {
/* Get the name and email. */
$query = $db->prepare('SELECT name, email
FROM users
WHERE username = :username');
$query->execute(array('username' => $clean['username']));
$result = $query->fetch(PDO::FETCH_ASSOC);
$name = $result['name'];
$email = $result['email'];
$admin = FALSE;
$moderator = FALSE;
}
?>

如果一個使用者提供一個能夠觸發elseif條件的使用者名稱(username)，　$admin 沒有被初始化，這可能會導致不必要的行為，或者更糟糕的情況，一個安全性漏洞。另外，一個類似的情況對於 $moderator　變數來說同樣存在，它在第一個條件中沒有被初始化。
通過初始化$admin 和 $moderator　，這是完全很容易避免這一情況的發生的： 複製代碼 代碼如下:<?php
$admin = FALSE;
$moderator = FALSE;
if (auth($username) == 'admin') {
$name = 'Administrator';
$email = 'admin@example.org';
$admin = TRUE;
} elseif (auth($username) == 'mod') {
$name = 'Moderator';
$email = 'mod@example.org';
$moderator = TRUE;
} else {
/* Get the name and email. */
$query = $db->prepare('SELECT name, email
FROM users
WHERE username = :username');
$query->execute(array('username' => $clean['username']));
$result = $query->fetch(PDO::FETCH_ASSOC);
$name = $result['name'];
$email = $result['email'];
}
?>

不管剩下的代碼是什麼，現在已經明確了 $admin 值　為FALSE ，除非它被顯式地設定為其它值。對於　$moderator　也是一樣的。最壞的可能發生的情況就是，在任何條件下都沒有修改$admin 或 $moderator ，導致某個是administrator 或moderator的人沒有被當作相應的administrator 或moderator　。
如果你想 shortcut something ,並且你看到我們的例子有包含有else覺得有點失望。我們有一個bonus tip 你可能會感興趣的。我們並不確定它可以被認為是a shortcut,但是我們希望它仍然是有協助的。
考慮一下一個用於檢測一個使用者是否被授權查看一個特定頁面的函數： 複製代碼 代碼如下:<?php
function authorized($username, $page) {
if (!isBlacklisted($username)) {
if (isAdmin($username)) {
return TRUE;
} elseif (isAllowed($username, $page)) {
return TRUE;
} else {
return FALSE;
}
} else {
return FALSE;
}
}
?>

這個例子是相當的簡單，因為只有三條規則需要考慮：
administrators　總是被允許訪問的，
處於黑名單的永遠是禁止訪問的，
isAllowed()決定其它人是否有權訪問。
（還有一個特例是：當一個administrator　處於黑名單中，但這似乎是不太可能的事，所以我們這裡直接忽視這種情況）。
我們使用函數來做這個判斷以保持代碼的簡潔然後集中注意力到商務邏輯上去。
如： 複製代碼 代碼如下:<?php
function authorized($username, $page) {
if (!isBlacklisted($username)) {
if (isAdmin($username) || isAllowed($username, $page)) {
return TRUE;
} else {
return FALSE;
}
} else {
return FALSE;
}
}
?>

事實上，你可以精減整個函數到一個複合條件： 複製代碼 代碼如下:<?php
function authorized($username, $page) {
if (!isBlacklisted($username) && (isAdmin($username) || isAllowed($username, $page)) {
return TRUE;
} else {
return FALSE;
}
}
?>

最後，這個可以被減少到只有一個return: 複製代碼 代碼如下:<?php
function authorized($username, $page) {
return (!isBlacklisted($username) && (isAdmin($username) || isAllowed($username, $page));
}
?>

如果你的目標是謄清代碼的行數，那麼這樣你做到的。但是，你要注意到，我們在用isBlacklisted(), isAdmin() 和 isAllowed() ，這取決於參與這些判斷的東西,減少代碼到只剩下一個複合條件可能不吸引人。
這下說到我們的小技巧上了，一個“立即返回”函數，所以，如果你儘快返回，你可以很簡單地表達這些規則： 複製代碼 代碼如下:<?php
function authorized($username, $page) {
if (isBlacklisted($username)) {
return FALSE;
}
if (isAdmin($username)) {
return TRUE;
}
return isAllowed($username, $page);
}
?>

這個例子使用了更多行數的代碼，但是它是非常簡單和不惹人注意的。更重要的是，這個方法減少了你必需考慮的內容相關的數量。例如，一旦你決定了使用者是否處於黑名單裡面，你就可以安全地忘掉這件事了。特別是你的邏輯很複雜的時候，這是相當的有協助的。
4. 總是使用大括弧
PS:原諒是“扔掉那些方括弧　Drop Those Brackets”
根據本文的內容, 我們相應作者的意思應該是 “braces,” 而不是brackets. “Curly brackets” 可能有大括弧的意思, 但是”brackets” 通常表示 “方括弧”的意思。這個技巧應該被無條件的忽略，因為，沒有大括弧，可讀性和可維護性被破壞了。
舉一個簡單的例子： 複製代碼 代碼如下:<?php
if (date('d M') == '21 May')
$birthdays = array('Al Franken',
'Chris Shiflett',
'Chris Wallace',
'Lawrence Tureaud');
?>

If you're good enough, smart enough, secure enough, notorious enough, or pitied enough, 你可能會想在5月21號參加社交聚會: 複製代碼 代碼如下:<?php
if (date('d M') == '21 May')
$birthdays = array('Al Franken',
'Chris Shiflett',
'Chris Wallace',
'Lawrence Tureaud');
party(TRUE);
?>

沒有大括弧，這個簡單的條件導致你每天參加社交聚會 。也許你有毅力,因此這個錯誤是一個受歡迎的。希望那個愚蠢的例子並不分散這一的觀點,那就是過度狂歡是一種出人意料的副作用。
為了提倡丟掉大括弧，先前的文章使用類似下面的簡短的語句作為例子： 複製代碼 代碼如下:<?php
if ($gollum == 'halfling') $height --;
else $height ++;
?>

因為每個條件被放在單獨的一行, 這種錯誤似乎會較少發生, 但是這將導致另一個問題:代碼的不一致和需要更多的時間來閱讀和理解。一致性是這樣一個重要的特性，以致開發人員經常遵守一個編碼通訊協定,即使他們不喜歡編碼通訊協定本身。
我們提倡總是使用大括弧： 複製代碼 代碼如下:<?php
if (date('d M') == '21 May') {
$birthdays = array('Al Franken',
'Chris Shiflett',
'Chris Wallace',
'Lawrence Tureaud');
party(TRUE);
}
?>

你天天聚會是沒關係的,但要保證這是經過思考的,還有，請一定要邀請我們!
5. 盡量用str_replace() 而不是 ereg_replace() 和 preg_replace()
我們討厭聽到的否認的話，但是（原文）這個用於示範誤用的小技巧導致了它試圖避免的同樣的濫用問題。（
We hate to sound disparaging, but this tip demonstrates the sort of misunderstanding that leads to the same misuse it's trying to prevent.）
很明顯字串函數比Regex函數在字元匹配方面更快速高效，但是作者糟糕地試圖從失敗中得出一個推論：
（FIX ME: It's an obvious truth that string functions are faster at string matching than regular expression functions, but the author's attempt to draw a corollary from this fails miserably:）
If you're using regular expressions, then ereg_replace() and preg_replace() will be much faster than str_replace().
Because str_replace() does not support pattern matching, this statement makes no sense. The choice between string functions and regular expression functions comes down to which is fit for purpose, not which is faster. If you need to match a pattern, use a regular expression function. If you need to match a string, use a string function.
6. 使用三重運算子
三元運算子的好處是值得討論的. 下面是一行從最近我們進行的審計的代碼中取出的： 複製代碼 代碼如下:<?php
$host = strlen($host) > 0 ? $host : htmlentities($host);
?>

啊，作者的真實意願是如果字串的長度大於0　就轉義 $host　，　但是卻意外地做了相反的事情。很容易犯的錯誤是吧？也許吧。在代碼審計過程中很容易錯過？當然。簡潔並不一定能使代碼變得很好。
三重運算子對於單行，原型，和模板也行是適合的，但是我們相信一個普通的條件陳述式總是更好的。PHP是描述性的和詳細的，我們認為代碼也應該是。
7. Memcached
磁碟訪問是慢速的，網路訪問也是慢的，資料庫通常使用這二者。
記憶體是很快的。使用本機快取可以避免網路和磁碟訪問的開銷。結合這些道理，然後，你想到了memcached，一個“分布式記憶體對象緩衝系統”，最初為基於Perl的部落格平台LiveJournal開發的。
如果你的程式不是分布在多個伺服器上，你可能並不需要memcached。單的緩衝方法——序列化資料然後將它儲存在一個臨時檔案中。例如 – 對每個請求可以消除很多多餘的工作。事實上，這是我們考慮協助我們的客戶最佳化他們的應用程式時，低掛水果的類型。
什麼是low-hanging fruit：
A fruit-bearing tree often contains some branches low enough for animals and humans to reach without much effort. The fruit contained on these lower branches may be not be as ripe or attractive as the fruit on higher limbs, but it is usually more abundant and easier to harvest. From this we get the popular expression “low hanging fruit”, which generally means selecting the easiest targets with the least amount of effort.
一種最簡易且最通用的將資料緩衝在記憶體的方式是使用APC中的共用類型輔助方法，APC是一個最初由我們的同事George Schlossnagle開發的緩衝系統，考慮如下例子： 複製代碼 代碼如下:<?php
$feed = apc_fetch('news');
if ($feed === FALSE) {
$feed = file_get_contents('http://example.org/news.xml');
// Store this data in shared memory for five minutes.
apc_store('news', $feed, 300);
}
// Do something with $feed.
?>

使用這種類型的緩衝，你不必在每一次請求時等待遠程伺服器發送Feed資料。一些延遲產生了 – 在這個例子中上限是五分鐘，但可以根據您的應用程式需要調整到接近即時。
8. 使用架構
所有決定都會有結果的，我們喜歡架構——事實上，CakePHP 和 Solar 的主要開發人員和我們一起在　OmniTI　工作——　但是使用一個架構並不會奇蹟般地使你在做的東西變得更好。
在十月份，我們的同事Paul Jones為HP Advent寫一了篇文章，叫做The Framework as Franchise ,在文章中他將架構與商業專營權相比較。他引用 Michael Gerber “電子神話再現”（”The E-Myth Revisited”）　一書中的建議：
　　格柏指出，運行一個成功的企業，企業家需要像他將要賣掉他的企業作為一個特許經營權的原型一樣行動。這是企業擁有者可以不親自參與每一項決策使企業運營的唯一方法。
（ Gerber notes that to run a successful business, the entrepreneur needs to act as if he is going to sell his business as a franchise prototype. It is the only way the business owner can make the business operate without him being personally involved in every decision.）
這是一個好的建議。無論你是打算使用架構或者定義你自己的標籤和慣例，從未來開發人員的角度來看價值是很重要的。
雖然我們很樂意給你一個放之四海而皆準的真理，延伸這個想法來表明一個架構總是合適的，並不是我們想做的事情。
如果你問我們是否應該使用一個架構，我們可以給出的最好的答案是，“這要看情況。”
9. 正確的使用錯誤抑制操作符
總是試著避免使用錯誤抑制操作符號。在前面的文章，作者表明:
　 @ 操作符是相當的慢的並且如果你需要寫高效能的代碼的話它會使得開銷很大。
錯誤抑制慢是因為在執行抑制語句前，PHP動態改變error_reporting等級到0　，然後然後立即將其還原。這是要開銷的。
更糟糕的是，使用錯誤抑制符使追蹤問題的根本原因很困難。
先前的文章使用如下例子來支援通過引用來給一個變數賦值的做法。。。（這句怎麼翻譯？我暈~~~ ）
The previous article uses the following example to support the practice of assigning a variable by reference when it is unknown if $albus is set: 複製代碼 代碼如下:<?php
$albert =& $albus;
?>

儘管這樣是工作的——對於現在——依靠奇怪的，未定義的行為，而對於為什麼這樣會工作有一個很好的理解是一個產生BUG的好方法。
因為　$albert 是引用了$albus的，後期對於$albus的修改將會同樣影響到$albert　.
一個更好的解決方案是使用isset(),加上大括弧： 複製代碼 代碼如下:<?php
if (!isset($albus)) {
$albert = NULL;
}
?>

給$albert　賦值NULL和給它賦一個不存在的引用的效果是相同的，但是更加明確了，大大提高了代碼的清晰度和避免的兩個變數之間的參考關聯性。
If you inherit code that uses the error suppression operator excessively, we've got a bonus tip for you. There is a new PECL extension called Scream that disables error suppression.
10. 使用 isset() 而不是 strlen()
這實際上是一個巧妙的方法，雖然前面的文章完全沒有解釋這個。下面是補充的例子： 複製代碼 代碼如下:<?php
if (isset($username[5])) {
// The username is at least six characters long.
}
?>

當你把字串當作一個數組時（荒野無燈：事實上，在C語言裡面，字元中通常以數組形式存在），字串裡的每一個字元都是數組的一個元素。通過檢測一個特定元素的存在與否，你可以檢測這個字串是否至少有那麼多的字元存在。（注意第一個字元是元素0，因此　$username[5]　是 $username中的第6個字元。）
這樣使用isset 比strlen稍快的原因是複雜的。簡單的解釋是，strlen() 是一個函數，而 isset() 是一個文法結構。通常來說，
調用一個函數是比使用語言結構的代價更為昂貴的。
關於作者：
Hi,我們是 Chris Shiflett和 Sean Coates. 我們都在 OmniTI (“the most important web company you've never heard of”)工作, blog about PHP and other stuff at shiflett.org and seancoates.com, curate PHP Advent, and do the Twitter thing as @shiflett and @coates.
譯　FROM : http://coding.smashingmagazine.com/2009/03/24/10-useful-php-tips-revisited/