在Eloquent模型关联中使用where子句是否合理？需重构吗？

问题分析 #

这段belongsToMany关联的写法存在逻辑错误，完全不符合Laravel的关联规范：

• Laravel中belongsToMany的参数顺序是：关联模型类、中间表名、当前模型在中间表的外键、关联模型在中间表的外键。
• 你代码里把第三个参数传成了id（当前模型的主键），第四个参数是user_id，这会让Eloquent用当前模型的id去匹配中间表的id字段（中间表自身的主键），而不是匹配关联用户的user_id字段。这种写法会导致关联查询结果完全错误，甚至查不到任何符合业务逻辑的数据。

正确的重构写法 #

假设当前模型是User，中间表user_invites包含user_id（关联用户表）和invite_id（关联邀请表）两个字段，正确的关联定义应该是：

public function invites(): BelongsToMany
{
    return $this->belongsToMany(
        Invite::class,
        'user_invites',
        'user_id', // 当前模型（User）在中间表的外键
        'invite_id' // 关联模型（Invite）在中间表的外键
    )->where('is_active', 1);
}

为什么不能忽略PHPStan警告 #

• PHPStan的警告不是误报，它是在提示你参数不符合Laravel框架的定义，本质是代码逻辑存在bug。忽略警告只会掩盖运行时的错误，后续业务依赖该关联数据时必然出现异常。
• 这种不符合规范的写法会增加维护成本，其他开发者接手时会因违反Laravel常规约定而困惑，甚至误改代码引发更多问题。

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