我目前正在学习WordPress核心,并且在我的学习中遇到了这个功能:
/**
* Determine if user is a site admin.
*
* @since 3.0.0
*
* @param int $user_id (Optional) The ID of a user. Defaults to the current user.
* @return bool True if the user is a site admin.
*/
function is_super_admin( $user_id = false ) {
if ( ! $user_id || $user_id == get_current_user_id() )
$user = wp_get_current_user();
else
$user = get_userdata( $user_id );
if ( ! $user || ! $user->exists() )
return false;
if ( is_multisite() ) {
$super_admins = get_super_admins();
if ( is_array( $super_admins ) && in_array( $user->user_login, $super_admins ) )
return true;
} else {
if ( $user->has_cap('delete_users') )
return true;
}
return false;
}
我仍然是一名初学程序员,所以这可能是一个愚蠢的问题,但首先if ... else语句中的逻辑对我来说似乎很奇怪:
if ( ! $user_id || $user_id == get_current_user_id() )
$user = wp_get_current_user();
else
$user = get_userdata( $user_id );
为什么我们需要这种逻辑比较?
$user_id == get_current_user_id()
在我看来,如果传入$ user_id,我们应该调用get_userdata($ user_id),如果传入NO $ user_id,那么我们应该调用wp_get_current_user(),如下所示:
$user = $user_id ? get_userdata($user_id) : wp_get_current_user();
$ user_id == get_current_user_id()的整个比较只会增加额外的工作,对吧?以这种方式编写函数有哪些可能的理由?
答案 0 :(得分:1)
! $ user_id || $ user_id == get_current_user_id() is statement为true所以如果使用user_id调用函数为false或者如果传递的user_id实际上是当前用户,则简单地加载当前用户会更有效。没有数据库查询。
否则,如果指定了user_id,则必须触发查询以获取其数据。
逻辑正常:)