请问假如有1000个配置项，如何避免使用这种写法，更加优雅的写法是怎样的？

public function save(Request $request)
{
    DB::transaction(function () use ($request) {

            $settings = Setting::all()->keyBy('key');

            foreach ($request->all() as $key => $value) {
                if (isset($settings[$key])) {
                    if ($settings[$key]->value <> $value) {
                        $settings[$key]->update(['value' => $value]);
                    }
                } else {
                    Setting::create([
                        'key' => $key,
                        'value' => $value
                    ]);
                }
            }
        });
    return Result::success();
}

要不改成Redis哈希吧,感觉挺适合的。

 Redis::HMSET('config',$request->all());

/*
Redis::HMSET('config',['logo' => 'http://baidu.com','admin_logo' => 'http://...']);
*/

减少数据库操作，减少数据处理的量级，事务没加可自行添加

// 获取所有数据
$requestData = $request->all();
// 获取提交的键
$requestKey = array_keys($requestData);
// 获取数据库存在的数据
$existsData = Setting::whereIn('key', $requestKey)->get();
// 取出键
$existsKey = $existsData->pluck('key')->toArray();
// 获取需要插入的键
$insertConfigData = array_diff($existsKey, $requestKey);
// 构造批量插入数据
$insertData = collect($requestData)->only($insertConfigData)->map(function ($item, $key) {
    return [
        'key' => $key,
        'value' => $item,
    ];
})->toArray();
// 批量插入
Setting::insert($insertData);
// 逐个更新已存在键
$existsData->map(function ($item) use ($requestData) {
    $item->value = $requestData[$item->key];
    $item->save();
});

不过感觉这种设计不合理，1000个配置这么处理？？？，可以换个方案

写成单个key查询配置不就好了，配置项不都是独立存在的吗

@Linxb

前端是这么搞的 :joy: :joy:，那么多配置项用同一个保存

@一个人的江湖 为什么不每个tab 设计成一条数据，反正前端每次都会传tab的所有值给你，直接单条数据全量更新

可以生产配置文件xx.config, 修改、添加直接操作文件不就OK了

是想批量更新数据？将key字段设置为唯一索引，然后用replace into或者on duplicate key update就好了

试试这种方式？ github.com/baijunyao/laravel-bjybl...

   /**
     * @param array<int,array<string,mixed>> $multipleData
     *
     * e.g.
     * $multipleData = [
     *     [
     *         'id' => 1 ,
     *         'name' => 'baijunyao'
     *     ],
     *     [
     *         'id' => 2 ,
     *         'name' => 'Junyao Bai'
     *     ]
     * ]
     */
    public function updateBatch(array $multipleData = []): int
    {
        if (empty($multipleData)) {
            return 0;
        }

        $tableName       = config('database.connections.mysql.prefix') . $this->getTable();
        $updateColumn    = array_keys($multipleData[0]);
        $referenceColumn = $updateColumn[0];

        unset($updateColumn[0]);

        $whereIn = '';
        $sql     = 'UPDATE ' . $tableName . ' SET ';

        foreach ($updateColumn as $uColumn) {
            $sql .= $uColumn . ' = CASE ';

            foreach ($multipleData as $data) {
                $sql .= 'WHEN ' . $referenceColumn . " = '" . $data[$referenceColumn] . "' THEN '" . $data[$uColumn] . "' ";
            }

            $sql .= 'ELSE ' . $uColumn . ' END, ';
        }

        foreach ($multipleData as $data) {
            $whereIn .= "'" . $data[$referenceColumn] . "', ";
        }

        $sql = rtrim($sql, ', ') . ' WHERE ' . $referenceColumn . ' IN (' . rtrim($whereIn, ', ') . ')';

        return DB::update($sql);
    }

"如下写法肯定是行不通的", 为什么要这么说呢，我感觉这么写问题也不是太大

delete in keys
然后在把接收到的值全部添加进去，2条sql解决所有变更😄