Laravel多对多关联查询优化：角色与权限多条件搜索问题

你的代码核心问题是搜索条件的逻辑分组错误：直接拼接whereHas和orWhere会导致后续的状态、日期筛选条件被逻辑或覆盖，比如即使设置了状态为1，只要满足搜索条件的角色都会被返回，完全忽略状态筛选规则。下面是修复后的优化实现：

优化后的代码 #

use Illuminate\Database\Eloquent\Builder;
use Carbon\Carbon;

$roles = Role::with(['permissions', 'users'])
    // 搜索逻辑：将角色名匹配、权限匹配封装为独立逻辑组
    ->when($request->search_term, function (Builder $query) use ($request) {
        $searchTerm = "%{$request->search_term}%";
        return $query->where(function (Builder $subQuery) use ($searchTerm) {
            $subQuery->where('name', 'like', $searchTerm)
                ->orWhereHas('permissions', function (Builder $permissionQuery) use ($searchTerm) {
                    $permissionQuery->where('name', 'like', $searchTerm);
                });
        });
    })
    // 状态筛选
    ->when($request->status !== null, function (Builder $query) use ($request) {
        $statuses = isIntegerOrCommaSeparatedInteger($request->status, 'statuses');
        return $query->whereIn('active', $statuses);
    })
    // 日期范围筛选
    ->when($request->filled(['start_date', 'end_date']), function (Builder $query) use ($request) {
        return $query->whereBetween('created_at', [
            Carbon::parse($request->start_date)->startOfDay(),
            Carbon::parse($request->end_date)->endOfDay(),
        ]);
    })
    ->get();

关键改进点 #

• 逻辑分组修复：将搜索的两个条件（角色名匹配、权限匹配）包裹在where(function(){...})闭包中，确保这两个条件是或的关系，而整个搜索组与后续的状态、日期筛选是与的关系，彻底解决原代码逻辑混乱的问题。
• 统一when语法：替代原有的if判断，保持链式调用的简洁性和可读性，符合Laravel最佳实践。
• 日期处理优化：
  • 用$request->filled(['start_date', 'end_date'])替代直接判断参数是否存在，更严谨（会检查参数是否存在且不为空）。
  • 去掉DB::raw('date(created_at)')，直接配合startOfDay()和endOfDay()处理日期范围，避免原生SQL依赖，同时确保包含结束日期当天的所有数据（比如23:59:59的记录）。
• 安全防护：通过Laravel查询构建器自动处理参数绑定，避免直接拼接字符串带来的SQL注入风险。

进阶优化建议 #

如果项目数据量较大，可以考虑：

• 给roles.name和permissions.name字段添加全文索引，like %xxx%的模糊查询在数据量多时性能较差，全文索引可大幅提升搜索速度。
• 将搜索逻辑封装到模型Scope中，进一步简化控制器代码：

// Role模型中添加Scope
public function scopeSearch($query, $term)
{
    $searchTerm = "%{$term}%";
    return $query->where(function ($subQuery) use ($searchTerm) {
        $subQuery->where('name', 'like', $searchTerm)
            ->orWhereHas('permissions', function ($permissionQuery) use ($searchTerm) {
                $permissionQuery->where('name', 'like', $searchTerm);
            });
    });
}

// 控制器中调用
$roles = Role::with(['permissions', 'users'])
    ->when($request->search_term, fn($q) => $q->search($request->search_term))
    // ...其他条件
    ->get();

内容的提问来源于stack exchange，提问作者Mofoluwasho Kolawole Jimoh