Laravel 5.4 Eloquent hasMany关联下邮件审批状态校验方案

嘿，我来帮你把这个逻辑简化得更优雅！其实Laravel的模型访问器和集合方法刚好能完美解决你的问题，不用写那些繁琐的嵌套循环。

咱们可以把审批状态的判断逻辑封装到Email模型里，让每封邮件自动拥有这个属性，后续用起来超方便：

1. 在Email模型中添加访问器 #

直接在你的Email模型里定义一个approvalStatus的访问器，它会自动帮你处理所有逻辑：

public function getApprovalStatusAttribute()
{
    // 只处理审批类邮件，非审批类直接返回null或者你需要的默认值
    if ($this->review_group_type !== 'approval') {
        return null;
    }

    // 没有审核记录的情况：这里可以根据你的业务需求返回false或者'pending'这类状态
    if ($this->emailReviews->isEmpty()) {
        return false;
    }

    // 一行代码判断所有审核记录的approved是否全为true
    return $this->emailReviews->every(fn($review) => $review->approved === true);
}

2. 保持你的原有查询（无需改动） #

你原来的预加载查询已经很到位了，这样访问器调用关联数据时不会产生额外的数据库查询：

$data = Email::with('emailReviews')
    ->where('created_by', '=', $personId)
    ->get();

3. 直接使用属性就好！ #

现在你可以在任何地方直接访问这个属性，不用再手动处理循环了：

foreach ($data as $email) {
    if ($email->review_group_type === 'approval') {
        // 直接用$email->approvalStatus就行，true就是全通过，false就是未通过或者没审核记录
        echo $email->approvalStatus ? '所有审核已通过' : '存在未通过审核/待审核';
    }
}

为啥这个方案更优？ #

• 把业务逻辑封装在模型里，符合Laravel的Eloquent最佳实践，代码更整洁
• 用集合的every()方法替代手动循环，一行代码搞定全量判断
• 自动处理了无审核记录的边界情况，逻辑清晰不遗漏
• 访问器自动生效，不用在查询后再额外处理集合

如果之后你需要基于这个状态做查询筛选（比如只查所有审核通过的审批邮件），还可以用子查询的方式把这个状态写到查询语句里，但如果只是需要在模型对象上获取状态，访问器绝对是最简洁的方案。

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