Не удается написать закрытому TextWriter

Question

Я пытаюсь записать текст в файл txt. После того, как первый писать аварии приложения с ошибкой

Невозможно записать в закрытом TextWriter

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

Мой код:

FileStream fs = new FileStream(
        "c:\\linksLog.txt", FileMode.Append, FileAccess.Write); 
StreamWriter sw = new StreamWriter(fs); 

for (int i = 0; i < linksList.Count; i++) 
{ 
    try 
    { 
     System.Diagnostics.Process.Start(browserType, linksList[i]); 
    } 
    catch (Exception) { } 

    using (sw) 
    { 
     sw.WriteLine(linksList[i]); 
     sw.Close(); 
    } 

    Thread.Sleep((int)delayTime); 

    if (!cbNewtab.Checked) 
    { 
     try 
     { 
      foreach (Process process in Process.GetProcesses()) 
      { 
       if (process.ProcessName == getProcesses) 
       { 
        process.Kill(); 
       } 
      } 
     } 
     catch (Exception) { } 
    } 
} 

Answer 1

Вы в for цикле, но закрыть и утилизировать ваши StreamWriter на первой итерации:

using (sw) 
{ 
    sw.WriteLine(linksList[i]); 
    sw.Close(); 
} 

В вместо него, удалить этот блок, и обернуть все в одном блоке using:

using (var fs = new StreamWriter(@"C:\linksLog.txt", true)) { 
    foreach (var link in linksList) { 
     try { 
      Process.Start(browserType, list);       
     } catch (Exception) {} 

     Thread.Sleep((int)delayTime); 

     if (!cbNewtab.Checked) { 
      var processes = Process.GetProcessesByName(getProcesses); 

      foreach (var process in processes) { 
       try { 
        process.Kill(); 
       } catch (Exception) {} 
      } 
     } 
    } 
} 

Answer 2

Проблема заключается в том, что вы закрываете вы поток в цикле, должны выполняться только после того, как ...

FileStream fs = new FileStream("c:\\linksLog.txt", FileMode.Append, FileAccess.Write); 
StreamWriter sw = new StreamWriter(fs); 

    for (int i = 0; i < linksList.Count; i++) 
    { 
     try 
     { 
      System.Diagnostics.Process.Start(browserType, linksList[i]);       
     } 
     catch (Exception) 
     { 

     } 
     // Removed the using blocks that closes the stream and placed at the end of loop 
     sw.WriteLine(linksList[i]); 

     Thread.Sleep((int)delayTime); 

     if (!cbNewtab.Checked) 
     { 
      try 
      { 
       foreach (Process process in Process.GetProcesses()) 
       { 
        if (process.ProcessName == getProcesses) 
        { 
         process.Kill(); 
        } 
       } 
      } 
      catch (Exception) 
      { } 
     } 
    } 

    sw.Close(); 

Answer 3

Линия

using (sw) 

закрывает/располагает свой StreamWriter.

Поскольку вы зацикливаете, вы располагаете уже установленным StreamWriter.

Лучше закрыть StreamWriterснаружи цикл, после завершения всех операций записи.

Кроме того, перехватывание исключений и игнорирование пойманного исключения - это почти всегда плохая идея. Если вы не можете обработать исключение, не поймите его.

Answer 4

Это потому, что вы действительно закрываете свой поток в середине цикла. У вас есть блок using (sw) в середине, который отлично работает в первом прогоне через цикл for, а затем сбой. Чтобы исправить это, просто перенесите sw.Close() вызов, и переместить using быть вне цикла for:

Answer 5

Dont писать sw.Close() в коде, потому что, если файл закрыт код не может прочитать файл.