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

Question

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

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 向数据透视表。其次，您用来添加支持的包看起来会不断将项目附加到该记录，因为没有同步，因此会在数据库中创建大量的数据重复。第三，如果您要通过在表中的这些字段上添加唯一索引来强制执行唯一记录，则不支持 insert ignore 查询。

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

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

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

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

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() 将返回创建的行数，这样您就可以很好地了解您尝试插入的项目有多少是重复的，如果您需要的话.但是，如果您在重复键更新上添加 ，请记住 MySQL 将返回受影响的 2 行，每行已更新但未插入，因此在这种情况下，该数字可能对您没有用。