如何重构我嵌套的 3 个深 foreach 循环

Question

我正在使用 laravel 允许三重枢轴 table 的作曲包，但它不支持批量插入。所以我一直循环遍历每个数组，以确保三个 table 中的每个 id 在我的三重枢轴 table 中都有匹配。我的工作有效，但我想要一些关于如何重构我的函数的建议，这样我就不必有那么多嵌套循环了。我正在考虑使用数组映射。

public static function addKeyPhraseToPivotTable($ids) {

    $locations = Location::where('account_id', self::$items[0]['account_id'])->get();
    $urls = Url::where('account_id', self::$items[0]['account_id'])->get();

    if($locations->count() > 0 && $urls->count() > 0) {
        foreach($ids as $keyPhraseId) {
            foreach($locations as $location) {
                foreach($urls as $url) {
                    $url->keywords()->attach([$keyPhraseId, $location->id]);
                }
            }
        }
    }
}

Answer 1

这是一个很难用 Eloquent 解决的问题，目前可能是不可能的。首先，正如您所发现的，它不支持开箱即用的 3 向枢轴 tables。其次，您用来添加支持的包看起来会不断地将项目附加到该记录，因为没有同步，因此会在数据库中创建大量的数据重复。第三，如果您要通过在 table 中的这些字段上添加唯一索引来强制执行唯一记录，则不支持 insert ignore 查询。

考虑到所有这些，为此使用原始 sql 可能会有用。但是，由于 SQL 中没有发生任何疯狂的事情，我认为在这种情况下会很好。

首先，您需要在数据库级别强制执行数据完整性。您执行此操作的方式取决于您的枢轴 table 的结构。如果这个 table 已经有一个 id 列，它是一个自动递增的主键，您需要向包含 3 table 的 ID 的 3 列添加一个唯一键您正在尝试 link。如果您没有 id 此处或其他一些主键，您可以设置一个包含这 3 列的复合主键。

如果已经有重复的记录，这可能会失败。您需要做的就是删除那些重复项并重新添加您的密钥。

综上所述，我会重构您的函数，以便它使用批量插入。

function addKeyPhraseToPivotTable($ids) {

    $locations = Location::where('account_id', self::$items[0]['account_id'])->get();
    $urls = Url::where('account_id', self::$items[0]['account_id'])->get();

    $deletes = [];
    if ($locations->count() > 0 && $urls->count() > 0) {
        foreach ($ids as $keyPhraseId) {
            foreach ($locations as $location) {
                foreach ($urls as $url) {
                    $deletes[] = [
                        'location' => $location->id,
                        'key' => $keyPhraseId,
                        'url' => $url->id, // Not sure what the ID is of this item, might need to change it.
                    ];
                }
            }
        }
    }

    $sql = 'insert ignore into accounts (location_id, key_phrase_id, url_id) values ';

    foreach ($deletes as $i => $delete) {
        $sql .= $i == 0 ? '' : ',';
        $sql .= sprintf('(%s, %s, %s)', $delete['location'], $delete['key'], $delete['url']);
    }

    DB::unprepared($sql);
}

这应该尝试插入您拥有的每条记录，因为您正在使用插入忽略，所以在尝试插入重复项时它不会中断。

我相信 DB::unprepared() 会 return 创建的行数，这样您就可以很好地了解您尝试插入的项目有多少是重复的（如果您也需要的话）。但是，如果您添加 on duplicate key update，请记住 MySQL 将 return 影响每行已更新但未插入的 2 行，因此在这种情况下，该数字可能对您毫无用处。

如何重构我嵌套的 3 个深 foreach 循环

How to refactor my nested 3 deep foreach loop

php

nested-loops

laravel