下面這個PHP的代碼實例,功能是幫助用戶重置密碼,requestResetPassword是接收用戶重置密碼的請求而且作了相應的檢查。爲了更好的複用性,我將重置密碼的操做單獨分配到一個新的resetPassword的函數,更改完密碼的後再調用sendEmail向用戶發送一封通知郵件。php
/** * 用戶請求重置密碼的接收器 */ 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函數。
這裏能夠看到,當代碼的複用性提升時,想提升性能是很簡單的,性能的瓶頸也很容易被發現和修改。
儘管這個例子對性能影響還不夠大,還有一些影響更大的,好比說遍歷,我可能爲了複用而將遍歷封裝到一個函數中,而且屢次使用它。這些開銷對個人項目根本沒有預想中那樣有太大的影響,或者說是微乎其微的。因此我更願意把時間花在如何提升代碼的複用性和維護性方面,而不是糾結於浪費多這一點性能。實際性能若是真的達不到要求,也能夠權衡增長硬件配置。緩存