3

Нужно получить все файлы по заданной маске. Метод EnumerateFiles выдает исключение, если какой-то файл имеет системные права доступа. Как это обойти?

var sourceDir = new DirectoryInfo(path);
foreach (var f in sourceDir.EnumerateFiles("*.txt", SearchOption.AllDirectories))
{
     var lvItem = new ListViewItem(Path.GetFileName(f.FullName), 0);
     lvItem.SubItems.Add(Path.GetDirectoryName(f.FullName));
     var fileInfo = new FileInfo(f.FullName);
     lvItem.SubItems.Add(fileInfo.Length + " байт");
     lvItem.SubItems.Add(Directory.GetLastWriteTime(f.FullName).ToShortDateString());
     uiContext.Send(d => listView1.Items.Add(lvItem), null);
}
5
  • 1
    Судя по всему, ничего лучше ручной рекурсии с отловом исключений нет.
    – VladD
    13 янв 2016 в 9:57
  • 2
    Рекурсивный обход. Пример можно посмотреть тут (многопоточность можете в расчет не брать): ru.stackoverflow.com/a/431921/106
    – andreycha
    13 янв 2016 в 10:15
  • 2
    Мне кажется здесь метка winforms не к месту, лучше бы её заменить на .net
    – Bald
    13 янв 2016 в 10:26
  • 1
    @Bald56rus в коде вопроса используется класс ListViewItem из System.Windows.Forms 13 янв 2016 в 10:36
  • 3
    @МстиславПавлов <зануда>а в тексте вопроса нет, данный вопрос можно решить без участия компонентов winforms, ведь проблема же не в добавлении списка в компонент а в обходе директории</зануда>
    – Bald
    13 янв 2016 в 10:39

2 ответа 2

4

Судя по всему, ничего лучше ручной рекурсии с отловом исключений нет.

Можно попробовать так:

static IEnumerable<FileInfo> EnumerateFilesDeepIgnoringAccessException(
    DirectoryInfo root, string mask)
{
    var localResult = new List<FileInfo>();
    try
    {
        localResult = root.EnumerateFiles(mask).ToList();
    }
    catch (UnauthorizedAccessException)
    {
        // ignore it
        yield break;
    }

    foreach (var fi in localResult)
        yield return fi;

    foreach (var di in root.EnumerateDirectories())
    {
        foreach (var fi in EnumerateFilesDeepIgnoringAccessException(di, mask))
            yield return fi;
    }
}

Код основан на официальном примере, модифицирован для ленивого обхода.


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


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

8
  • 1
    "localResult = root.EnumerateFiles(mask).ToList();" -- если файлов 10000. в них только 10-й выдает UnauthorizedAccess, то что будет в localResult?
    – Stack
    13 янв 2016 в 11:16
  • 1
    @Stack: UnauthorizedAccess выдаёт не получение FileInfo, а доступ к root. То есть, это случится не на 10-ом файле, а до первого.
    – VladD
    13 янв 2016 в 11:17
  • 1
    "UnauthorizedAccess выдаёт не получение FileInfo, а доступ к root" -- если так, то до ToList() вызов не доходит. и он же не нужен?
    – Stack
    13 янв 2016 в 11:35
  • 1
    и с файлами может быть PathTooLongException
    – Stack
    13 янв 2016 в 11:37
  • 1
    @Stack: EnumerateFiles возвращает ленивую последовательность, поэтому вполне возможно, что исключение будет выброшено лишь при попытке энумерации. Это было бы логично: доступ к файловой системе для проверки прав не должен происходить до начала фактической энумерации (иначе теряется смысл ленивости). (Впрочем, я не проверял.)
    – VladD
    13 янв 2016 в 11:38
3
using System;
using System.Collections;
using System.Collections.Generic;
using System.IO;
using System.Linq;

public sealed class FileSystemEnumerable : IEnumerable<FileSystemInfo>
{    
    private readonly DirectoryInfo _root;
    private readonly IList<string> _patterns;
    private readonly SearchOption _option;

    public FileSystemEnumerable(DirectoryInfo root, string pattern, SearchOption option)
    {
        _root = root;
        _patterns = new List<string> { pattern };
        _option = option;
    }

    public FileSystemEnumerable(DirectoryInfo root, IList<string> patterns, SearchOption option)
    {
        _root = root;
        _patterns = patterns;
        _option = option;
    }

    public IEnumerator<FileSystemInfo> GetEnumerator()
    {
        if (_root == null || !_root.Exists) yield break;

        IEnumerable<FileSystemInfo> matches = new List<FileSystemInfo>();
        try
        {
            foreach (var pattern in _patterns)
            {
                matches = matches.Concat(_root.EnumerateDirectories(pattern, SearchOption.TopDirectoryOnly))
                                 .Concat(_root.EnumerateFiles(pattern, SearchOption.TopDirectoryOnly));
            }
        }
        catch (UnauthorizedAccessException)
        {
            yield break;
        }
        catch (PathTooLongException ptle)
        {
            yield break;
        } catch (System.IO.IOException e)
        {
            yield break;
        }

        foreach (var file in matches)
        {
            yield return file;
        }

        if (_option == SearchOption.AllDirectories)
        {
            foreach (var dir in _root.EnumerateDirectories("*", SearchOption.TopDirectoryOnly))
            {
                var fileSystemInfos = new FileSystemEnumerable(dir, _patterns, _option);
                foreach (var match in fileSystemInfos)
                {
                    yield return match;
                }
            }
        }
    }

    IEnumerator IEnumerable.GetEnumerator()
    {
        return GetEnumerator();
    }
}

Использование:

var root = new DirectoryInfo(path);
var searchPattern = @"*.txt";
var searchOption = SearchOption.AllDirectories;
var enumerable = new FileSystemEnumerable(root, searchPattern, searchOption);

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.