Процесс не может получить доступ к файлу, потому что он используется другим процессом с использованием streamwriter

Question

У меня есть 2 программы: приложение C# GUI и служба Windows C#, обращающиеся к одному и тому же текстовому файлу;

a) the C# GUI application will write/append to the text file 
b) the windows service will copy the file to a network location every 20 mins. 

Когда действие происходило одновременно, я получил сообщение об ошибке, как показано ниже:

2014/09/08 21:15:56 mscorlib 
The process cannot access the file 'C:\09082014.log' because it is being used by another process. 
    at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath) 
    at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy) 
    at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options) 
    at System.IO.StreamWriter.CreateFile(String path, Boolean append) 
    at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize) 
    at System.IO.StreamWriter..ctor(String path, Boolean append) 
    at DataloggerUI.DataHelper.WriteDataLog(String msg, Int64& downTimeSince) 
    at DataloggerUI.Form1.ReceiveData(IAsyncResult asyncResult) 

---- часть C# Windows сервис, как показано ниже ----------

if (File.Exists(destination + @"\" + fi.Name)) 
      { 
       FileInfo fi_dest = new FileInfo(destination + @"\" + fi.Name); 
       if (fi.LastWriteTime > fi_dest.LastWriteTime) 
       { 
        File.Copy(fi.FullName, destination + @"\" + fi.Name, true); 
        WriteLog("Send " + fi.FullName + " to server"); 
       } 
      } 
      else 
      { 
       File.Copy(fi.FullName, destination + @"\" + fi.Name, true); 
       WriteLog("Send " + fi.FullName + " to server"); 
      } 
} 

------- C# код окна приложения GUI как ниже -------

string logfile = DataHelper.GetAppConfigString("MPRS_LogDir") + @"\" + DateTime.Now.ToString("MMddyyyy") + ".log"; 
    using (StreamWriter sw = new StreamWriter(logfile, true)) 
    { 
     sw.WriteLine(tick + " " + "KYEC" + Environment.MachineName + " " + msg); 
     sw.Close(); 
    } 

Сообщение об ошибке высылается приложением GUI. Какая у вас ошибка или плохая практика?

------------ модифицированный код к следующему согласно совету Петра --------------

try 
    { 
     using (StreamWriter sw = new StreamWriter(logfile, true)) 
     { 
      sw.WriteLine(tick + " " + "KYEC" + Environment.MachineName + " " + msg); 
     } 
    } 
    catch (IOException ex) 
    { 
     WriteErrorLog("IOException " + ex.Message); 
     System.Threading.Thread.Sleep(2000); //2 secs     
     using (StreamWriter sw = new StreamWriter(logfile, true)) 
     { 
      sw.WriteLine(tick + " " + "KYEC" + Environment.MachineName + " " + msg); 
     } 
    } 

Answer 1

основе Peter Duniho's answer и собственных правок это должно быть правильный подход:

// try to write maximum of 3 times 
var maxRetry = 3; 
for (int retry = 0; retry < maxRetry; retry++) 
{ 
    try 
    { 
     using (StreamWriter sw = new StreamWriter(logfile, true)) 
     { 
      sw.WriteLine("{0} KYEC{1} {2}", tick, Environment.MachineName, msg); 

      break; // you were successfull so leave the retry loop 
     } 
    } 
    catch (IOException) 
    { 
     if(retry < maxRetry - 1) 
     { 
      System.Threading.Thread.Sleep(2000); // Wait some time before retry (2 secs) 
     } 
     else 
     { 
      // handle unsuccessfull write attempts or just ignore. 
     } 
    } 
} 

это дает возможность определить, как долго вы повторить ваши попытки записи.

Answer 2

У вас есть по крайней мере, здесь есть несколько вариантов. Простейшим концептуальным является просто перенос ввода/вывода файлов в try/catch, и если возникает событие IOException (и только если возникает событие IOException), задержите операцию ненадолго (например, Thread.Sleep(), установите таймер) и повторите попытку ,

Альтернативным подходом было бы использовать именованный мьютекс, чтобы позволить сервису и графическому интерфейсу осуществлять координацию доступа. Каждый из них должен был получить мьютекс, прежде чем пытаться выполнить их соответствующую операцию; если другой в настоящее время обращается к файлу, то попытка получить мьютекс заставит процесс ждать, пока другой не будет выполнен.

код Retry иногда может запутаться и многословным, поэтому, когда я думаю, что повторная попытка подход легче понять, ИМХО мьютекс подход на самом деле более читаемым и проще получить правильно (как только вы пошли в беде узнать, как создать именованный mutex & hellip, это не сложно, и в MSDN есть примеры).

Answer 3

Вы можете использовать FileStream в режиме Shared ReadWrite, чтобы одновременно записывать и копировать файл.

Попробуйте следующий код:

//To write file use 
    using (FileStream fs = new FileStream(fileToReadPath, FileMode.Append, FileAccess.Write, FileShare.ReadWrite)) 
    { 
     using (StreamWriter StreamWriter = new StreamWriter(fs)) 
     { 
      StreamWriter.WriteLine(tick + " " + "KYEC" + Environment.MachineName + " " + msg); 
      StreamWriter.Close(); 
     } 
    } 

//To copy file use 
    using (FileStream inStream = new FileStream(fileToReadPath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite)) 
    { 
     using (FileStream outStream = File.Create(fileToWritePath)) 
     { 
      while (inStream.Position < inStream.Length) 
      { 
       outStream.WriteByte((byte)inStream.ReadByte()); 
      } 
     } 
    } 

Таким образом, вы можете достичь своей задачи без каких-либо используется другой ошибки процесса.