|
|
復制代碼 代碼如下:
/**
* 用戶請求重置密碼的接收器
*/
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函數。
這里可以看到，當代碼的復用性提高時，想提高性能是很簡單的，性能的瓶頸也很容易被發現和修改。
盡管這個例子對性能影響還不夠大，還有一些影響更大的，比如說遍歷，我可能為了復用而將遍歷封裝到一個函數中，并且多次使用它。這些開銷對我的項目根本沒有預想中那樣有太大的影響，或者說是微乎其微的。所以我更愿意把時間花在如何提高代碼的復用性和維護性方面，而不是糾結于浪費多這一點性能。實際性能如果真的達不到要求，也可以權衡增加硬件配置。
php技術：使用PHP靜態變量當緩存的方法，轉載需保留來源！
鄭重聲明：本文版權歸原作者所有，轉載文章僅為傳播更多信息之目的，如作者信息標記有誤，請第一時間聯系我們修改或刪除，多謝。
