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

Question

[Алексей Скляров]

Есть такая иерархия:

Пост
--Отзыв
----Комментарии к отзыву
------Комментарии к комментарию

При удалении поста, мне необходимо удалить сам отзыв, комменты к этому отзыву и комментарии к комментариям (ответы на комментарии).

Для каждой из модели использую соответствующий класс Observer, в котором реализую метод deleting(). Получается что-то вроде этого:

PostObserver.php

public function deleting(Post $post)
    {
        $post->loadMissing('reviews');

        foreach ($post->reviews as $review) {
            $review->delete();
        }
    }

ReviewObserver.php

public function deleting(Review $review)
    {
        $review->loadMissing('replies');

        foreach ($review->replies as $reply) {
            $reply->delete();
        }
    }

CommentObserver.php

public function deleting(Comment $comment)
    {
        $comment->loadMissing('replies');

        foreach ($comment->replies as $reply) {
            $reply->delete();
        }   
    }

Cпервая я в каждом методе хотел просто вызывать такую цепочку: $model->relationship()->delete();, но при таком использовании события удаления не будут срабатывать на других моделях.

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

Есть ли способы оптимизировать данный код?

Comments

[Дмитрий Кузнецов]

Удалять через foreach на мой взгляд не очень хорошая идея. если есть такой метод (точно не помню), то связью вы получаете все данные к этой модели, а после делаете delete(); вроде так делается.

Answer 1

[Илья]

возможно подойдёт on delete cascade, но в таком случае не будут выполняться евенты моделей, если они как-то задействованы

Comments

[jazzus]

Запускать евент для каждой отдельной модели при групповом удалении нет смысла. Событие (декременты инвалидации итд) можно запустить после удаления всех моделей. on delete cascade для этой задачи самое то.

[Алексей Скляров]

jazzus, проблема в том, что модель Comment связана с другими моделями полиморфной связью, поэтому просто прописать в миграции delete cascade не выйдет.

[jazzus]

Алексей Скляров, такую инфу нужно выносить в заголовок. В любом случае, не использовать для этого циклы и обсерверы. Запросы к бд и массовое удаление. И покрыть процесс тестами. Я бы вообще не заморачивался с удалением пока нет такой потребности. Отметил deleted_at и никакого гемороя)

Answer 2

[Константин Б.]

Удалять средствами базы данных если нет другой бизнес логики в коде через delete cascade

Answer 3

[James026]

Можно еще вариантик, как пишут выше - использовать средства db, но зайюзать ON DELETE SET NULL
И сделать команду, которая по крону будет удалять записи у которых ключи null
Таким образом не будет подлагивания интерфейса при удалении модели с большим количеством связей + сохранятся эвенты у удаляемых моделей