Правильно ли добавлять свойство в объект модели таким образом?

Question

[Сергей Хлопов]

Здравствуйте, у меня есть модель Category, у одной категории могут быть подкатегории, поэтому в модели есть метод children.

Метод children в модели

public function children()
{
   return $this->hasMany('App\Models\Admin\Category','parent_id');
}

И сейчас мне нужно построить дерево категорий но строю я не с помощью with и рекурсии, а просто получаю все категории одним запросом, и потом стою дерево.

Метод построения дерева

private static function makeTree($items, $parentId)
{
    $tree = collect();
    if(is_array($items) && isset($items[$parentId])) {
        foreach ($items[$parentId] as $item) {
                $item->children = self::makeTree($items, $item->id); //$item - тут объект категории
                $tree->push($item);
         }
     }
     else {
            return collect();
     }
     return $tree;
}

Вопрос в том что правильно ли делать так:

$item->children = self::makeTree($items, $item->id);

В Debugbar это выглядит вот так:

Подскажите пожалуйста правильно ли это, заранее благодарю за ответ.

Answer 1

[Vitalionus]

А почему не правильно то? Все работает логично. Можешь конечно tree пакет прикрутить, что бы не писать свой велосипед
Вот этот думаю один из удобных и простых, почти ничего делать не нужно просто добавить наслодование и указать ключи родителя
https://github.com/lazychaser/laravel-nestedset
Индексы еще в базе добавь обязательно для оптимизации выборки

Comments

[Сергей Хлопов]

Понял, благодарю вас за ответ, да уже знаком с данной библиотекой) там на других операциях правда много запросов идёт, поэтому отказался от неё

Answer 2

[Вячеслав Плиско]

'App\Models\Admin\Category' - так вы потеряете данные при рефакторингах, лучше использовать \App\Models\Admin\Category::class или в вашем случае self::class
Зачем вам статический метод, если вы его дергаете из себя, то логично делать $this->makeTree()
Если у вас дерево ограниченной вложенности (обычно меньше 2-3 уровня), то достаточно воспользоваться дот нотацией в with

$categories->with('children', 'children.children', 'children.children.children');

Comments

[Сергей Хлопов]

Здравствуйте, благодарю за ответ. =)

Зачем вам статический метод, если вы его дергаете из себя, то логично делать $this->makeTree()

просто у меня это не единственная модель где древовидная структура, да кстати можно было бы этот метод вынеси в trait, об этом только сейчас подумал, но я создал отдельный класс TreeServices для этого.

Если у вас дерево ограниченной вложенности (обычно меньше 2-3 уровня), то достаточно воспользоваться дот нотацией в with

Проблема в том что нет ограничений для вложенности

[Вячеслав Плиско]

Понятно.
Но и не стоит перегружать $item->children, это может привести к непредсказуемому поведению.

Данные вы извлекли, теперь их выгружайте в Презентер или Ресурс в терминологии ларавел https://laravel.com/docs/7.x/eloquent-resources

Answer 3

[jazzus]

А почему с with нельзя? Там же parent with children все стыкуется. Если правильно понял задачу то можно так

Category::parents()
        ->with('children')
        ->get();

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

public function scopeParents($query)
{
    return $query->whereNull('parent_id');
}

Comments

[Сергей Хлопов]

Благодарю вас за ответ =), только правда вроде как такая запись выберет только категории вот так:

• Категория 1
  
• Категория 2
  
  • Категория 2.1
    
  • Категория 2.2
    
  • Категория 2.2
    
  
  
  
• Категория 3
  

А у меня просто есть такая вот вложенность

• Категория 1
  
• Категория 2
  
  • Категория 2.1
    
    • Категория 2.1.1
      
    • Категория 2.1.2
      
    • Категория 2.1.3
      
    
    
    
  • Категория 2.2
    
  • Категория 2.2
    
  
  
  
• Категория 3