Laravel集合map()返回数据异常问题排查与解决

1. 问题原因 #

• 核心问题是共享引用类型变量的赋值错误：你大概率在map()闭包外部定义了存储详情的数组（比如$details = []），或者在闭包内没有为每个广告实例初始化新的数组。
• PHP中数组属于引用类型，当你把这个共享数组赋值给每个Advert实例的details属性时，所有实例的details都会指向同一个内存地址的数组。
• 闭包内循环时，你看到的是当前循环对数组的修改结果，所以输出正确；但当整个map()执行完毕，数组已经被最后一个广告的详情覆盖，因此遍历原集合时所有details都变成了最后一个的内容。
• 此外，Advert是模型对象（引用类型），直接修改原集合中的对象属性而不做引用隔离，也会加剧这个问题。

2. 修复方法 #

方式一：在闭包内每次初始化独立的详情数组 #

在map()闭包内部为每个广告实例创建新的数组，彻底避免引用共享：

// 查询并预加载关联数据
$adverts = Advert::where('category_id', 1312)
    ->with('attributesValues.attribute')
    ->get()
    ->map(function ($advert) {
        // 每次循环都初始化新数组，确保每个广告的详情独立
        $details = [];
        foreach ($advert->attributesValues as $attrValue) {
            $details[$attrValue->attribute->name] = $attrValue->value;
        }
        // 为当前广告实例赋值独立的详情数组
        $advert->details = $details;
        return $advert;
    });

// 遍历处理后的集合，此时每个details都是独立的
foreach ($adverts as $advert) {
    var_dump($advert->details);
}

方式二：克隆模型实例避免修改原对象引用 #

如果不想修改原始的Advert模型实例，可以在map()内克隆实例后再赋值，确保每个实例的属性完全独立：

$adverts = Advert::where('category_id', 1312)
    ->with('attributesValues.attribute')
    ->get()
    ->map(function ($advert) {
        $details = [];
        foreach ($advert->attributesValues as $attrValue) {
            $details[$attrValue->attribute->name] = $attrValue->value;
        }
        // 克隆原模型实例，隔离引用
        $newAdvert = clone $advert;
        $newAdvert->details = $details;
        return $newAdvert;
    });

额外提示 #

如果不需要保留原集合的原始数据，也可以用transform()方法代替map()，它会直接修改原集合，不过核心还是要确保每个循环的详情数组是独立初始化的。

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