Подтвердить что ты не робот

Как удалить каталог с файлами только для чтения в С#?

Мне нужно удалить каталог, содержащий файлы только для чтения. Какой подход лучше:

  • Используя DirectoryInfo.Delete(), или

  • ManagementObject.InvokeMethod("Delete")?

С DirectoryInfo.Delete(), мне нужно вручную отключить атрибут только для чтения для каждого файла, но ManagementObject.InvokeMethod("Delete"), похоже, не требуется. Есть ли какая-то ситуация, когда предпочтительнее другого?

Пример кода (test.txt только для чтения).

Первый способ:

DirectoryInfo dir = new DirectoryInfo(@"C:\Users\David\Desktop\");
dir.CreateSubdirectory("Test");

DirectoryInfo test = new DirectoryInfo(@"C:\Users\David\Desktop\Test\");
File.Copy(@"C:\Users\David\Desktop\test.txt", @"C:\Users\David\Desktop\Test\test.txt");
File.SetAttributes(@"C:\Users\David\Desktop\Test\test.txt", FileAttributes.Archive);
test.Delete(true);

Второй способ:

DirectoryInfo dir = new DirectoryInfo(@"C:\Users\David\Desktop\");
dir.CreateSubdirectory("Test");

DirectoryInfo test = new DirectoryInfo(@"C:\Users\David\Desktop\Test\");
File.Copy(@"C:\Users\David\Desktop\test.txt", @"C:\Users\David\Desktop\Test\test.txt");

string folder = @"C:\Users\David\Desktop\Test";
string dirObject = "Win32_Directory.Name='" + folder + "'";
using (ManagementObject managementObject = new ManagementObject(dirObject))
{
    managementObject.Get();
    ManagementBaseObject outParams = managementObject.InvokeMethod("Delete", null,
    null);
    // ReturnValue should be 0, else failure
    if (Convert.ToInt32(outParams.Properties["ReturnValue"].Value) != 0)
    {
    }
}
4b9b3361

Ответ 1

Вот метод расширения, который рекурсивно устанавливает Attributes в Normal, а затем удаляет элементы:

public static void DeleteReadOnly(this FileSystemInfo fileSystemInfo)
{
    var directoryInfo = fileSystemInfo as DirectoryInfo;    
    if (directoryInfo != null)
    {
        foreach (FileSystemInfo childInfo in directoryInfo.GetFileSystemInfos())
        {
            childInfo.DeleteReadOnly();
        }
    }

    fileSystemInfo.Attributes = FileAttributes.Normal;
    fileSystemInfo.Delete();
}

Ответ 2

Самый простой способ избежать рекурсивных вызовов - использовать параметр AllDirectories при получении FileSystemInfo s, например:

public static void ForceDeleteDirectory(string path) 
{
    var directory = new DirectoryInfo(path) { Attributes = FileAttributes.Normal };

    foreach (var info in directory.GetFileSystemInfos("*", SearchOption.AllDirectories))
    {
        info.Attributes = FileAttributes.Normal;
    }

    directory.Delete(true);
}

Ответ 3

Попробуйте это,

private void DeleteRecursiveFolder(string pFolderPath)
{
    foreach (string Folder in Directory.GetDirectories(pFolderPath))
    {
        DeleteRecursiveFolder(Folder);
    }

    foreach (string file in Directory.GetFiles(pFolderPath))
    {
        var pPath = Path.Combine(pFolderPath, file);
        FileInfo fi = new FileInfo(pPath);
        File.SetAttributes(pPath, FileAttributes.Normal);
        File.Delete(file);
    }

    Directory.Delete(pFolderPath);
}

Ответ 4

Другой метод без необходимости рекурсии.

public static void ForceDeleteDirectory(string path)
{
    DirectoryInfo root;
    Stack<DirectoryInfo> fols;
    DirectoryInfo fol;
    fols = new Stack<DirectoryInfo>();
    root = new DirectoryInfo(path);
    fols.Push(root);
    while (fols.Count > 0)
    {
        fol = fols.Pop();
        fol.Attributes = fol.Attributes & ~(FileAttributes.Archive | FileAttributes.ReadOnly | FileAttributes.Hidden);
        foreach (DirectoryInfo d in fol.GetDirectories())
        {
            fols.Push(d);
        }
        foreach (FileInfo f in fol.GetFiles())
        {
            f.Attributes = f.Attributes & ~(FileAttributes.Archive | FileAttributes.ReadOnly | FileAttributes.Hidden);
            f.Delete();
        }
    }
    root.Delete(true);
}

Ответ 5

private void DeleteRecursiveFolder(DirectoryInfo dirInfo)
{
    foreach (var subDir in dirInfo.GetDirectories())
    {
        DeleteRecursiveFolder(subDir);
    }

    foreach (var file in dirInfo.GetFiles())
    {
        file.Attributes=FileAttributes.Normal;
        file.Delete();
    }

    dirInfo.Delete();
}

Ответ 6

Лучшее решение - отметить все файлы как нечитаемые, а затем удалить каталог.

// delete/clear hidden attribute
File.SetAttributes(filePath, File.GetAttributes(filePath) & ~FileAttributes.Hidden);

// delete/clear archive and read only attributes
File.SetAttributes(filePath, File.GetAttributes(filePath) 
    & ~(FileAttributes.Archive | FileAttributes.ReadOnly));

Обратите внимание, что ~ является Побитовым логическим оператором, который возвращает дополнение к данному двоичному значению. Я не тестировал это, но он должен работать.

Спасибо!

Ответ 7

Я бы сказал, что ваш первый подход выглядит более ясным и читаемым. Второй метод пахнет отражением, не является безопасным и выглядит странно. ManagementObject может представлять несколько вещей, поэтому не очевидно, что .InvokeMethod("Delete") фактически удаляет каталог.

Ответ 8

То, что мне не нравится в первом подходе (directory.delete), - это тот случай, когда есть подкаталоги, которые также содержат файлы только для чтения, и у них есть подкаталоги, в которых есть файлы только для чтения, и поэтому на. Похоже, вам придется отключить этот флаг для каждого файла в каталоге и всех подкаталогов рекурсивно.

При втором подходе вы можете просто удалить этот первый каталог и не проверить, доступны ли файлы только для чтения. Тем не менее, это первый раз, когда я использовал WMI в С#, поэтому мне не все так нравится. Поэтому я не уверен, когда следует использовать подход WMI для других приложений, а не просто использовать методы System.IO.

Ответ 9

На первый взгляд, использование WMI-подхода кажется более эффективным, чем повторение всей файловой системы (предположим, например, что каталог имеет 10 тысяч файлов). Но я не знаю, что WMI также не выполняет итераций. Если это так, если быть ближе к металлу (опять же, предположения), он должен быть более эффективным.

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

Тестирование производительности должно отвечать на вопрос об эффективности. И либо может быть элегантным, если он завернут в метод расширения DirectoryInfo.

Ответ 10

Вот еще одно решение, которое позволяет избежать рекурсии.

public static void DirectoryDeleteAll(string directoryPath)
{
    var rootInfo = new DirectoryInfo(directoryPath) { Attributes = FileAttributes.Normal };
    foreach (var fileInfo in rootInfo.GetFileSystemInfos()) fileInfo.Attributes = FileAttributes.Normal;
    foreach (var subDirectory in Directory.GetDirectories(directoryPath, "*", SearchOption.AllDirectories))
    {
        var subInfo = new DirectoryInfo(subDirectory) { Attributes = FileAttributes.Normal };
        foreach (var fileInfo in subInfo.GetFileSystemInfos()) fileInfo.Attributes = FileAttributes.Normal;
    }
    Directory.Delete(directoryPath, true);
}

Это работает с атрибутами resettings в папках и файлах перед удалением, поэтому вы можете просто удалить последнюю строку для метода DirectoryResetAttributes и использовать удаление отдельно.

В отношении связанной заметки, в то время как это сработало, у меня возникли проблемы с удалением путей, которые были "слишком длинными", и в итоге использовалось решение robocopy, размещенное здесь: С# удаление папки, которая имеет длинные пути

Ответ 11

Чтобы следить за решением Виталия Улантьева, я добавил его с помощью метода переименования/перемещения:

  public static void renameFolder(String sourcePath, String targetPath) {
     try
     {
        if (System.IO.Directory.Exists(targetPath))
           DeleteFileSystemInfo(new DirectoryInfo(targetPath));
        System.IO.Directory.Move(sourcePath, targetPath);
     }
     catch (Exception ex)
     {
        Console.WriteLine("renameFolder: " + sourcePath + " " + targetPath + " " + ex.Message);
        throw ex;
     }
  }

  private static void DeleteFileSystemInfo(FileSystemInfo fsi) {
     fsi.Attributes = FileAttributes.Normal;
     var di = fsi as DirectoryInfo;

     if (di != null)
     {
        foreach (var dirInfo in di.GetFileSystemInfos())
        {
           DeleteFileSystemInfo(dirInfo);
        }
     }

     fsi.Delete();
  }