Question

问题：

如何根据该对象中的特定键更新列表中的对象？

说明

说我的mongodb中有以下对象：

{
    "keysteps":[
        {"timecode":"01:00:00", "title": "Chapter 1"}
    ]
}

并说我想用键盘列表更新“keysteps”（又名：“章节”），我如何确保如果时间码已经存在，我更新对象而不是创建新的一个？

如果我用这些按键步骤更新我的mongodb：

{
    "keysteps":[
        {"timecode":"01:00:00", "title": "Chapter 1 - edited"}
    ]
}

以下是我得到的输出：

keysteps": [
      {
        "timecode": "01:00:00",
        "title": "Chapter 1"
      },
      {
        "timecode": "01:00:00",
        "title": "Chapter 1 - edited"
      }
    ]

它不是更新键盘，而是在键盘列表中添加了一个新对象。 如果我发送了相同的时间码对象（相同的时间码和标题），则不会创建新的。

我做了什么：

这是我用来更新的Symfony控制器的片段：

class VideoToolsController extends Controller
{
    //More code

    public function keystepsUpdateAction(Request $request, int $video_id){
        //If the video id already exists, throw an error

        $keystepsS = $this->get('video_tools.keysteps');
        $exists = $keystepsS->exists($video_id);

        //Return immediately if keysteps doesn't exist.
        if(!$exists)
            return JsonResponseService::getErrorBadRequest("Keystep does not exist.");

        $keysteps = json_decode($request->getContent(), true);
        //die(var_dump($data));
        $form = $this->createForm(KeystepsType::class);
        $form->submit($keysteps);
        //return new JsonResponse($data);
        //Return bad request if the form is not valid.
        if(!$form->isValid())
            return JsonResponseService::getErrorBadRequest("Keysteps are not valid, try again.");

        try {
            $res = $keystepsS->update($video_id, $keysteps['keysteps']);
            return JsonResponseService::getSuccess($res, "Updated keysteps.");

        } catch(Exception $exception){
            return JsonResponseService::getErrorBadRequest($exception->getMessage(), $keysteps);
        }
    }
    //More code
}

这是我用来更新的Php服务的片段：

class KeystepsService {

    //More code

    //TODO: Make timecode UNIQUE
    public function update($video_id, $keysteps) : int {

        $updatedOne = $this->mongoCollection->updateOne(
            ['video_id' => $video_id ],
            ['$addToSet' => [ "keysteps" => [ '$each' => $keysteps ]]]
        );

        return $updatedOne->getModifiedCount();
    }

    //More code
}

的mongodb /蒙戈-PHP-库：

mongodb/mongo-php-library

随意给我一些关于如何标题/问题的建议，我觉得好像不是重点。

Answer 1

正如您所发现的那样，$addToSet运算符不适合在此处使用。这是设计，因为＆＃34; set＆＃34;属于＆＃34;对象＆＃34;在数组中，如果你以任何方式在键中给出不同的值组合，那么这就是＆＃34; new＆＃34;对象并被添加。

所以处理这个的正确方法是使用＆＃34;两个＆＃34;操作：

尝试按键找到数组中确实存在的项目，然后使用$set更新
如果密钥不存在

，则尝试将$push项目添加到数组中。
显然，您不想编写并等待数据库对多个操作的响应，因此您可以使用bulkWrite()方法同时发送这两个操作：

$this->mongoCollection->bulkWrite([ [ 'updateOne' => [ [ 'video_id' => $video_id, 'keysteps.timecode' => $keysteps[0]['timecode'] ], [ '$set' => [ 'keysteps.$.title' => $keysteps[0]['title'] ] ] ]], [ 'updateOne' => [ [ 'video_id' => $video_id, 'keysteps.timecode' => [ '$ne' => $keysteps[0]['timecode'] ] ], [ '$push' => [ 'keysteps' => $keysteps[0] ] ]] ])

请注意，虽然仍然保留了数组对象结构，但由于此两步操作的性质，故意不在此处使用$each。如果您的输入中可以包含多个数组项，那么您将循环这些项并创建＆＃34;对＆＃34;通过索引拉出数组值的操作。

因此这里的0索引用法只是索引变量的占位符，您可以使用它来支持输入中的多个数组项。你也可以生成＆＃34;内部＆＃34; ＆＃34;操作的文件＆＃34;而不是实际多次发出bulkWrite()。毕竟这个方法的重点只是一次调用它。

编辑工作解决方案：

public function update($video_id, $keysteps) : int { $modifiedData = 0; foreach($keysteps as $keystep){ $bulkUpdate = $this->mongoCollection->bulkWrite([ [ 'updateOne' => [ [ 'video_id' => $video_id, 'keysteps.timecode' => $keystep['timecode'] ], [ '$set' => [ 'keysteps.$.title' => $keystep['title'] ] ] ]], [ 'updateOne' => [ [ 'video_id' => $video_id, 'keysteps.timecode' => [ '$ne' => $keystep['timecode'] ] ], [ '$push' => [ 'keysteps' => $keystep ] ]] ]]); $modifiedData += $bulkUpdate->getModifiedCount(); } return $modifiedData; }

Answer 2

与Neil的答案相似，可能只需先删除重复项，然后再推新章节：

public function update($video_id, $keysteps) : int {
    $timecodes = array_map(function($chapter){return $chapter['timecode'];}, $keysteps);

    $updatedOne = $this->mongoCollection->updateOne(
        ['video_id' => $video_id ],
        ['$pull' => [ 'keysteps' => [ 'timecode' => ['$in' => $timecodes]]]]
    );

    $updatedOne = $this->mongoCollection->updateOne(
        ['video_id' => $video_id ],
        ['$addToSet' => [ 'keysteps' => [ '$each' => $keysteps]]]
    );

    return $updatedOne->getModifiedCount();
}

我的理由是它应该使用带有关键字段＆＃39;时间码的任意子文档，所以例如希望你添加一个＆＃39;描述＆＃39;字段到子文档而不是更改标题，它仍然有效。

使用批量写入包装它是一个好主意，但不是必需的。它不是事务的替代品，因此如果处理并发更新至关重要，则应使用乐观锁等内容。

如何根据对象的键（而不是整个对象）更新列表中的对象？

问题：

说明

我做了什么：

的mongodb /蒙戈-PHP-库：

2 个答案: