下面这个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函数。
这里可以看到,当代码的复用性提高时,想提高性能是很简单的,性能的瓶颈也很容易被发现和修改。
尽管这个例子对性能影响还不够大,还有一些影响更大的,比如说遍历,我可能为了复用而将遍历封装到一个函数中,并且多次使用它。这些开销对我的项目根本没有预想中那样有太大的影响,或者说是微乎其微的。所以我更愿意把时间花在如何提高代码的复用性和维护性方面,而不是纠结于浪费多这一点性能。实际性能如果真的达不到要求,也可以权衡增加硬件配置。
复制代码 代码如下:
/**
* 用户请求重置密码的接收器
*/
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函数。
这里可以看到,当代码的复用性提高时,想提高性能是很简单的,性能的瓶颈也很容易被发现和修改。
尽管这个例子对性能影响还不够大,还有一些影响更大的,比如说遍历,我可能为了复用而将遍历封装到一个函数中,并且多次使用它。这些开销对我的项目根本没有预想中那样有太大的影响,或者说是微乎其微的。所以我更愿意把时间花在如何提高代码的复用性和维护性方面,而不是纠结于浪费多这一点性能。实际性能如果真的达不到要求,也可以权衡增加硬件配置。
广告合作:本站广告合作请联系QQ:858582 申请时备注:广告合作(否则不回)
免责声明:本站资源来自互联网收集,仅供用于学习和交流,请遵循相关法律法规,本站一切资源不代表本站立场,如有侵权、后门、不妥请联系本站删除!
免责声明:本站资源来自互联网收集,仅供用于学习和交流,请遵循相关法律法规,本站一切资源不代表本站立场,如有侵权、后门、不妥请联系本站删除!
暂无评论...
稳了!魔兽国服回归的3条重磅消息!官宣时间再确认!
昨天有一位朋友在大神群里分享,自己亚服账号被封号之后居然弹出了国服的封号信息对话框。
这里面让他访问的是一个国服的战网网址,com.cn和后面的zh都非常明白地表明这就是国服战网。
而他在复制这个网址并且进行登录之后,确实是网易的网址,也就是我们熟悉的停服之后国服发布的暴雪游戏产品运营到期开放退款的说明。这是一件比较奇怪的事情,因为以前都没有出现这样的情况,现在突然提示跳转到国服战网的网址,是不是说明了简体中文客户端已经开始进行更新了呢?
更新日志
2024年11月26日
2024年11月26日
- 凤飞飞《我们的主题曲》飞跃制作[正版原抓WAV+CUE]
- 刘嘉亮《亮情歌2》[WAV+CUE][1G]
- 红馆40·谭咏麟《歌者恋歌浓情30年演唱会》3CD[低速原抓WAV+CUE][1.8G]
- 刘纬武《睡眠宝宝竖琴童谣 吉卜力工作室 白噪音安抚》[320K/MP3][193.25MB]
- 【轻音乐】曼托凡尼乐团《精选辑》2CD.1998[FLAC+CUE整轨]
- 邝美云《心中有爱》1989年香港DMIJP版1MTO东芝首版[WAV+CUE]
- 群星《情叹-发烧女声DSD》天籁女声发烧碟[WAV+CUE]
- 刘纬武《睡眠宝宝竖琴童谣 吉卜力工作室 白噪音安抚》[FLAC/分轨][748.03MB]
- 理想混蛋《Origin Sessions》[320K/MP3][37.47MB]
- 公馆青少年《我其实一点都不酷》[320K/MP3][78.78MB]
- 群星《情叹-发烧男声DSD》最值得珍藏的完美男声[WAV+CUE]
- 群星《国韵飘香·贵妃醉酒HQCD黑胶王》2CD[WAV]
- 卫兰《DAUGHTER》【低速原抓WAV+CUE】
- 公馆青少年《我其实一点都不酷》[FLAC/分轨][398.22MB]
- ZWEI《迟暮的花 (Explicit)》[320K/MP3][57.16MB]