Как избежать ошибки Необработанное исключение типа «System.IO.IOException» в mscorlib.dll?

Question

[skvoshiz]

Здравствуйте, иногда вылетает данная ошибка:
Необработанное исключение типа "System.IO.IOException" в mscorlib.dll

Сам код:

using System;
using System.IO;
 
namespace TurboBotNew
{
    public static class Log
    {
        static string s_logFile = System.IO.Directory.GetCurrentDirectory() + "\\log.txt";
        static int i_lastLog = 0;
 
        public static void Write(string message)
        {
            string TimeNow = DateTime.Now.ToString("HH:mm:ss");
            message = "[" + TimeNow + "] " + message + "\n";
            File.AppendAllText(s_logFile, message);
 
        }
 
        public static void Close()
        {
            if (System.IO.File.Exists(s_logFile))
                System.IO.File.Delete(s_logFile);
        }
 
        public static string Read(string s_log)
        {
            if (!System.IO.File.Exists(s_logFile))
                Write("Создание лога");
            var mass = System.IO.File.ReadAllLines(s_logFile);
            string s_message = "";
            for (int i = mass.Length - 1; i > i_lastLog; i--)
            {
                s_message = mass[i] + Environment.NewLine + s_message;
            }
            i_lastLog = mass.Length - 1;
 
            return s_log + s_message;
        }
    }
}

Как в данном коде избежать этой ошибки?

Answer 1

[mayorovp]

Ошибка возникает потому что вы пытаетесь открыть файл лога из нескольких потоков одновременно. Или вы просто одновременно открыли его в текстовом редакторе.

PS
Совет номер 1: откройте для себя NLog
Совет номер 2: никогда не делайте вот так в цикле

s_message = mass[i] + Environment.NewLine + s_message;

- для таких вещей есть StringBuilder, который работает быстрее.
Совет номер 3: А еще лучше - вместо таких циклов использовать удобный метод

s_message  = string.Join(Environment.NewLine, mass)

Совет номер 4: Еще лучше - использовать File.ReadAllText вместо File.ReadAllLines в таких случаях.

Comments

[skvoshiz]

NLog понравилось, буду изучать и применять данную библиотекую

Answer 2

[Александр Таратин]

Используйте StreamWriter для записи.
Открыли один раз при первой попытке записи и закрыли перед завершением программы.

Comments

[Алексей Немиро]

Да, методы ReadAllLines и AppendAllText, как и другие подобные методы чтения и изменения файлов из класса File, лучше не использовать, т.к. можно получить блокировку доступа к файлам.

Проблема может быть, если приложение вызывает одновременно несколько методов для записи в журнал. Либо при чтении файл блокируется.

Сам я частенько использую, в маловажных проектах (обычно во всяких роботах), запись журналов методом AppendAllText, ибо это просто и всего одна строчка кода. И все это на 100% в одном потоке делается. Совмещать с чтением этого же файла, я бы не стал, т.к. наверняка возникнут проблемы с доступом, т.к. нет контроля над потоком.

Про блоки try ... catch не стоит забывать. Весь код работы с файлами, особенно с файлами журналов (они обычно играют второстепенное значение), необходимо помещать в блоки try ... catch. При желании, можно реализовать ожидание доступа к файлу, используя перехват исключений.

Когда-то писал статью на тему работы с файлами в C#. Ниже некоторые примеры кода из нее.
Запись в текстовой файл:

using (var fs = new FileStream(@"C:\Мой файл.txt", FileMode.OpenOrCreate, FileAccess.Write, FileShare.ReadWrite))
{
 using (var sw = new StreamWriter(fs, Encoding.UTF8))
 {
   sw.WriteLine("новая запись");
 }
}

Чтение из текстового файла:

using (var fs = new FileStream(@"C:\Мой файл.txt", FileMode.Open, FileAccess.Read, FileShare.ReadWrite))
{
  using (var sr = new StreamReader(fs, Encoding.UTF8))
  {
    // читаем весь файл
    Console.WriteLine(sr.ReadToEnd());
  }
}

Вот еще пример метода для удаления файла, с возможностью указать время ожидания доступа к файлу:

public void DeleteFile(string path, TimeSpan accessTimeout)
{
  var interval = new TimeSpan(0, 0, 0, 0, 100);
  var totalTime = new TimeSpan();
  while (File.Exists(path))
  {
    try
    {
      File.Delete(path);
    }
    catch (IOException ex)
    {
      Thread.Sleep(interval);
      totalTime += interval;
      if (accessTimeout.Ticks == 0 || totalTime > accessTimeout)
      {
        throw ex;
      }
    }
  }
}