本文介紹如何使用PHP靜態變數當緩衝,減少代碼重複執行,最佳化程式效能
下面這個PHP的代碼執行個體,功能是協助使用者重設密碼,requestResetPassword是接收使用者重設密碼的請求並且做了相應的檢查。為了更好的複用性,我將重設密碼的操作單獨分配到一個新的resetPassword的函數,更改完密碼的後再調用sendEmail向使用者發送一封通知訊息。
/** * 使用者請求重設密碼的接收器 */function requestResetPassword() { //檢查使用者是否存在 if( !checkUserExists( $_GET['userid'] ) ) { exit('抱歉,使用者不存在,請確認使用者帳號。'); } resetPassword( $_GET['userid'] ); //最後向使用者發送一封郵件 sendEmail( $_GET['userid'], '重設密碼成功', '新的密碼是xxxx' ); exit('新密碼已經發送到你的郵箱。');}/** * 協助使用者重設密碼 */function resetPassword( $userid ) { //檢查使用者是否存在 if( !checkUserExists( $userid ) ) { return false; } //進行重設使用者密碼的操作 //略... return true;}/** * 向使用者發送一封郵件 */function sendEmail( $userid, $title, $content ) { //檢查使用者是否存在 if( !checkUserExists( $userid ) ) { return false; } //發送郵件操作 //略... return true;}/** * 檢查某個使用者是否存在 */function checkUserExists( $userid ) { $user = getUserInfo( $userid ); return !empty( $user );}/** * 擷取某個使用者的資料 */function getUserInfo( $userid ) { //假設我有一個query的函數,它用來查詢資料庫並返回資料 $user = query( "SELECT * FROM `user` WHERE `uid`=" . intval( $userid ) ); return is_array( $user ) ? $user : array() ;}
現在問題是,這三個函數都同時使用checkUserExists這個函數來檢查使用者不存在,資料庫查詢了三次,這樣帶來了一些額外的開銷。
如果要去掉三者之間任意一個checkUserExists,看上去是可能的。但是如果之後有某些功能要調用resetPassword或者sendEmail,使用者不存在時,系統可能會發生錯誤。
還有一個解決方案是,將resetPassword的邏輯寫到requestResetPassword裡,再過一點,把sendEmail的邏輯也寫進去。這樣函數調用減少,資料庫查詢也變成一次了,效能得到了提高。但是重設密碼和發送郵件的功能將不能得到複用,並且違背了單一責任的原則,代碼複雜度也提高了。
不過,因為函數分離和複用性都很好,如果實際效能受到影響,可能考慮用緩衝的方法減少資料庫查詢,我改動了它們共用的checkUserExists函數:
/** * 檢查某個使用者是否存在 */function checkUserExists( $userid ) { //增加一個緩衝,用以記錄檢查使用者的結果 static $cache = array(); //檢查目前使用者是否已經檢查過一次 if( isset( $cache[ $userid ] ) ) { return $cache[ $userid ]; } $user = getUserInfo( $userid ); //把結果記錄到緩衝中 $cache[ $userid ] = !empty( $user ); return $cache[ $userid ];}
也可以用同樣的方法改動getUserInfo函數。
這裡可以看到,當代碼的複用性提高時,想提高效能是很簡單的,效能的瓶頸也很容易被發現和修改。
儘管這個例子對效能影響還不夠大,還有一些影響更大的,比如說遍曆,我可能為了複用而將遍曆封裝到一個函數中,並且多次使用它。這些開銷對我的項目根本沒有預想中那樣有太大的影響,或者說是微乎其微的。所以我更願意把時間花在如何提高代碼的複用性和維護性方面,而不是糾結於浪費多這一點效能。實際效能如果真的達不到要求,也可以權衡增加硬體設定。