PHP - 在foreach循环外保存到数据库

Question

我有下面的 foreach 循环，它迭代了我的用户可以应用的一堆不同的自定义规则。

public function parse(Document $document)
{
        $content = $document->text;
        $rules = $document->stream->fieldrules;

        foreach ($rules as $rule) {
            $class = $this->getClass($rule);
            $content = $class->apply($content);

            //Save to database.
            $fieldresult = new FieldRuleResult();
            $fieldresult->create([
                'field_rule_id' => $rule->field_id,
                'document_id' => $document->id,
                'content' => $content
            ]);
        }
}

如您所见，我在每次迭代时都调用数据库。一些用户可能定义了多达 50 个规则，这将导致 50 个插入查询。所以我相信我可能遇到了n+1 的问题。

我想知道 - 这里的最佳做法是什么？我试过搜索 Laravel 文档，找到了 createMany 方法。所以我尝试将迭代重构为以下内容：

$result = [];
foreach($rules as $rule)
{
  $class = $this->getClass($rule);
  $content = $class->apply($content);
  $fieldresult = new FieldRuleResult();

  $result[] = [
     'field_rule_id' => $rule->field_id, 
     'document_id' => $document->id, 
     'content' => $content];
}

return $rules->createMany($result);

这给了我以下错误：

Method Illuminate\Database\Eloquent\Collection::createMany does not exist.

现在我想这是因为$rules 返回了一个集合。我试图将其更改为：

return $document->stream->fieldrules()->createMany($result);

这给了我以下错误：

Call to undefined method Illuminate\Database\Eloquent\Relations\HasManyThrough::createMany()

Answer 1

Illuminate/Database/Query/Builder 中有一个可用的方法，名为 insert。此方法允许您在一个查询中插入多个模型。请注意，这不会像所有多合一查询方法那样触发雄辩的事件。

就像你已经做的那样，将数据存储在一个数组中，并在循环完成后执行查询。不过这次你可以使用insert。

$content = 'your content here...';

$data = [];

foreach($rules as $rule) {

    $class = $this->getClass($rule);

    $content = $class->apply($content);

    $data[] = [
        // ...
        'content' => $content
    ];
}

FieldRuleResult::insert($data);

不是 100% 确定这直接适用于模型。但是，在 documentation 中，您可以找到带有 DB 外观的示例。

Answer 2

如果出于任何原因需要触发 Eloquent 事件，并且您想针对 n+1 问题进行优化，我建议使用事务。这仍然会产生n 查询，但数据库可以通过批量插入来优化它（因为事务）。它具有数据一致性的额外好处（所有插入都成功，或者都不成功）。

$result = getResult();

\DB::transaction(function() use ($result) {
    foreach($result as $item) {
        FieldRuleResult::create($item);
    }
});

如果您不关心 Eloquent 事件被触发，而只想“批量插入”，则可以使用 Fabian 提到的 insert 方法。不过，为了数据的一致性，我仍然建议使用数据库事务。

$result = getResult();

\DB::transaction(function() use ($result) {
    FieldRuleResult::insert($result);
});

在后一种情况下，如果您要插入的条目数量可能“很大”，您可能还需要对这些插入进行分块。 （例如，一次 100 个，而不是一次全部）