首页> PHP的>如何重构我的嵌套3深foreach循环

我使用的是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]);
                }
            }
        }
    }
}

解决方法:

这是使用Eloquent可能无法解决的棘手问题.首先,您发现它不支持开箱即用的3路数据透视表.其次,用于添加支持的程序包似乎将不断地将项目附加到该记录,因为没有同步,从而在数据库中创建了大量的数据重复.第三,如果要通过在表中的那些字段上添加唯一索引来强制执行唯一记录,则不支持插入忽略查询.

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

首先,您将需要在数据库级别上强制执行数据完整性.执行此操作的方式将取决于数据透视表的结构.如果该表已经有一个ID列,它是一个自动递增的主键,则需要在3列中添加一个唯一键,其中包含要链接的3个表的ID.如果此处没有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()会返回已创建的行数,因此,如果需要,您可以很好地了解尝试插入多少项是重复项.但是,如果您添加重复的密钥更新,请记住,MySQL将为每行已更新且未插入的行返回2行,因此在这种情况下,该数字可能对您没有用.

标签：laravel,php,nested-loops
来源： https://codeday.me/bug/20191027/1941076.html