Как работает рекурсивное удаление файлов в php?

Question

[Quintis]

На стековерфлоу нашол функцию для удаления файлов или папок внутри которых могут быть другие файлы или папки :

function Delete($path)
{
    if (is_dir($path) === true)
    {
        $files = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path), RecursiveIteratorIterator::CHILD_FIRST);

        foreach ($files as $file)
        {
            if (in_array($file->getBasename(), array('.', '..')) !== true)
            {
                if ($file->isDir() === true)
                {
                    rmdir($file->getPathName());
                }

                else if (($file->isFile() === true) || ($file->isLink() === true))
                {
                    unlink($file->getPathname());
                }
            }
        }

        return rmdir($path);
    }

    else if ((is_file($path) === true) || (is_link($path) === true))
    {
        return unlink($path);
    }

    return false;
}

Вопрос , для чего эта проверка в функции :

if (in_array($file->getBasename(), array('.', '..')) !== true)

Comments

[Дмитрий]

На самом деле это кривая функция, у итератора есть метод isDot

[Дмитрий]

Эта проверка что ссылка не содержит указатели на текущую и родительскую директории

[Quintis]

Дмитрий, тоесть можно заменить на
if ($file->isDot !== true){

[Дмитрий]

Quintis, просто !$file->isDot()

[Дмитрий]

Quintis, вроде можно было ещё флаг добавить на SKIP_DOTS как то так

[Quintis]

Дмитрий, спасибо !

[Дмитрий]

https://www.php.net/manual/ru/class.recursivedirec...

[Quintis]

Дмитрий,
Но возникла ошибка :

Fatal error: Uncaught Error: Call to undefined method SplFileInfo::isDot()

function Delete($path)
{
    if (is_dir($path) === true)
    {
        $files = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path), RecursiveIteratorIterator::CHILD_FIRST);

        foreach ($files as $file)
        {
            if (!$file->isDot())
            {
                if ($file->isDir() === true)
                {
                    rmdir($file->getPathName());
                }

                else if (($file->isFile() === true) || ($file->isLink() === true))
                {
                    unlink($file->getPathname());
                }
            }
        }

        return rmdir($path);
    }

    else if ((is_file($path) === true) || (is_link($path) === true))
    {
        return unlink($path);
    }

    return false;
}

[Дмитрий]

Quintis,

$files = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path, FilesystemIterator::SKIP_DOTS), RecursiveIteratorIterator::CHILD_FIRST);

И проверку убрать

[Дмитрий]

Quintis, ошибка потому что итератор может возвращать разные объекты

[Quintis]

Дмитрий, работает , но почему не сработал isDot метод ?

[Дмитрий]

Quintis, в документации про флаги почитайте, там можно разные объекты возвращать

[Quintis]

Дмитрий, спасибо , а можете скинуть статью какие обекты когда возвращаются ?

[Дмитрий]

Quintis, ссылка на пхп.нет же выше

[Дмитрий]

Quintis, https://www.php.net/manual/ru/class.filesystemiter...

[Дмитрий]

Quintis, у подобной функции может быть подводный камень, так как rmdir не удалит директорию, если в ней остались файлы, так что тестируйте лучше, я долго возился когда первый раз писал

[Quintis]

Дмитрий, на вордпресе нормально работает , а тестировать функцию на PHPUnit лучше ?

[Дмитрий]

Quintis, ну можно руками, просто например в зависимости от имён, может получиться что удалится не все с директории, и тогда будет печалька
Могу свою кинуть

[Quintis]

Дмитрий, если можете скиньте )

[Quintis]

ну можно руками, просто например в зависимости от имён,

тоесть имена методов RecursiveDirectoryIterator в имена файлов вставить ?

[Дмитрий]

Quintis, ну создайте файлов и папок и пробуйте вызвать свою функцию
Вложите по вкусу

function clear(string $dir): bool
    {
        $iterator = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($dir,
            FilesystemIterator::SKIP_DOTS), RecursiveIteratorIterator::CHILD_FIRST);
        $iterator = iterator_count($iterator) ? $iterator : [];

        foreach ($iterator as $item) {
            $item->isDir() && !$item->isLink() ? rmdir($item->getPathname()) : unlink($item->getPathname());
        }

        return rmdir($dir);
    }

[Quintis]

Дмитрий, ошибка

Fatal error: Uncaught UnexpectedValueException: RecursiveDirectoryIterator::__construct

[Дмитрий]

Quintis, а как передали папку в функцию? Возможно надо обернуть в realpath() путь до папки

[Quintis]

Дмитрий, realpath() помог , я так понимаю ваша функция быстрее той что я нашел ?

[Дмитрий]

Quintis, ну можете померить. Ну она как минимум меньше места занимает))

[Quintis]

Дмитрий, спасибо за помощь !)

Answer 1

[Денис Юрьев]

. - текущий каталог
.. - вышестоящий каталог

проверяет, чтобы рекурсивно не удалить файлы и каталоги выше данного

Comments

[Quintis]

спасибо , можете скинуть ссылку на актуальную статью как работает файловая система в php

[Денис Юрьев]

Quintis, это не php а вообще общепринятая структура каталогов

у меня ща не винда, но думаю там тоже если в строке пути до каталога в конец дописать \..\ то перекинет на каталог выше

[Quintis]

Денис Юрьев, понял , спасибо