Laravel9查询用户所属任务时预加载client解决N+1查询问题

问题根因 #

• 你当前通过hasManyThrough拿到的查询结果，本质是中间表JobUser模型的集合，并非直接的Job模型集合，这是你直接追加with('client')不生效的核心原因：JobUser模型本身没有定义client关联，预加载指令无法命中。
• 视图中访问路径为$item->job->client->title，实际触发两层N+1查询：先循环每条JobUser记录单独查询关联的Job，再循环每个Job单独查询关联的Client。你移除Client输出后仍有N+1提示，就是第一层$item->job的访问触发的。
• 现有代码存在两个隐性问题：一是getPlannerData方法中查询语句多写了一个等号存在语法错误；二是Job模型中client关联关系定义错误，Job持有client_id外键，应当使用belongsTo而非hasOne，手动指定外键虽能临时跑通逻辑，但会导致预加载、关联操作出现隐性bug。

修复步骤 #

1. 使用嵌套预加载修正查询逻辑 #

跨多层关联的预加载需要使用点语法，关联路径和视图中访问的路径完全一致即可。修正getPlannerData方法如下：

public function getPlannerData()
{
    $data = [];
    // 修正语法错误，通过嵌套预加载一次性加载job关联、以及job下的client关联
    $myJobs = auth()->user()->jobs()
        ->where('is_active', 1)
        ->with('job.client')
        ->get();
    $data['jobs'] = $myJobs;

    return $data;
}

上述写法全程仅执行3次SQL查询，完全消除循环内的逐条查询：

1. 查询当前用户关联的、is_active=1的JobUser中间表记录
2. 批量查询上述中间表记录关联的所有Job记录
3. 批量查询上述Job记录关联的所有Client记录

2. 修正Job模型的关联定义 #

将Job模型中的client关联修正为正确的从属关系，因为外键符合Laravel默认命名规则，可以省略手动指定的外键参数：

public function client()
{
    return $this->belongsTo(Client::class);
}

3. 可选：全局预加载配置 #

如果多个业务场景都需要通过JobUser访问Job和Client，可以直接在JobUser模型中配置默认预加载属性，无需每次查询手动调用with：

// app/Models/JobUser.php
/**
 * 默认自动加载的关联
 */
protected $with = ['job.client'];

残留N+1排查方法 #

如果修改后仍收到N+1提示，按以下步骤定位：

• 开启查询日志打印当前请求所有执行的SQL，定位循环中重复执行的语句：

  // 在控制器返回数据前加入调试代码
  \DB::enableQueryLog();
  // 执行业务查询逻辑
  dd(\DB::getQueryLog());
  

• 检查视图中是否存在其他未预加载的关联访问，例如$item->job->creator、$item->user这类未写入with规则的关联调用。

跨关联预加载通用规则 #

• 预加载的关联路径必须和实际访问路径完全一致，多层关联用.分隔，例如访问$a->b->c->d时，预加载规则写为with('b.c.d')。
• 直接查询目标模型时无需嵌套，例如直接查询Job模型并加载Client，直接写with('client')即可，和官方文档示例一致。
• 预加载支持追加查询条件，例如仅加载启用状态的客户，可以通过闭包自定义约束：

  ->with([
      'job.client' => function($query) {
          $query->where('is_active', 1);
      }
  ])
  

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