Laravel 5.6 Eloquent方法未返回数据，关联与权限判断异常排查

我来帮你一步步排查这个Eloquent关联和方法的问题哈，主要有两个核心点需要修正：

1. 先修正hasManyThrough关联的参数错误 #

你当前的roles()关联参数顺序不对，hasManyThrough的第四个参数应该是中间表（UserRole）关联目标表（Role）的外键，而不是目标表的主键id。

正确的关联定义应该是这样的（假设你的user_roles表中关联角色的字段是role_id，如果你的字段名不同，替换成实际字段即可）：

/**
 * 用户的角色
 *
 * @return \Illuminate\Database\Eloquent\Relations\HasManyThrough
 */
public function roles() {
    // 参数顺序：目标模型，中间模型，中间模型的用户外键，中间模型的角色外键
    return $this->hasManyThrough(Role::class, UserRole::class, 'user_id', 'role_id');
}

小提示：如果你的主键或外键不是Laravel默认的命名规则，可能还需要补充第五、第六个参数，但按照常规命名的话，上面的写法就足够了。

2. 完善并优化hasRole方法的逻辑 #

你的hasRole方法目前未完成，而且存在两个可以优化的点：

问题1：误用集合查询而非数据库查询 #

$this->roles会先把用户所有角色加载到内存中再过滤，而$this->roles()是查询构建器，直接在数据库层面过滤，性能更好，尤其是用户角色较多的时候。

问题2：缺少完整的过滤条件与结果判断 #

假设你的roles表中存储角色标识的字段是short_name（替换成你实际的字段名），完整的hasRole方法应该是这样：

/**
 * 用户是否拥有指定角色？
 *
 * @param string|null $roleShort
 * @return bool
 */
public function hasRole($roleShort = null) {
    if (empty($roleShort)) {
        return false;
    }
    // 使用查询构建器直接在数据库查询，exists()直接返回布尔值，简洁高效
    return $this->roles()->where('short_name', $roleShort)->exists();
}

额外优化：支持批量检查角色 #

如果需要支持传入多个角色标识（比如hasRole(['admin', 'editor'])），可以扩展一下方法：

public function hasRole($roleShort = null) {
    if (empty($roleShort)) {
        return false;
    }

    $query = $this->roles();
    if (is_array($roleShort)) {
        return $query->whereIn('short_name', $roleShort)->exists();
    }

    return $query->where('short_name', $roleShort)->exists();
}

3. 验证关联是否正确 #

可以在Laravel Tinker里快速测试关联是否正常：

php artisan tinker
// 取一个测试用户
$user = User::find(1);
// 打印关联的角色，看是否正确加载
dd($user->roles);

如果能正确输出用户的角色列表，说明关联已经修复，再测试hasRole方法就能得到预期结果了。

内容的提问来源于stack exchange，提问作者TheRealPapa