Как ускорить запись отношений belongsToMany в бд?

Question

[Dmitriy]

Паршу ексель файл, с него пачками записываю категории и продукты. После этого мне надо записать отношения для них в таблицу category_product.
Сейчас записываю так, но сильно долго выполняется скрипт, как ускорить данный процесс?

class ProductImport implements ToCollection, WithChunkReading
{
    private const NO_CONST = 'Нет';
    private const AVAILABLE = 'есть в наличие';
    private string $productsTable = 'products';

    private function isHeadingRow(int $rowNumber)
    {
        return $rowNumber === 0;
    }

    public function collection(Collection $rows)
    {
        $productsCollection = [];
        $categoriesCollection = collect();
        $categoriesProductMap = [];
        foreach ($rows as $rowNumber => $row) {
            if (!$this->isHeadingRow($rowNumber)) {
                if (isset($row[10])) {
                    $row->shift();
                }
                $row = $row->values();

                if ($row[4]) {
                    $productsCollection[] = [
                        'producer' => $row[3],
                        'name' => $row[4],
                        'vendor_code' => $row[5],
                        'description' => $row[6],
                        'price' => $row[7],
                        'warranty' => $row[8] !== self::NO_CONST ? $row[8] : 0,
                        'available' => $row[9] === self::AVAILABLE,
                    ];
                    $categoriesCollection->add($row[0]);
                    $categoriesCollection->add($row[1]);
                    $categoriesCollection->add($row[2]);
                    $categoriesProductMap[$row[5]] = [
                        $row[0],
                        $row[1],
                        $row[2],
                    ];
                }
            }
        }
        $categoriesCollection = collect($categoriesCollection)->filter()->unique();
        DB::table('categories')
            ->insertOrIgnore(
            $categoriesCollection
                ->map(fn($category) => [
            'name' => $category
        ])->toArray());

        DB::table($this->productsTable)
            ->insertOrIgnore(
                collect($productsCollection)->map(fn($product)=> [
            'producer' => $product['producer'],
            'name' => $product['name'],
            'vendor_code' => $product['vendor_code'],
            'description' => $product['description'],
            'price' => $product['price'],
            'warranty' => $product['warranty'],
            'available' => $product['available'],
        ])
                    ->toArray());

          $categoriesProductMap = collect($categoriesProductMap)
            ->map(fn($categories) => collect($categories)
                ->unique()
                ->filter())
            ->toArray();
        foreach (Product::all() as $product) {
            if (isset($categoriesProductMap[$product->vendor_code])) {
                $productCategories = $categoriesProductMap[$product->vendor_code];
                $categoryIds = [];
                foreach ($productCategories as $productCategory) {
                    $catId = Category::where('name', '=', $productCategory)->first()->id;
                    if (!in_array($catId, $product->categories->map(fn($cat) => $cat->id)->toArray())) {
                        $categoryIds[] = $catId;
                    }
                }
                $product->categories()->attach($categoryIds);
            }
        }
    }

    public function chunkSize(): int
    {
        return 1500;
    }
}

Answer 1

[Александр Sqweez]

Попробуй переписать часть с записью категорий в товары на:

Product::all()->each(function($product) {
    $productCategories = $categoriesProductMap[$product->vendor_code];
    $categoryIds = Category::whereIn('name', $productCategories)->select('id')->get()->pluck('id');
    $product->categories()->syncWithoutDetaching($categoryIds);
})

Если хочешь воспользоваться своим методом с проверкой категорий, то пользуйся Eager Loading

Аналог твоего Product::all(), только без проблемы N + 1 запроса.

Comments

[Dmitriy]

буду знать, спасибо