如何使用非托管实体的数据正确更新托管实体？

Question

背景

我的目标是使用同一类的非Doctrine管理对象中的数据对托管实体执行更新。

如果可以执行“简单更新”，即替换属性，那将很酷，但实际上，如果我清除ArrayCollection，旧数据似乎不会被删除（即使我从ArrayCollection的元素中清除了所有与小提琴的引用，或者设置了orphanRemoval为true）。

但让我们进入一个具体的例子。 我有this entity，它有很多OneToOne / OneToMany关系来表示fiddle。 我可以使用Symfony2命令导入小提琴样本（先前从另一个环境导出为json）。

如果样本已经存在，如何正确地进行更新？

错误的想法：执行DELETE + INSERT

我使用以下代码（简化）构建我的实体：

$fiddle = new Fiddle();
$fiddle->setHash($this->get($json, 'hash'));
$fiddle->setRevision($this->get($json, 'revision'));

$context = $fiddle->getContext();
$context->setFormat($this->get($json, 'context', 'format'));
$context->setContent($this->get($json, 'context', 'content'));

$fiddle->clearTemplates();
$jsonTemplates = $this->get($json, 'templates') ? : array ();
foreach ($jsonTemplates as $jsonTemplate)
{
    $template = new FiddleTemplate();
    $template->setFilename($this->get($jsonTemplate, 'filename'));
    $template->setContent($this->get($jsonTemplate, 'content'));
    $template->setIsMain($this->get($jsonTemplate, 'is-main'));
    $fiddle->addTemplate($template);
}

// ...

我现在可以在已存在的情况下删除实体并持久化它：

    $check = $this
       ->getContainer()
       ->get('doctrine')
       ->getRepository('FuzAppBundle:Fiddle')
       ->getFiddle($fiddle->getHash(), $fiddle->getRevision());

    if (!is_null($check->getId()))
    {
        $em->remove($check);
        $em->flush();
    }

    $em->persist($fiddle);
    $em->flush();

但是如果样本已经存在，这将创建一个DELETE + INSERT而不是UPDATE。这很奇怪，因为用户可以收藏小部件，并且关系是通过id建立的。

丑陋的想法：对主实体和一对一关系进行更新，对一对多关系进行删除+插入

我首先获取我的小部件，如果它已经存在，我就清理它并用新数据填充它...代码运行良好，但非常丑陋，您可以在这里检查它。

作为示例，请检查tags属性：由于标签可能已被删除/更改，因此我应该通过用新标签替换旧标签来正确设置新标签。

// remove the old tags
foreach ($fiddle->getTags() as $tag)
{
   if (\Doctrine\ORM\UnitOfWork::STATE_MANAGED === $em->getUnitOfWork()->getEntityState($tag))
   {
      $em->remove($tag);
      $em->flush();
   }
}

// set the new tags
$tags = new ArrayCollection();
$jsonTags = $this->getFromArray($json, 'tags');
foreach ($jsonTags as $jsonTag)
{
   $tag = new FiddleTag();
   $tag->setTag($jsonTag);
   $tags->add($tag);
}
$fiddle->setTags($tags);

由于标签是使用fiddle的id引用的，因此即使这很丑陋，我也可以使用->remove。在这里这样做是可以的，但如果id是自动生成的，则必须有更好的解决方案。

我还尝试了将旧的fiddle id设置为新的id并进行合并，但导致了以下异常：

[Symfony\Component\Debug\Exception\ContextErrorException]  
Notice: Undefined index: 00000000125168f2000000014b64e87f

赏金？

我希望使用此更新样式将表单绑定到非托管实体并仅在需要时更新现有实体，而不仅仅是简单的“导入功能”。因此，我的目标是创建一个通用的适用于所有类型实体的东西。

但我当然不期望整个代码。处理托管ArrayCollection的更新的良好实践以及在编写此功能之前应考虑的一些提示/警告应该足够了。

- Alain Tiemblo

---

我猜我对正在发生的事情感到困惑。为什么不只是获取实体，进行更新，然后刷新呢？为什么要删除/持久化？ - Jason Roman

我只是希望更好地理解您的具体问题，或者您所指的托管实体的自动刷新等内容。 - Jason Roman

当我将一个托管实体绑定到已发布的表单($form->handleRequest($fiddle);)并且表单被正确填写($form->isValid()返回true)时，我需要进行一些其他检查，例如查看当前用户是否能够保存它。如果他有权保存，那就没问题；但是如果他没有权限保存，我应该分离实体以避免自动更新。我在3天前发现了这个令人难以置信的自动更新，并且在完全理解发生了什么之前，再也不会使用托管实体来处理表单了。 - Alain Tiemblo

在表单检查是否有效之前，难道不应该进行安全性检查吗？比如使用@Security注释来检查？ - Jason Roman

他们并非恶意访问此路由。如果用户无权保存小提琴，则会复制小提琴。 "创建新版本"按钮只调用与保存相同的路由（没有版本号-但如果定义了版本号，则应进行安全检查）。此外，小提琴可以由匿名用户拥有（他们可以编辑它，直到他们的会话结束）。看起来很复杂，但实际上就是这么简单。 - Alain Tiemblo

Answer 1

控制Doctrine所持久化的内容

仅在必要时更新现有实体。

使用Doctrine可以很容易地实现此目标:

您需要的是变更跟踪策略延迟显式。

Doctrine默认使用变更跟踪策略延迟隐式。这意味着当您调用$em->flush()时，Doctrine将检查所有已管理的实体以计算更改集。然后，所有更改都被持久化。

当使用更改跟踪策略Deferred Explicit并调用$em->flush()时，Doctrine将仅处理您明确调用$em->persist()的实体。换句话说：您可能有数千个托管实体，在其中2个上调用了$em->persist()，Doctrine只会计算这2个实体的更改集（如果需要，会持久化更改）。

更改跟踪策略可以在实体类级别上设置。因此，如果要让某个实体类使用Deferred Explicit，只需向类文档块添加注释即可：

/**
 * @Entity
 * @ChangeTrackingPolicy("DEFERRED_EXPLICIT")
 */
class Fiddle
{

然后，只需要在确实需要时调用$em->persist($fiddle)即可。

为整个聚合（根实体和其所有子实体）设置相同的更改跟踪策略可能是明智的选择。

PS：还有第三种名为Notify的更改跟踪策略，它需要更多的设置工作，但在调用$em->flush()时可以提供更细粒度的控制。但我认为你不需要走得这么远。

更新Fiddle

看到您用于更新Fiddle实体的代码，我认为您可以改进一些东西。

首先，将管理关联的责任移回实体：

/**
 * @Entity
 * @ChangeTrackingPolicy("DEFERRED_EXPLICIT")
 */
class Fiddle
{
    // ...

    /**
     * @return FiddleTag[]
     */
    public function getTags()
    {
        return $this->tags->toArray();
    }

    /**
     * @param FiddleTag $tag
     */
    public function addTag(FiddleTag $tag)
    {
        if (!$this->tags->contains($tag)) {
            $this->tags->add($tag);
            $tag->setFiddle($this);
        }
    }

    /**
     * @param FiddleTag $tag
     */
    public function removeTag(FiddleTag $tag)
    {
        if ($this->tags->contains($tag)) {
            $this->tags->removeElement($tag);
            $tag->setFiddle(null);
        }
    }

    /**
     * @param FiddleTag[] $newTags
     */
    public function replaceTags(array $newTags)
    {
        $currentTags = $this->getTags();

        // remove tags that are not in the new list of tags
        foreach ($currentTags as $currentTag) {
            if (!in_array($currentTag, $newTags, true)) {
                $this->removeTag($currentTag);
            }
        }

        // add tags that are not in the current list of tags
        foreach ($newTags as $newTag) {
            if (!in_array($newTag, $currentTags, true)) {
                $this->addTag($newTag);
            }
        }
    }

    // ...
}

现在你的ImportCommand中的代码可能如下所示：

$jsonTags = $this->getFromArray($json, 'tags');
$newTags  = [];

foreach ($jsonTags as $jsonTag) {
    $tag = $tagRepo->findOneByTag($jsonTag);

    if ($tag === null) {
        $tag = new FiddleTag();
        $tag->setTag($jsonTag);
    }

    $newTags[] = $tag;
}

$fiddle->replaceTags($newTags);

当一切都正常且可以持久化时，执行以下操作：

$em->persist($fiddle);

foreach ($fiddle->getTags() as $tag) {
    $em->persist($tag);
}

$em->flush();

当您在关联上配置了cascade=persist时，应该能够省略手动持久化标签的循环。

专业提示：

您可以查看JMS Serializer库以及将其集成到Symfony中的Bundle。

- Jasper N. Brouwer

---

这是一个非常有趣的回答，感谢分享。我会在几个小时内查看。延迟隐式是我对于意外持久性感到惊讶的原因，非常好的发现。 - Alain Tiemblo