From 28dbf773f146522afe071bf954fdec77b57fd20c Mon Sep 17 00:00:00 2001 From: babaev-an Date: Sun, 8 Jun 2025 11:45:40 +0300 Subject: [PATCH] =?UTF-8?q?20250608=20[File]=20=D0=BE=D0=B1=D0=BD=D0=BE?= =?UTF-8?q?=D0=B2=D0=BB=D1=91=D0=BD=20=D0=BC=D0=B5=D1=82=D0=BE=D0=B4=20Rem?= =?UTF-8?q?oveDir.=20=D0=A2=D0=B5=D0=BF=D0=B5=D1=80=D1=8C=20=D0=BE=D0=BD?= =?UTF-8?q?=20=D0=BA=D0=BE=D1=80=D1=80=D0=B5=D0=BA=D1=82=D0=BD=D0=BE=20?= =?UTF-8?q?=D1=83=D0=B4=D0=B0=D0=BB=D1=8F=D0=B5=D1=82=20=D0=BD=D0=B5=20?= =?UTF-8?q?=D0=BF=D1=83=D1=81=D1=82=D1=83=D1=8E=20=D0=B4=D0=B8=D1=80=D0=B5?= =?UTF-8?q?=D0=BA=D1=82=D0=BE=D1=80=D0=B8=D1=8E.=20[File]=20=D0=B4=D0=BE?= =?UTF-8?q?=D0=B1=D0=B0=D0=B2=D0=BB=D0=B5=D0=BD=20=D0=BC=D0=B5=D1=82=D0=BE?= =?UTF-8?q?=D0=B4=20DirectoryExists=20(string=20$directory,=20bool=20$chec?= =?UTF-8?q?kReadAccess=20=3D=20true,=20=20=20=20=20=20=20=20=20=20=20=20?= =?UTF-8?q?=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20=20bool?= =?UTF-8?q?=20$checkWriteAccess=20=3D=20false):=20bool?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- composer.json | 2 +- composer.lock | 34 +++++++------- sources/classes/File.php | 90 ++++++++++++++++++++++++++------------ tests/classes/FileTest.php | 16 ++++++- 4 files changed, 94 insertions(+), 48 deletions(-) diff --git a/composer.json b/composer.json index 132de5f..38967bf 100644 --- a/composer.json +++ b/composer.json @@ -20,7 +20,7 @@ "ext-mbstring": "*" }, "require-dev": { - "phpunit/phpunit": "^12.1.6" + "phpunit/phpunit": "^12.2.1" }, "autoload": { "psr-4": { diff --git a/composer.lock b/composer.lock index df8a959..c8a915f 100644 --- a/composer.lock +++ b/composer.lock @@ -4,7 +4,7 @@ "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "This file is @generated automatically" ], - "content-hash": "0c5a35b9c8987ac3fd0f7480fea87d4c", + "content-hash": "f5bf0ec4042cb12fb3a702cad65f099d", "packages": [], "packages-dev": [ { @@ -69,16 +69,16 @@ }, { "name": "nikic/php-parser", - "version": "v5.4.0", + "version": "v5.5.0", "source": { "type": "git", "url": "https://github.com/nikic/PHP-Parser.git", - "reference": "447a020a1f875a434d62f2a401f53b82a396e494" + "reference": "ae59794362fe85e051a58ad36b289443f57be7a9" }, "dist": { "type": "zip", - "url": "https://api.github.com/repos/nikic/PHP-Parser/zipball/447a020a1f875a434d62f2a401f53b82a396e494", - "reference": "447a020a1f875a434d62f2a401f53b82a396e494", + "url": "https://api.github.com/repos/nikic/PHP-Parser/zipball/ae59794362fe85e051a58ad36b289443f57be7a9", + "reference": "ae59794362fe85e051a58ad36b289443f57be7a9", "shasum": "" }, "require": { @@ -121,9 +121,9 @@ ], "support": { "issues": "https://github.com/nikic/PHP-Parser/issues", - "source": "https://github.com/nikic/PHP-Parser/tree/v5.4.0" + "source": "https://github.com/nikic/PHP-Parser/tree/v5.5.0" }, - "time": "2024-12-30T11:07:19+00:00" + "time": "2025-05-31T08:24:38+00:00" }, { "name": "phar-io/manifest", @@ -579,16 +579,16 @@ }, { "name": "phpunit/phpunit", - "version": "12.1.6", + "version": "12.2.1", "source": { "type": "git", "url": "https://github.com/sebastianbergmann/phpunit.git", - "reference": "2fdf0056c673c8f0f1eed00030be5f8243c1e6e0" + "reference": "5f09fda04e7caea93cff50b4e90319184f3e6ee3" }, "dist": { "type": "zip", - "url": "https://api.github.com/repos/sebastianbergmann/phpunit/zipball/2fdf0056c673c8f0f1eed00030be5f8243c1e6e0", - "reference": "2fdf0056c673c8f0f1eed00030be5f8243c1e6e0", + "url": "https://api.github.com/repos/sebastianbergmann/phpunit/zipball/5f09fda04e7caea93cff50b4e90319184f3e6ee3", + "reference": "5f09fda04e7caea93cff50b4e90319184f3e6ee3", "shasum": "" }, "require": { @@ -602,7 +602,7 @@ "phar-io/manifest": "^2.0.4", "phar-io/version": "^3.2.1", "php": ">=8.3", - "phpunit/php-code-coverage": "^12.2.1", + "phpunit/php-code-coverage": "^12.3.0", "phpunit/php-file-iterator": "^6.0.0", "phpunit/php-invoker": "^6.0.0", "phpunit/php-text-template": "^5.0.0", @@ -610,7 +610,7 @@ "sebastian/cli-parser": "^4.0.0", "sebastian/comparator": "^7.0.1", "sebastian/diff": "^7.0.0", - "sebastian/environment": "^8.0.1", + "sebastian/environment": "^8.0.2", "sebastian/exporter": "^7.0.0", "sebastian/global-state": "^8.0.0", "sebastian/object-enumerator": "^7.0.0", @@ -624,7 +624,7 @@ "type": "library", "extra": { "branch-alias": { - "dev-main": "12.1-dev" + "dev-main": "12.2-dev" } }, "autoload": { @@ -656,7 +656,7 @@ "support": { "issues": "https://github.com/sebastianbergmann/phpunit/issues", "security": "https://github.com/sebastianbergmann/phpunit/security/policy", - "source": "https://github.com/sebastianbergmann/phpunit/tree/12.1.6" + "source": "https://github.com/sebastianbergmann/phpunit/tree/12.2.1" }, "funding": [ { @@ -680,7 +680,7 @@ "type": "tidelift" } ], - "time": "2025-05-21T12:36:31+00:00" + "time": "2025-06-07T05:17:47+00:00" }, { "name": "sebastian/cli-parser", @@ -1621,4 +1621,4 @@ }, "platform-dev": {}, "plugin-api-version": "2.6.0" -} +} \ No newline at end of file diff --git a/sources/classes/File.php b/sources/classes/File.php index cd55852..ddb8077 100644 --- a/sources/classes/File.php +++ b/sources/classes/File.php @@ -2,12 +2,15 @@ namespace goodboyalex\php_components_pack\classes; +use RecursiveDirectoryIterator; +use RecursiveIteratorIterator; + /** * Класс, реализующий функционал работы с файлами и выполнение операций над файлами одной командой. * * @author Александр Бабаев * @package php_components_pack - * @version 1.0 + * @version 1.0.1 * @since 1.0.21 */ final class File @@ -111,38 +114,67 @@ final class File */ public static function RemoveDir (string $directory): bool { - // Если директории не существует - if (!file_exists($directory)) - // - то удаление не требуется - return true; + // Проверяем, существует ли директория + if (!self::DirectoryExists(directory: $directory, checkWriteAccess: true)) + // - если нет, то возвращаем false + return false; - // Если это не директория - if (!is_dir($directory)) - // - то удаляем файл - return unlink($directory); + // Получаем список файлов и каталогов в заданной директории + $files = new RecursiveIteratorIterator( + new RecursiveDirectoryIterator($directory), + RecursiveIteratorIterator::CHILD_FIRST + ); - // Перебираем все файлы и поддиректории - foreach (scandir($directory) as $item) { - // - если это '.' или '..' - if ($item == '.' || $item == '..') - // -- то пропускаем - continue; - - // - задаём переменную - целевую директорию - $target = $directory . DIRECTORY_SEPARATOR . $item; - - // - если это директория - if (is_dir($target)) - // -- то рекурсивно удаляем её - if (!self::RemoveDir($target)) - // --- и прерываем удаление директории, если ошибка - return false; - else - // -- иначе удаляем файл - unlink($target); - } + // Перебираем файлы и каталоги + foreach ($files as $file) + // - если это каталог + if ($file->isDir()) + // -- то удаляем его + rmdir($file->getRealPath()); + else + // -- иначе удаляем файл + unlink($file->getRealPath()); // Удаляем директорию return rmdir($directory); } + + /** + * Проверяет, существует ли директория. + * + * @param string $directory Путь к директории. + * @param bool $checkReadAccess Проверять ли доступ на чтение директории (по умолчанию true). + * @param bool $checkWriteAccess Проверять ли доступ на запись директории (по умолчанию false). + * + * @return bool Результат проверки. + */ + public static function DirectoryExists (string $directory, bool $checkReadAccess = true, + bool $checkWriteAccess = false): bool + { + // Очищаем кэш + clearstatcache(); + + // Проверяем, существует ли директория + if (!file_exists($directory)) + // - если нет, то возвращаем false + return false; + + // Проверяем, является ли директория директорией, а не файлом + if (!is_dir($directory)) + // - если нет, то возвращаем false + return false; + + // Проверяем, есть ли доступ на чтение директории + if ($checkReadAccess && !is_readable($directory)) + // - если нет, то возвращаем false + return false; + + // Проверяем, есть ли доступ на запись директории + if ($checkWriteAccess && !is_writable($directory)) + // - если нет, то возвращаем false + return false; + + // Если все проверки пройдены успешно, то возвращаем true + return true; + } } \ No newline at end of file diff --git a/tests/classes/FileTest.php b/tests/classes/FileTest.php index 2bbe3d2..c5a01c0 100644 --- a/tests/classes/FileTest.php +++ b/tests/classes/FileTest.php @@ -59,4 +59,18 @@ class FileTest extends TestCase $this->assertEquals("test", File::ExtractFileNameWithoutExtension("c:/tmp/test.txt")); $this->assertEquals("test", File::ExtractFileNameWithoutExtension("\\tmp\\test.txt")); } -} + + public function testRemoveDir () + { + /** + * ВАЖНО! Перед запуском теста необходимо создать директорию D:\TestDelete и наполнить её ненужными файлами + */ + + $this->PrepareForTest(); + + $result = File::RemoveDir("D:\\TestDelete"); + + $this->assertTrue($result); + $this->assertFalse(File::DirectoryExists("D:\\TestDelete")); + } +} \ No newline at end of file