Как оформить сиды в laravel [best practics]?

Question

[raingo]

Есть модели ларавел Group, Students с отношениями один-многие. Сидинг выполняю через фабрики и faker.

Нормально ли создавать связанные модели в одном файле? В разных файлах связывать сложнее.

<?php

use Illuminate\Database\Seeder;
use App\Group;
use App\Student;

class GroupsTableSeeder extends Seeder
{
    public function run()
    {
         //создаем группы
        factory(Group::class, 3)->create()->each(function ($g) {
            //создаем студентов и сразу связываем ч-з фабрику
        	factory(Student::class, 10)->create([
            		'group_id' => $g->id
            	]);
        });
    }
}

Comments

[raingo]

Вроде нарушается принцип SOLID, но код то читабельный и красивый с моей точки зрения. Для связывания в отдельном файле из StudentsTableSeeder придется лишний раз обращаться к бд и создавать коллекции групп, чтобы получить их id.

Answer 1

[Алексей Уколов]

Нормально. В сидах и фабриках можно смело игнорить паттерны проектирования в пользу здравого смысла - это же довольно простой код, который ещё и выполняется вне продакшена.