2

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

Вот код:

try
{
    var files = from file in Directory.EnumerateFiles(
                                     @"c:\test\", "*.txt", SearchOption.AllDirectories)
                from line in File.ReadLines(file)
                where line.Contains("NoOneLivesForever")
                select ne
                {
                    File = file,
                    Line = line
                };

    foreach (var f in files)
    {
        Console.WriteLine("{0}\t{1}", f.File, f.Line);
        System.IO.File.Delete(f.File);
    }

    Console.WriteLine("{0} files found.", files.Count().ToString());
}
catch (UnauthorizedAccessException UAEx)
{
    Console.WriteLine(UAEx.Message);
}
catch (PathTooLongException PathEx)
{
    Console.WriteLine(PathEx.Message);
}
2
  • Как это вы файл не открывали, но получили его содержимое? 18 дек 2015 в 9:44
  • Вы бы хоть постеснялись того, что кладёте логику программы в OnClick, и не показывали это, стыдно же. Убрал лишний код.
    – VladD
    18 дек 2015 в 13:52

2 ответа 2

3

File.ReadLines надо заменить на File.ReadAllLines.
Иначе File.Delete вызывается когда файл открыт во время File.ReadLines.

8
  • Плохой совет. Вы при этом заставляете грузить весь файл в память, то есть разменяли одну проблему на другую.
    – VladD
    18 дек 2015 в 13:49
  • @VladD "Вы при этом заставляете грузить весь файл в память" -- не заставляю. Мы не знаем размер файла. и обратите внимание, что в коде указан select new {File=file, Line=line}
    – Stack
    18 дек 2015 в 14:02
  • File.ReadAllLines грузит в память по-любому весь файл же, не смотря на Select.
    – VladD
    18 дек 2015 в 14:08
  • загружается и закрывается, поэтому в момент вызова Delete не происходит ошибка. доступа. Но если итератор обращается к ReadLines, то файл будет открыт до завершения работы итератора.
    – Stack
    18 дек 2015 в 14:15
  • А теперь представьте себе, что в файле 3 гигабайта строк. Зачем грузить его весь в память, если вам нужно только проверить наличие одной строки?
    – VladD
    18 дек 2015 в 14:44
0

Вы делаете что-то лишнее. Делайте просто:

var files = from file in Directory.EnumerateFiles(
                                 path, "*.txt", SearchOption.AllDirectories)
            where File.ReadLines(file).Contains("NoOneLivesForever")
            select file;

foreach (var f in files)
    File.Delete(f);

Вот код с неизменённой по отношению к вопросу семантикой:

var files = from file in Directory.EnumerateFiles(
                                 path, "*.txt", SearchOption.AllDirectories)
            let lines = (from line in File.ReadLines(file)
                         where line.Contains("NoOneLivesForever")
                         select line).ToList()
            where lines.Count > 0
            select new { File = file, Lines = lines };

foreach (var f in files)
{
    foreach (var l in f.Lines)
        Console.WriteLine("{0}\t{1}", f.File, l);
    File.Delete(f.File);
}
8
  • в вопросе: перед удалением файла, строки из него выводятся на Console.
    – Stack
    18 дек 2015 в 14:05
  • @Stack: Выводится только NoOneLivesForever, а не весь файл.
    – VladD
    18 дек 2015 в 14:07
  • дело не в том, что выводится, а в том, что файл остается открытым пока работает итератор, который использует ReadLines. поэтому Delete вызовет ошибку доступа к файлу.
    – Stack
    18 дек 2015 в 14:16
  • Ну да. У меня-то в решении файл закрывается сразу, проблем нет. В чём ваша претензия? В том, что мой код не сможет вывести эту строчку? Может: foreach (var f in files) Console.WriteLine("{0}\tNoOneLivesForever", f);.
    – VladD
    18 дек 2015 в 14:43
  • если код из вопроса: select new { File=file,Line=line}; и Console.WriteLine("{0}\t{1}", f.File, f.Line); копировать в ваш код, то Delete работает без ошибок?
    – Stack
    18 дек 2015 в 14:56

Ваш ответ

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

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