-
/**
- * 用户请求重置密码的接收器
- */
- 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的函数，它用来查询数据库并返回数据
复制代码
问题：这三个函数都同时使用checkUserExists这个函数来检查用户不存在，数据库查询了三次，这样带来了一些额外的开销。 如果要去掉三者之间任意一个checkUserExists，看上去是可能的。但是如果之后有某些功能要调用resetPassword或者sendEmail，用户不存在时，系统可能会发生错误。
另一种解决方法，将resetPassword的逻辑写到requestResetPassword里，再过一点，把sendEmail的逻辑也写进去。这样函数调用减少，数据库查询也变成一次了，性能得到了提高。但是重置密码和发送邮件的功能将不能得到复用，并且违背了单一责任的原则，代码复杂度也提高了。
不过，因为函数分离和复用性都很好，如果实际性能受到影响，可能考虑用缓存的方法减少数据库查询，改动它们共用的checkUserExists函数：
复制代码
也可以用同样的方法改动getUserInfo函数。
当代码的复用性提高时，想提高性能是很简单的，性能的瓶颈也很容易被发现和修改。
考虑对程序性能的影响，比如遍历数据时，可能为了复用而将遍历封装到一个函数中，并且多次使用它。 这些开销对项目根本没有预想中那样有太大的影响，微乎其微。 因此，重点可以放在提高代码的复用性和维护性方面，而不是纠结于浪费多这一点性能。 实际性能如果真的达不到要求，也可以权衡增加硬件配置。
|
