Laravel关联三张表的多条件搜索逻辑优化问题

问题背景 #

现有三张关联数据表：

• Reports表：字段 {id, date, employee_id}，模型关联：

public function employee()
{
    return $this->belongsTo(Employee::class, "employee_id", "id");
}

• Employees表：字段 {id, name, department_id}，模型关联：

public function department()
{
    return $this->belongsTo(Department::class);
}

public function report()
{
    return $this->hasMany(Report::class, "employee_id", "id");
}

• Departments表：字段 {id, name}，模型关联：

public function employee()
{
    return $this->hasMany(Employee::class);
}

需求是实现Reports表的多条件搜索，支持的条件包括：日期范围（start至end）、employee_id、department_id、report_status_id。当前使用的控制器代码存在逻辑问题，导致搜索结果不符合预期：例如仅选择「Incoming」部门时返回1484条结果，同时选择该部门和日期范围时却返回1512条结果。

问题根源 #

1. 未判断参数有效性：当用户未填写某个条件时，代码依然会将空参数加入查询，导致错误过滤（比如employee_id为空时，会查询employee_id = null的记录）。
2. 逻辑运算符优先级错误：orWhere直接跟在where之后，让department_id的条件和前面所有条件形成「或」关系，而非预期的「且」关系，导致查询范围意外扩大。

优化方案 #

方案1：DB查询构建器优化逻辑 #

通过动态判断参数是否存在来添加条件，同时确保所有条件为「且」关系：

$reportsQuery = DB::table('reports')
    ->leftJoin('employees', 'employees.id', '=', 'reports.employee_id')
    ->leftJoin('departments', 'departments.id', '=', 'employees.department_id');

// 日期范围条件（仅当起止日期都存在时添加）
if ($request->filled('start_date') && $request->filled('end_date')) {
    $reportsQuery->whereBetween('reports.date', [$request->start_date, $request->end_date]);
}

// employee_id条件（仅当参数存在时添加）
if ($request->filled('employee_id')) {
    $reportsQuery->where('reports.employee_id', $request->employee_id);
}

// report_status_id条件（仅当参数存在时添加）
if ($request->filled('report_status_id')) {
    $reportsQuery->where('reports.report_status_id', $request->report_status_id);
}

// department_id条件（仅当参数存在时添加）
if ($request->filled('department_id')) {
    $reportsQuery->where('employees.department_id', $request->department_id);
}

$reports = $reportsQuery->get();

方案2：利用Eloquent关联简化查询 #

采用Laravel原生Eloquent关联写法，自动处理表连接，代码更简洁易维护：

$reportsQuery = Report::with(['employee.department']);

// 日期范围条件
if ($request->filled('start_date') && $request->filled('end_date')) {
    $reportsQuery->whereBetween('date', [$request->start_date, $request->end_date]);
}

// employee_id条件
if ($request->filled('employee_id')) {
    $reportsQuery->where('employee_id', $request->employee_id);
}

// report_status_id条件
if ($request->filled('report_status_id')) {
    $reportsQuery->where('report_status_id', $request->report_status_id);
}

// department_id条件，通过关联查询实现
if ($request->filled('department_id')) {
    $reportsQuery->whereHas('employee', function ($query) use ($request) {
        $query->where('department_id', $request->department_id);
    });
}

$reports = $reportsQuery->get();

关键说明 #

• 使用$request->filled()判断参数是否存在且不为空，避免空参数干扰查询结果。
• 所有条件默认采用「且」关系，确保多条件组合时的逻辑一致性。
• Eloquent写法无需手动处理表连接，减少出错概率，同时支持关联数据预加载，提升性能。

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