Как удалить несколько файлов при одновременном удалении поста и информации в базе данных?

Question

[Maксим Волков]

Работаю над админкой – CRUD для сайта. Каждый пост имеет галерею изображений, информация о которых хранится в отдельной таблице базы данных. Файлы изображений загружаются и хранятся в /storage/app/public.

Возникла некоторая проблема при удалении файлов изображений из папки storage при удалении поста. С удалением информации о данных файлах в базе данных проблем нет, но имею некоторое непонимание как при удалении поста удалить группу файлов, привязанных к посту.

Прошу помощи. Подскажите наиболее рациональный способ?

//Pechnik -  модель/таблица содержит данные поста
//Images - модель/таблица содержит информацию о  изображениях галереи 

// - метод ресурсного контролера для удаления поста
 public function destroy($id)
    {
        Images::where('pechnik_id', $id)->delete();
        Pechnik::find($id)->delete();
        return redirect()->route('admin.index')->with('success', 'Информаця успешно удалена');
    }

Comments

[Maксим Волков]

VicTHOR, Благодарю за помощь!Уверен это должно работать, Но в моем случае при удалении записей из базы данных, файлы изображений все равно не удаляются, остаются на диске.

Полагаю, это происходит по-тому, $path = $images->get('img');' возвращает объект -c информацией о файлах с указанием пути в виде: storage/images/file.jpeg
В моем случае фактически данные фалы имеют другой путь: storage/app/public/images/file.jpeg

Наверное, для успешного удаления файлов необходимо изменить полученные в переменную $path данные о путях к файлам storage/images/ на реальный путь /storage/app/public/images/ . Но как это сделать? Перебрать в цикле с использованием str_replace() на каждой итерации? Как думаете, есть какой-нибудь более рациональный способ решения проблемы?

[HabrDima20]

onDelete('cascade');
указать в миграции, в таблице промежуточной

Answer 1

[Maксим Волков]

Опишу решение данной задачи. Мною задача по одновременному удалению файлов галерей при удалении поста, решена так:

//Pechnik -  модель/таблица содержит данные поста
//Images - модель/таблица содержит информацию о  изображениях галереи 
 
// - метод ресурсного контролера для удаления поста
    public function destroy($id)
    {
        $images = Images::where('pechnik_id', $id); //получаю колекцию файлов галереи превязанных к ID поста p из таблицы Images
        $pathImages = $images->get('img'); //получаю данные наименование и путь файлов галереи превязанных к ID поста
        if (!$pathImages->isEmpty()){ 
            foreach ($pathImages as $img){ 
                $path = $img->img; 
                Storage::disk('public')->delete(str_replace('storage', '', $path)); //в цикле удаляю фалы галереи превязанных к ID поста
            }
        }
        $images->delete(); // удаляю записи в таблице Images:: о файлах галереи
        $pechnik= Pechnik::find($id);   // получаю данные поста по ID из таблицы Pechnik::
        Pechnik::find($id)->delete();  // удаляю запись в таблице Pechnik:: - данные поста
        return redirect()->route('admin.index')->with('success', 'Информаця успешно удалена');
    }

Answer 2

[Aleksey]

Посмотрите в сторону событий https://laravel.com/docs/8.x/eloquent#events

// В контроллере картинки не трогаем
public function destroy($id)
{
    Pechnik::destroy($id);
    // return ..
}

// Pechnik.php
protected static function booted()
{
    // При удалении удаляем связаные картинки
    static::deleted(function (Pechnik $pechnik) {
        // Обновлено, т.к. mass delete не вызывает событие для модели Image
        // $pechnik->images()->delete();
        // Подход тоже не оптимальный, лишние запросы к БД. Есть что улучшать.
        foreach ($pechnik->images as $image) {
            $image->delete();
        }
    });
}

// Image.php
protected static function booted()
{
    // Удаление картинки из БД - удаляем файл
    static::deleted(function (Image $image) {
        // Удаляем файл
    });
}

Comments

[jazzus]

для такого есть обсерверы

[Aleksey]

jazzus, Да, вы правы конечно. Я просто самый короткий вариант написал, как пример реализации. Не копировать же сюда всю доку.

[vism]

$pechnik->images()->delete();

Мде, думаете это вызовет события у модели Image?

Maксим Волков Документацию читай, а то знаешь в интернете советы дают люди, которые едва ли понимают, что советуют.

[Aleksey]

vism, Ну да, вы правы, при mass delete событие не вызывается.

[vism]

Подход тоже не оптимальный, лишние запросы к БД. Есть что улучшать.

Документацию читать надо, раздел про жадную загрузку, чтоб кучу селектов не делать.

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

[Aleksey]

vism, При чём тут жадная загрузка? SELECT будет 1. Но DELETE в обновлённом варианте вызывается для каждой модели. Про эти лишние запросы я писал. При большом количестве картинок оптимальнее будет пройти по ним в цикле для удаления файлов, а затем сделать изначальный $pechnik->images()->delete();

[vism]

Aleksey, Да, верно, тут нет жадной загрузки. Значит оптимальное.

Про эти лишние запросы я писал. При большом количестве картинок оптимальнее будет пройти по ним в цикле для удаления файлов, а затем сделать изначальный $pechnik->images()->delete();

это экономия на спичках засчёт трудозатрат программистов и будущей поддержки.
Конечно можно предположить, что там будет 10000 картинок, но лучше такое решать и рефакторить когда будет, чем заранее предусматривать сотни разных редких проблем и всё это программировать.
А то бюджет в 100 раз раздуется и время на разработку.

[Aleksey]

vism, Тут соглашусь, если там 3 картинки то, вместо 'оптимизации', получим бессмысленное усложнение кода.

Answer 3

[pLavrenov]

На событие удаления Images нужно реализовать удаление изображения с диска. при таком подходе не важно удаляем мы картинки определенной модели или рандомные.

https://laravel.com/docs/8.x/eloquent#observers