Как правильно создать сидер когда имеешь связь с родителем (laravel 8)?

Question

[NewSantaClaus]

У меня есть данные, а именно:
- города
- населенные пункты в каждом городе
- тип населенного пункта

Мне нужно наполнить базу таким образом, чтобы населенный пункт быть привязан к конкретному городу по его id. База должна наполнятся реальными данными, а не с помощью факера.

В теории, я бы мог сделать что-то вроде:

<?php

namespace Database\Seeders;

use App\Models\Locality;
use App\Models\LocalityType;
use App\Models\RegionalCenter;
use Illuminate\Database\Seeder;

class LocalitySeeder extends Seeder
{
    public $localities = [
        [
            'name' => 'Населенный пункт 1',
            'type' => 'Название типа 1',
            'region' => 'Название обл. 1'
        ],
        [
            'name' => 'Населенный пункт 2',
            'type' => 'Название типа 2',
            'region' => 'Название обл. 2'
        ]
    ];

    public function run()
    {
        foreach($this->localities as $locality){
            Locality::insert([
                'name' => $locality['name'],
                'regional_center_id' => function() use($locality){
                    return RegionalCenter::findOrCreate(['name' => $locality['region']])->id;
                },
                'locality_type_id' => function() use($locality){
                    return LocalityType::findOrCreate(['name' => $locality['type']])->id;
                }
            ]);
        }
    }
}

Но что-то мне подсказывает что так не гуд. Как это сделать правильно?

Либо формат данных еще может быть такой, в зависимости от оптимального решения задачи

public $regional_centers = [
        [
            'name' => 'Название обл. 1',
            'localities' => [
                [
                    'name' => 'Населенный пункт 1',
                    'type' => 'Название типа 1'
                ]
            ]
        ],
        [
            'name' => 'Название обл. 2',
            'localities' => [
                [
                    'name' => 'Населенный пункт 2',
                    'type' => 'Название типа 2'
                ]
            ]
        ],
    ];

Answer 1

[toratoda]

все почти так. два нюанса:
1. каждый шаг в цикле делать два запроса на регион и тип это не хорошо : слишком много запросов. Решение: обойди в цикле данные выбери все уникальные значения для регионов и типов и в двух запросов получи все нужные регионы и типы (а потом оперируй этими данными).
2. делать операцию вставки на каждый шаг тоже не хорошо : слишком много запросов. Решение: сформируй массив со всеми данными для вставки и выполните запрос вставки к базе данных.

примечание: следует оценить объем данных и если он очень большой то его нужно разбить на части

Comments

[NewSantaClaus]

Спасибо что уделили время моему вопросу

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

• областей
  
• типов населенных пунктов
  
• населенных пунктов
  

И это нормально. Но тут возникает вопрос. При вставке населенного пункта, и получения недостающих полей для него, я могу столкнутся с ошибкой. А именно, теоретически может быть такое что при поиске по имени области или типу населенного пункта я могу получить в ответ null, если вдруг окажется что такого в базе нет. И как быть тогда? Все равно придется его дописать, или бросать какое-то исключение и откатывать сидеры, что-ли? Так как в отличие от этого варианта, я сразу нахожу, или создаю

Что касается второго, да, согласен. У меня сейчас в районе 1к населенных пунктов, что по сути не так много. Но пачками по 500, за раз, можно было бы сделать

[toratoda]

NewSantaClaus,
если я вас верно понял, то полагаю не стоит смешивать все в одну кучу. Первая задача загрузка всех областей, вторая загрузка все типов населенных пунктов, если они обе прошли успешны можно приступать к населенным пунктам в таком случае в базе будут уже все требуемые данные.

[NewSantaClaus]

toratoda, да, я с вами согласен. Но тогда мне нужно будет разделить все на три сидера. Но в таком подходе сложность в том что я не буду знать нужный мне id который нужно присвоить той или иной записи. А чтобы этого избежать, мне нужно будет дублировать массив с данными с каждый из тих сидеров, что не приемлемо

[toratoda]

NewSantaClaus, в последнем сиде следует сделать селект к базе за получением требуемых ид (а как следствие проверку наличия всех необходимых данных). Это может быть показаться излишними запросами, однако они позволяют абстрагироваться (не зависеть) последнему сидеру от остальных.