Laravel中foreach循环内whereNotIn条件失效问题求助

核心问题 #

你说的“查询失败”并非SQL执行报错，而是当对应产品的status为0时，whereNotIn('status', [0])会过滤掉该产品，导致$productre返回null；而移除该条件后，无论状态如何都会返回产品，所以看起来“正常”。

优化点与修正方案 #

1. 单个值过滤无需用whereNotIn
   针对单个值的排除逻辑，用where('status', '!=', 0)或whereNot('status', 0)更直观，效果完全一致：

$productre = Product::where('id', $pro->product_id)
    ->where('status', '!=', 0)
    ->first();

2. 解决N+1查询性能问题（关键优化）
   你当前的嵌套循环查询会产生大量重复SQL，效率极低。可以通过Laravel的关联预加载直接过滤出符合条件的产品：
   首先在ProductCategory模型中定义关联方法（根据你的实际中间表字段调整）：

// 假设CatProduct是分类与产品的关联中间表
public function activeProducts()
{
    return $this->hasManyThrough(
        Product::class,
        CatProduct::class,
        'category_id', // 中间表关联分类的字段
        'id', // 产品表主键
        'id', // 分类表主键
        'product_id' // 中间表关联产品的字段
    )->where('products.status', '!=', 0);
}

然后直接查询获取结果：

// 单个分类ID查询用first()替代get()更合理
$category = ProductCategory::with('activeProducts')
    ->where('id', $categoryDetials->id)
    ->first();

// 直接遍历状态正常的产品
foreach ($category->activeProducts as $product) {
    // 处理业务逻辑
}

3. 原循环写法的容错处理
   如果坚持使用原循环逻辑，需要判断查询结果是否存在，避免null引发后续问题：

foreach ($pr as $value) {
    foreach ($value->catproduct as $pro) {
        $productre = Product::where('id', $pro->product_id)
            ->where('status', '!=', 0)
            ->first();
        if ($productre) {
            // 仅处理状态不为0的产品
        }
    }
}

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