Laravel 6. Жадная загрузка почему не работает?

Question

[EEM]

Генерируются очень много SQL запросов при отображение страниц.

8 statements were executed 22.93ms
select * from `users` where `id` = 1 limit 1

select * from `books_categories` where `parent_id` = '0'

select * from `books_categories` where `books_categories`.`parent_id` in (1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33)

select * from `books_categories` where `books_categories`.`parent_id` = 35 and `books_categories`.`parent_id` is not null
select * from `books_categories` where `books_categories`.`parent_id` = 36 and `books_categories`.`parent_id` is not null
select * from `books_categories` where `books_categories`.`parent_id` = 38 and `books_categories`.`parent_id` is not null
select * from `books_categories` where `books_categories`.`parent_id` = 39 and `books_categories`.`parent_id` is not null
select * from `books_categories` where `books_categories`.`parent_id` = 40 and `books_categories`.`parent_id` is not null

***

Контроллере:

public function index()
    {
        return view('admin.book_categories.index', [
                'books_list' => BookCategory::with('children')->parent()->get(),
                'delimiter' => '',
                'i' => 0
        ]);
    }

В моделе:

public function children()
    {
        return $this->hasMany(self::class, 'parent_id');
    }

 public function scopeParent($query)
    {
        return $query->where('parent_id', '0');
    }

Как оптимизировать код чтобы запросов к базу был максимально оптимально ?

Answer 1

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

Вот запросы, которые генерирует ваша выборка:

select * from `books_categories` where `parent_id` = '0'

select * from `books_categories` where `books_categories`.`parent_id` in (1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33)

Она работает корректно. Видимо, у вас в шаблоне идёт доступ к каким-то незагруженным данным, но код шаблона вы не предоставили...

Comments

[EEM]

код шаблона

@isset($books_list)
    @foreach ($books_list as $item)
        <tr>
            <td>{{ $i++ }}</td>
            <td>{!! $delimiter ?? "" !!} {{ $item->title }}</td>
            <td>{{ $item->slug }}</td>
            <td class="text-center" style="width: 30px;">

                <div class="btn-group-sm">
                    <form action="{{ route('admin.category.destroy', [ 'category' => $item->id ])}}" method="post"
                          style="display: flex;">
                        <a href="{{ route('admin.category.edit',$item->id ) }}" class="btn-sm btn-success"
                           role="button"> <i class="fas fa-edit"></i></a>
                        @csrf
                        @method('DELETE')
                        <button class="btn-sm btn-danger" type="submit" style="margin-left: .5rem;">
                            <i class='fas fa-trash-alt'></i>
                        </button>
                    </form>
                </div>
            </td>
        </tr>
        @if (count($item->children) > 0)
            @include('admin.book_categories.partials.table', [
              'books_list' => $item->children,
              'delimiter'  => ' - ' . $delimiter,
              'i'=>$i++
            ])

        @endif
    @endforeach
@endisset

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

Запросы вот здесь, похоже:

@include('admin.book_categories.partials.table', [
  'books_list' => $item->children,
  'delimiter'  => ' - ' . $delimiter,
  'i'=>$i++
])

Там, я так понимаю, примерно дублируется код шаблона. Тогда запрос инициируется вот этой проверкой:
@if (count($item->children) > 0)
Вы в контроллере грузите детей, а потом в цикле проверяете наличие детей у этих детей.

Если у вас только два уровня вложенности, передавайте в паршиал флаг, что это последний уровень или вообще уберите оттуда итерирование по детям.

Если вложенность неограниченная, то при такой структуре данных загрузить всё стандартным функционалом не выйдет.
Но можно указать столько уровней, сколько есть обычно: BookCategory::with('children.children.children').
Или выбирать все категории плоским списком, а дерево строить рекурсивно уже в контроллере.

Ну а лучше совсем поменять структуру хранения и использовать, например, nested set.

[EEM]

Спасибо. Перешел на nested set