下面这个PHP的代码实例,功能是帮助用户重置密码,requestResetPassword是接收用户重置密码的请求并且做了相应的检查。为了更好的复用性,我将重置密码的操作单独分配到一个新的resetPassword的函数,更改完密码的后再调用sendEmail向用户发送一封通知邮件。
/**<BR> * 用户请求重置密码的接收器<BR> */<BR>function requestResetPassword() {<BR> //检查用户是否存在<BR> if( !checkUserExists( $_GET['userid'] ) ) {<BR> exit('抱歉,用户不存在,请确认用户帐号。');<BR> }<BR> resetPassword( $_GET['userid'] );<BR> //最后向用户发送一封邮件<BR> sendEmail( $_GET['userid'], '重置密码成功', '新的密码是xxxx' );<BR> exit('新密码已经发送到你的邮箱。');<BR>}<br><br>/**<BR> * 帮助用户重置密码<BR> */<BR>function resetPassword( $userid ) {<BR> //检查用户是否存在<BR> if( !checkUserExists( $userid ) ) {<BR> return false;<BR> }<br><br> //进行重置用户密码的操作<BR> //略...<BR> return true;<BR>}<br><br>/**<BR> * 向用户发送一封邮件<BR> */<BR>function sendEmail( $userid, $title, $content ) {<BR> //检查用户是否存在<BR> if( !checkUserExists( $userid ) ) {<BR> return false;<BR> }<br><br> //发送邮件操作<BR> //略...<BR> return true;<BR>}<br><br>/**<BR> * 检查某个用户是否存在<BR> */<BR>function checkUserExists( $userid ) {<BR> $user = getUserInfo( $userid );<BR> return !empty( $user );<BR>}<br><br>/**<BR> * 获取某个用户的数据<BR> */<BR>function getUserInfo( $userid ) {<BR> //假设我有一个query的函数,它用来查询数据库并返回数据<BR> $user = query( "SELECT * FROM `user` WHERE `uid`=" . intval( $userid ) );<BR> return is_array( $user ) ? $user : array() ;<BR>}<BR>
现在问题是,这三个函数都同时使用checkUserExists这个函数来检查用户不存在,数据库查询了三次,这样带来了一些额外的开销。
如果要去掉三者之间任意一个checkUserExists,看上去是可能的。但是如果之后有某些功能要调用resetPassword或者sendEmail,用户不存在时,系统可能会发生错误。
还有一个解决方法是,将resetPassword的逻辑写到requestResetPassword里,再过一点,把sendEmail的逻辑也写进去。这样函数调用减少,数据库查询也变成一次了,性能得到了提高。但是重置密码和发送邮件的功能将不能得到复用,并且违背了单一责任的原则,代码复杂度也提高了。
不过,因为函数分离和复用性都很好,如果实际性能受到影响,可能考虑用缓存的方法减少数据库查询,我改动了它们共用的checkUserExists函数:
/**<BR> * 检查某个用户是否存在<BR> */<BR>function checkUserExists( $userid ) {<BR> //增加一个缓存,用以记录检查用户的结果<BR> static $cache = array();<br><br> //检查当前用户是否已经检查过一次<BR> if( isset( $cache[ $userid ] ) ) {<BR> return $cache[ $userid ];<BR> }<br><br> $user = getUserInfo( $userid );<BR> //把结果记录到缓存中<BR> $cache[ $userid ] = !empty( $user );<br><br> return $cache[ $userid ];<BR>}<BR>
也可以用同样的方法改动getUserInfo函数。
这里可以看到,当代码的复用性提高时,想提高性能是很简单的,性能的瓶颈也很容易被发现和修改。
尽管这个例子对性能影响还不够大¥本文来源gaodai#ma#com搞@@代~&码网^搞gaodaima代码,还有一些影响更大的,比如说遍历,我可能为了复用而将遍历封装到一个函数中,并且多次使用它。这些开销对我的项目根本没有预想中那样有太大的影响,或者说是微乎其微的。所以我更愿意把时间花在如何提高代码的复用性和维护性方面,而不是纠结于浪费多这一点性能。实际性能如果真的达不到要求,也可以权衡增加硬件配置。