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
  • Как это вы файл не открывали, но получили его содержимое? Commented 18 дек. 2015 в 9:44
  • Вы бы хоть постеснялись того, что кладёте логику программы в OnClick, и не показывали это, стыдно же. Убрал лишний код.
    – VladD
    Commented 18 дек. 2015 в 13:52

2 ответа 2

3

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

8
  • Плохой совет. Вы при этом заставляете грузить весь файл в память, то есть разменяли одну проблему на другую.
    – VladD
    Commented 18 дек. 2015 в 13:49
  • @VladD "Вы при этом заставляете грузить весь файл в память" -- не заставляю. Мы не знаем размер файла. и обратите внимание, что в коде указан select new {File=file, Line=line}
    – Stack
    Commented 18 дек. 2015 в 14:02
  • File.ReadAllLines грузит в память по-любому весь файл же, не смотря на Select.
    – VladD
    Commented 18 дек. 2015 в 14:08
  • загружается и закрывается, поэтому в момент вызова Delete не происходит ошибка. доступа. Но если итератор обращается к ReadLines, то файл будет открыт до завершения работы итератора.
    – Stack
    Commented 18 дек. 2015 в 14:15
  • А теперь представьте себе, что в файле 3 гигабайта строк. Зачем грузить его весь в память, если вам нужно только проверить наличие одной строки?
    – VladD
    Commented 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
    Commented 18 дек. 2015 в 14:05
  • @Stack: Выводится только NoOneLivesForever, а не весь файл.
    – VladD
    Commented 18 дек. 2015 в 14:07
  • дело не в том, что выводится, а в том, что файл остается открытым пока работает итератор, который использует ReadLines. поэтому Delete вызовет ошибку доступа к файлу.
    – Stack
    Commented 18 дек. 2015 в 14:16
  • Ну да. У меня-то в решении файл закрывается сразу, проблем нет. В чём ваша претензия? В том, что мой код не сможет вывести эту строчку? Может: foreach (var f in files) Console.WriteLine("{0}\tNoOneLivesForever", f);.
    – VladD
    Commented 18 дек. 2015 в 14:43
  • если код из вопроса: select new { File=file,Line=line}; и Console.WriteLine("{0}\t{1}", f.File, f.Line); копировать в ваш код, то Delete работает без ошибок?
    – Stack
    Commented 18 дек. 2015 в 14:56

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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