Laravel多商品折扣模型foreach失效，仅获取最后一行数据

核心问题：循环覆盖导致仅保留最后一行折扣值 #

你的foreach循环中每次都会重新赋值$p_price，且未判断当前折扣项是否对应目标商品，循环结束后自然只剩最后一次循环的计算结果。

其他关键问题梳理 #

• 未关联当前商品ID：遍历所有typeItems时，未筛选出与传入$p_id匹配的折扣项，无关商品的折扣也被纳入计算。
• 冗余错误代码：$orderdetail和$discprice完全未被使用，且pluck('value')返回集合，explode操作无意义还可能引发错误。
• 空值调用风险：当$discount为null时，直接调用$discount->value会抛出致命错误，必须先判断是否存在。
• 用户ID查询不严谨：用like匹配逗号分隔的user_id字符串，会出现误匹配（如ID=2会匹配到ID=12、22的记录），改用FIND_IN_SET更准确。
• 条件逻辑错误：!is_null($multidiscount) || is_null($discount)的逻辑不符合折扣优先级需求，应调整为多折扣优先、单折扣兜底的判断逻辑。

修复后的完整代码 #

public static function discount($p_id)
{
    // 获取当前用户的单折扣
    $discount = Discount::where('user_id', Auth::user()->id)->first();
    // 获取目标商品（找不到则抛出异常，避免后续空值操作）
    $product = Product::findOrFail($p_id);
    // 初始化商品价格为原价
    $finalPrice = $product->price;

    // 用FIND_IN_SET准确匹配逗号分隔的user_id
    $multiDiscount = MultipleDiscount::whereRaw('FIND_IN_SET(?, user_id)', [Auth::user()->id])
        ->with('typeItems')
        ->first();

    // 优先处理多折扣
    if ($multiDiscount) {
        // 筛选当前商品对应的多折扣项（注意子表字段是product_id，你写的"product it"是笔误）
        $targetDiscount = $multiDiscount->typeItems->firstWhere('product_id', $p_id);

        if ($targetDiscount) {
            // 计算多折扣后价格
            if ($targetDiscount->type === 'percentage') {
                $discountAmount = $product->price * ($targetDiscount->value / 100);
                $finalPrice = $product->price - $discountAmount;
            } else {
                // 支持固定金额折扣（按需补充）
                $finalPrice = $product->price - $targetDiscount->value;
            }
        }
    } 
    // 多折扣未匹配时，处理单折扣
    elseif ($discount) {
        if ($discount->type === 'percentage') {
            $discountAmount = $product->price * ($discount->value / 100);
            $finalPrice = $product->price - $discountAmount;
        } else {
            $finalPrice = $product->price - $discount->value;
        }
    }

    return $finalPrice;
}

额外优化建议 #

1. 修正多折扣子表的字段名：你写的product it是笔误，实际应改为product_id，确保关联匹配正确。
2. 规范关联关系：在MultipleDiscount模型中定义关联方法，确保typeItems关联正确的子表模型：

// MultipleDiscount模型内
public function typeItems()
{
    return $this->hasMany(MultiDiscountType::class, 'discount_id');
}

3. 优化用户存储方式：建议将逗号分隔的user_id改为多对多关联表，避免FIND_IN_SET的性能问题，同时更符合数据库设计规范。

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