问题:

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

描述:

假设我的 mongodb 中有以下对象:

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

然后说我想用关键步骤列表(又名:“章节”)更新“关键步骤”，我将如何确保如果时间码已经存在，我更新对象而不是创建新对象一个？

如果我用这些关键步骤更新我的 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 Controller 的片段:

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/mongo-php-library:

mongodb/mongo-php-library

请随意给我建议如何措辞标题/问题，我觉得好像不是重点。

最佳答案

正如您所发现的，$addToSet 操作符不适合在这里使用。这是设计使然，因为“集合”属于数组中的“对象”，如果您以任何方式在键中提供不同的值组合，那么这是一个"new"对象并被添加。

所以处理这个问题的正确方法是使用“两个”操作:

1. 尝试通过键在数组中找到它确实存在的项目，然后用$set更新它
2. 尝试$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。如果您的输入中可以包含多个数组项，那么您将循环这些项并创建“一对”操作，按索引拉出数组值。

因此，此处的 0 索引用法只是索引变量的占位符，您将使用它来驱动它以支持输入中的多个数组项。您还可以只为“操作”生成“内部”文档，而不是实际发出 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;
}

关于php - 如何根据对象的键(而不是整个对象)更新列表中的对象？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/44128806/