Laravel Eloquent薪资筛选问题：实现OR逻辑且兼容多条件过滤

你的问题核心在于OR条件打破了查询的逻辑优先级——原来的写法会让后续的地理、关键词等条件变成和薪资条件平级的OR关系，完全不符合我们需要的「薪资满足条件 AND 其他条件也满足」的逻辑。正确的做法是把薪资的两个OR条件包裹在一个闭包里，让它们成为一个独立的逻辑块，再和其他条件做AND运算。

核心解决方案：用闭包包裹OR条件 #

把薪资筛选的部分改成下面这样，Laravel会自动为闭包内的条件添加括号，保证逻辑优先级正确：

$salary = explode(',',$request->salary);
$jobs = Job::where(function($query) use ($salary) {
    $query->whereBetween('min_salary', $salary)
          ->orWhereBetween('max_salary', $salary);
});

这样生成的SQL会是：

WHERE (min_salary BETWEEN ? AND ? OR max_salary BETWEEN ? AND ?) 
AND ... 其他筛选条件 ...

而原来的写法生成的SQL是：

WHERE min_salary BETWEEN ? AND ? 
OR max_salary BETWEEN ? AND ? 
AND ... 其他筛选条件 ...

因为AND的优先级高于OR，这会导致逻辑完全偏离预期——只要满足薪资其中一个条件，或者满足其他任意一个条件，就会被筛选出来，这显然不是你想要的结果。

完整修改后的处理函数 #

下面是调整后的完整代码，同时优化了几个性能细节：

if($request->has('offset')) {
    $salary = explode(',',$request->salary);
    // 核心修改：用闭包包裹薪资的OR条件，保证逻辑优先级
    $jobs = Job::where(function($query) use ($salary) {
        $query->whereBetween('min_salary', $salary)
              ->orWhereBetween('max_salary', $salary);
    });

    if ($request->has('lat')) {
        $sqlDistance = DB::raw (' ( 6371 * acos ( cos ( radians (' . $request->lat . ') ) * cos ( radians ( lat ) ) * cos ( radians ( lon ) - radians (' . $request->lon . ') ) + sin ( radians (' . $request->lat . ') ) * sin ( radians ( lat ) ) ) ) ');
        $jobs->when($sqlDistance != null, function ($query) use ($sqlDistance,$request){
            $query->whereHas('address', function ($subQuery) use ($sqlDistance,$request) {
                $subQuery->addSelect(DB::raw("{$sqlDistance} AS distance"));
                $subQuery->havingRaw("distance <= ?", [(int)$request->range]);
            });
        })
        ->with('company')
        ->with('address');
    }

    if ($request->has('key')) {
        $jobs->where('title', 'like', '%' . $request->key . '%');
    }

    if ($request->has('cat')) {
        $cat = explode(',',$request->cat);
        $jobs->whereIn('category_id', $cat);
    }

    if ($request->has('type')) {
        $type = explode(',',$request->type);
        $jobs->whereIn('type', $type);
    }

    if($request->has('hs')) {
        return view("General::browseJobs", [
            'jobs' => $jobs->orderBy('created_at','desc')->skip($request->offset * 2)->take(2)->get(),
            'count' => $jobs->count() // 优化：直接用查询构造器的count()，避免加载全量数据
        ]);
    }

    $view = view('General::loaders.jobs', [
        'jobs' => $jobs->orderBy('created_at','desc')->skip($request->offset * 2)->take(2)->get()
    ])->render();
    return response()->json(['html' => $view , 'count' => $jobs->count() ]); // 同样优化计数方式
}

return view("General::browseJobs", [
    'jobs' => Job::orderBy('created_at','desc')->take(2)->get(),
    'count' => Job::count() // 优化：用模型的count()代替count(Job::all())
]);

关于平均值筛选方法的说明 #

你尝试的计算平均值再筛选的方法不生效，是因为whereBetween只能作用于数据库中存在的物理字段，而计算出来的临时字段需要用having子句实现，但这种方式反而复杂，不如直接用闭包包裹OR条件来得简洁高效。

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