Laravel预加载自定义关联报addEagerConstraints()空值错误的解决

报错本质 #

这个报错的原因一点都不复杂：Laravel 的 with() 预加载逻辑，只认返回标准关联关系实例（比如hasOne/belongsTo这类）的模型方法。
你现在写的userVoteThread根本不是关联定义方法——你在里面直接写数据库查询、if判断、给模型赋值属性，压根没返回任何可供框架预加载的对象，框架找了一圈拿不到预期的关联实例，自然就抛出Call to a member function addEagerConstraints() on null的错误。
顺带说，你最初用foreach循环逐条查用户投票的代码，是典型的N+1查询反模式：哪怕分页只拿4条线程的数据，最多也要额外跑8次Vote表查询，性能是有问题的。你想改成预加载的优化方向完全没错，只是关联方法的写法完全踩了框架的规则。

可落地的优化实现方案 #

1. 修正模型内的关联定义 #

把之前错误的userVoteThread方法删掉，替换成可以被预加载识别的标准关联：

// Thread 模型文件内
// 先定义Thread和Vote的基础一对多关联
public function votes()
{
    return $this->hasMany(Vote::class);
}

// 定义针对当前登录用户的专属投票关联，注意必须返回关联查询实例
public function userVote()
{
    return $this->hasOne(Vote::class)
        ->where('user_id', auth()->id());
}

这里别踩坑：关联方法里只负责定义查询规则、返回查询构造器，绝对不能在方法内部直接写终结查询的逻辑（比如exists()、直接return查询结果），也不要在关联方法里直接给模型属性赋值，框架的预加载机制根本识别不了这种写法。

2. 控制器内正确调用预加载 #

// Thread 控制器内
$threads = Thread::query()
    ->orderBy('created_at', 'desc')
    ->with(['communities', 'author', 'userVote']) // 直接预加载上面定义的userVote关联
    ->withCount(['replies', 'upvotes', 'downvotes'])
    ->paginate(4);

3. 批量处理自定义属性 #

预加载完成后，不需要再循环查数据库，直接拿已经加载好的关联结果补需要的自定义字段就行，全程零额外数据库查询：

// 处理分页结果集，补充前端需要的标记字段
$threads->getCollection()->transform(function ($thread) {
    $thread->user_has_voted = $thread->userVote ? 'true' : 'false';
    $thread->user_vote_type = $thread->userVote?->vote_type;
    return $thread;
});

return $threads;

方案实际效果 #

相比你最初的foreach循环写法，这套实现只需要在预加载阶段额外跑1次Vote表的查询，就可以拿到所有4条thread对应的当前用户投票记录，彻底解决了原来的N+1查询性能问题，同时也兼容了未登录用户的场景（auth()->id()会返回null，关联查询自然匹配不到任何投票记录，返回null值）。

内容的提问来源于stack exchange，提问作者David Encina Martínez