Laravel关联查询优化：属性为空时返回商品数据的方案咨询

针对你的需求——当$attribute_id和$attribute_value都为空时返回所有商品，有属性参数时过滤符合条件的商品——我有几种更优雅的查询层面解决方案，同时也会帮你理清原有代码的潜在问题：

问题根源 #

你之前修改后的代码，在两个参数都为空时，whereHas依然会生效：它会自动筛选出至少拥有一条属性记录的商品，如果你的商品表中存在没有关联属性的商品，这部分商品会被意外排除。要实现“返回所有商品”的目标，需要在参数全为空时完全移除whereHas约束。

方案一：条件式添加whereHas约束 #

通过判断是否有属性参数，决定是否启用关联过滤逻辑，这样参数全空时直接查询所有商品：

// 初始化基础查询，预加载属性关联
$query = Product::with('attributes');

// 只有当至少一个属性参数不为空时，才添加关联过滤
if (!empty($attribute_id) || !empty($attribute_value)) {
    $query->whereHas('attributes', function ($subQuery) use ($attribute_id, $attribute_value) {
        if (!empty($attribute_id)) {
            $subQuery->whereIn('attribute_id', $attribute_id);
        }
        if (!empty($attribute_value)) {
            $subQuery->whereIn('value', $attribute_value);
        }
    });
}

// 执行分页查询
$products = $query->paginate(10);

方案二：用when()方法简化动态逻辑 #

Laravel的when()方法可以根据条件动态添加查询约束，让代码更简洁易读：

$products = Product::with('attributes')
    // 只有当有属性参数时，才添加whereHas过滤
    ->when(!empty($attribute_id) || !empty($attribute_value), function ($query) use ($attribute_id, $attribute_value) {
        return $query->whereHas('attributes', function ($subQuery) use ($attribute_id, $attribute_value) {
            // 分别对两个参数做条件判断
            $subQuery->when(!empty($attribute_id), fn($q) => $q->whereIn('attribute_id', $attribute_id))
                     ->when(!empty($attribute_value), fn($q) => $q->whereIn('value', $attribute_value));
        });
    })
    ->paginate(10);

额外提示 #

注意你的模型关联定义里有个小笔误：'product_id '后面多了一个空格，建议修正为：

public function attributes() {
    return $this->hasMany(ProductsAttribute::class, 'product_id');
}

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