Laravel跨多层模型关联实现用户按城市区域搜索过滤问题咨询

• 逻辑错误：按城市筛选时，你先查出城市下所有区域，再循环调用area作用域，相当于给查询加了N个AND形式的关联条件，要求用户同时绑定该城市下所有区域才能被匹配到。正常业务逻辑下，用户只要绑定该城市下任意一个区域就属于该城市用户，现有写法会漏掉绝大多数符合条件的用户。
• 性能冗余：每次按城市筛选都要先全量查询对应城市下的所有Area记录，再循环拼接多个whereHas子查询，生成的SQL会非常臃肿，城市下区域数量越多查询性能越差。
• 复用性差：城市筛选逻辑直接写在控制器中，其他业务场景如果需要按城市筛选用户，只能重复写相同逻辑。
• 可读性差：用一层无意义的外层where闭包包裹筛选条件，if嵌套的写法也不如Laravel内置方法简洁。

第一步：优化User模型的关联作用域 #

直接在模型中封装好区域、城市的筛选逻辑，不需要提前查询Area表数据，利用关联表的外键直接做条件过滤：

// App\Models\User.php
public function scopeArea($query, $search)
{
    // 兼容单区域ID、多区域ID数组传参
    $areaIds = is_array($search) ? $search : [$search];
    return $query->whereHas('areas', function ($q) use ($areaIds) {
        $q->whereIn('area_id', $areaIds);
    });
}

public function scopeCity($query, $cityId)
{
    // 直接通过关联的areas表的city_id字段过滤，无需提前查Area数据
    return $query->whereHas('areas', function ($q) use ($cityId) {
        $q->where('city_id', $cityId);
    });
}

前提是你的areas表本身带有city_id外键字段，这是Area和City一对多关联的必要字段，正常建表都会存在。

第二步：简化控制器的搜索逻辑 #

用Laravel内置的when方法替代if条件判断，去掉冗余的闭包包裹，用filled方法过滤空值参数：

// 控制器search方法
public function search(Request $request)
{
    return User::where('status', 1)
        ->where('type', User::TYPE_INSTRUCTOR)
        ->when($request->filled('city'), function ($query) use ($request) {
            $query->city($request->city);
        })
        ->when($request->filled('area'), function ($query) use ($request) {
            $query->area($request->area);
        })
        ->orderBy('id')
        ->paginate(8);
}

• 修复了原逻辑的条件匹配错误，按城市筛选时只要用户关联了该城市下任意一个区域就会被正确返回，符合业务预期。
• 减少了一次查询Area列表的SQL请求，同时避免了多个重复whereHas子查询带来的性能损耗，数据量越大优化效果越明显。
• 筛选逻辑全部封装在模型层，其他业务场景需要相同筛选规则时直接调用作用域即可，不用重复编码。
• 代码结构更简洁，when方法会自动判断请求参数是否有效，只有参数存在且非空时才会执行对应筛选逻辑，可读性和可维护性更高。
• 现有写法默认是多条件AND筛选，也就是同时传城市和区域参数时，会返回同时符合两个条件的用户，符合绝大多数搜索场景的需求，如果需要OR逻辑只要调整闭包结构即可。

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