Laravel Seeder条件判断不生效 所有hasAddresses字段被更新为true如何解决

你的代码存在3个核心错误，直接导致全表hasAddresses都被更新为true：

• foreach循环内错误覆盖了$area变量：遍历$areas得到单个区域对象$area后，你又执行了$area=Area::get('id')，直接把当前区域对象替换成了所有区域的id集合，完全丢失了当前遍历的区域标识。
• 更新语句没有加where条件：DB::table('areas')->update()不带条件时会更新整张表的所有记录，只要任意一次判断$exists为真，全表的hasAddresses都会被设为true。
• 关联存在性判断逻辑错误：Address::exists()的传参和使用方式不符合Laravel的语法规范，无法正确判断当前区域是否有关联地址。

方案1：高效批量更新（推荐，无需循环遍历） #

仅需执行2次SQL，不管数据量多大性能都远高于循环写法：

class AddhasAdressesSeeder extends Seeder
{
    /**
     * Run the database seeds.
     *
     * @return void
     */
    public function run()
    {
        // 先将所有区域的hasAddresses默认设为false
        DB::table('areas')->update(['hasAddresses' => false]);
        // 取所有存在关联地址的区域id，批量更新为true
        $hasAddressAreaIds = Address::distinct()->pluck('areaId');
        DB::table('areas')->whereIn('id', $hasAddressAreaIds)->update(['hasAddresses' => true]);
    }
}

方案2：修复原有循环写法（兼容原有实现思路） #

如果需要保持循环的逻辑写法，修正后代码如下：

class AddhasAdressesSeeder extends Seeder
{
    /**
     * Run the database seeds.
     *
     * @return void
     */
    public function run()
    {
        $areas = Area::get();
        foreach ($areas as $area) {
            // 直接判断当前区域是否存在关联地址
            $exists = Address::where('areaId', $area->id)->exists();
            // 必须加where条件，仅更新当前遍历的单条区域记录
            DB::table('areas')->where('id', $area->id)->update([
                'hasAddresses' => $exists
            ]);
        }
    }
}

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