c# - 閉じた TextWriter に書き込めません

Question

txt ファイルにテキストを書き込もうとしています。最初の書き込み後、エラーでアプリケーションがクラッシュします

閉じた TextWriter に書き込めません

私のリストには、ブラウザーが開くリンクが含まれており、それらすべてを txt ファイル (ログなど) に保存したいと考えています。

私のコード:

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) { }
    }
}

score 13 · Accepted Answer

あなたはループしていますが、最初の反復forで閉じて破棄します:StreamWriter

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

代わりに、そのブロックを削除し、すべてを 1 つのusingブロックにラップします。

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

        sw.WriteLine(link);

        Thread.Sleep((int)delayTime);

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

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

score 3 · Accepted Answer

この線

using (sw)

を閉じる/破棄しますStreamWriter。

ループしているので、すでに破棄されているを破棄しますStreamWriter。

すべての書き込み操作が完了したら、ループのStreamWriter 外側を閉じることをお勧めします。

さらに、例外をキャッチし、キャッチされた例外を無視することは、ほとんどの場合、悪い考えです。例外を処理できない場合は、キャッチしないでください。

score 2 · Accepted Answer

問題は、ストリームをループで閉じていることです。後でのみ行う必要があります...

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();

score 1 · Accepted Answer

これは、ループの途中で実際にストリームを閉じているためです。using (sw)真ん中にブロックがあり、ループの最初の実行では問題なく動作し、その後forクラッシュします。これを修正するには、sw.Close()呼び出しをドロップし、ループusingの外に移動します。for

score 0 · Accepted Answer

sw.Close()ファイルが閉じているとコードがファイルを読み取れないため、コードにを記述しないでください。

c# - 閉じた TextWriter に書き込めません

5 に答える 5

Related

Reference