使用PHP静态变量当缓存的方法
2024-05-04 22:23:26
供稿：网友
下面这个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的逻辑也写进去。这样函数调用减少，数据库查询也变成一次了，性能得到了提高。但是重置密码和发送邮件的功能将不能得到复用，并且违背了单一责任的原则，代码复杂度也提高了。
