Question

我有一个日志文件旋转功能，如下所示：

    private static void RotateLogs()
    {
        FileInfo LogFile = new FileInfo(@"C:\temp\dataTransferErrorLog.txt");

        if (LogFile.Exists && (LogFile.Length) >= 10 * 1048576)
        {
            Compress(LogFile);
            LogFile.Delete();
            File.Create(@"C:\temp\dataTransferErrorLog.txt");
        }
    }

    private static void Compress(FileInfo fi)
    {
        // Get the stream of the source file.
        using (FileStream inFile = fi.OpenRead())
        {
            // Prevent compressing hidden and 
            // already compressed files.
            if ((File.GetAttributes(fi.FullName)
                & FileAttributes.Hidden)
                != FileAttributes.Hidden & fi.Extension != ".gz")
            {
                string destFileName = fi.FullName.Substring(0, fi.FullName.LastIndexOf('.')) + System.DateTime.Now.ToString("yyyyMMddHHmm") + fi.FullName.Substring(fi.FullName.LastIndexOf('.')) + ".gz";
                // Create the compressed file.
                using (FileStream outFile =
                            File.Create(destFileName))
                {
                    using (GZipStream Compress =
                        new GZipStream(outFile,
                        CompressionMode.Compress))
                    {
                        // Copy the source file into 
                        // the compression stream.
                        inFile.CopyTo(Compress);
                    }
                }
            }
        }
    }

每次尝试旋转日志文件时都会抛出IOException。我认为它在尝试写入.gz文件时抛出异常，但我不确定。这是堆栈跟踪：

Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.IO.IOException
Stack:
   at System.IO.__Error.WinIOError(Int32, System.String)
   at System.IO.FileStream.Init(System.String, System.IO.FileMode, System.IO.FileAccess, Int32, Boolean, System.IO.FileShare, Int32, System.IO.FileOptions, SECURITY_ATTRIBUTES, System.String, Boolean, Boolean)
   at System.IO.FileStream..ctor(System.String, System.IO.FileMode, System.IO.FileAccess, System.IO.FileShare, Int32, System.IO.FileOptions)
   at System.IO.StreamWriter.CreateFile(System.String, Boolean)
   at System.IO.StreamWriter..ctor(System.String, Boolean, System.Text.Encoding, Int32)
   at System.IO.StreamWriter..ctor(System.String, Boolean)
   at System.IO.File.AppendText(System.String)
   at XRayDataTransferService.XRayDataTransferService.LogMessage(System.String)
   at XRayDataTransferService.XRayDataTransferService.RunAgent()
   at System.Threading.ThreadHelper.ThreadStart_Context(System.Object)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
   at System.Threading.ThreadHelper.ThreadStart()

在尝试将压缩信息写入.gz文件时，有人可以确认它是在抛出异常，并告诉我这种情况会导致它抛出异常吗？

修改

这是LogMessage函数。大部分时间都有效，它抛出异常的唯一时间就是旋转日志。

static void LogMessage(string messageText)
    {
        string ErrorLogFileName = @"C:\temp\dataTransferErrorLog.txt";

        using (StreamWriter Log = File.AppendText(ErrorLogFileName))
        {
            try
            {
                Log.WriteLine("{0}: {1}", dateStamp, messageText);
            }
            catch { }
        }
    }

更新：

static void RunAgent()
    {
        while (!shutdown)
        {
            LogMessage("Starting data transfer...");
            errors = 0;

           // Do some data processing

            LogMessage("Finished running with " + errors.ToString() + " error(s).");

            RotateLogs();
        }
        shutdownSignal.Set();
    }

我在下面的评论中说过，但是很明显，只有一个线程正在运行。这是一个服务，所以它必须在一个单独的线程中，但只有一个线程在运行。

Answer 1

编辑：根据您的评论，我已经尝试过您的代码并发现了您遇到的问题。当您拨打此行File.Create(@"C:\temp\dataTransferErrorLog.txt");时，它会返回永不关闭的FileStream。因此，您可以使用两种方法。首先，您需要将FileStream分配给变量，并在Close()上明确调用FileStream，如：

if (!LogFile.Exists)
{
        Compress(LogFile);
        LogFile.Delete();
        FileStream fs = File.Create(@"C:\Users\On\Documents\dataTransferErrorLog.txt");
        fs.Close();
}

或更好的方法是甚至不在那里创建文件，因为稍后调用File.AppendText()将创建它不存在的文件。因此，我建议您按照以下方式RotateLogs创建方法：

private static void RotateLogs()
{
    FileInfo LogFile = new FileInfo(@"C:\temp\dataTransferErrorLog.txt");

    if (LogFile.Exists && (LogFile.Length) >= 10 * 1048576)
    {
         Compress(LogFile);
         LogFile.Delete();
    }
}

Answer 2

在堆栈跟踪中，此代码中未发生异常。在LogMessage函数中，您正在调用AppendText，这就是失败的地方。在代码顶部添加一个异常处理程序（主应用程序入口点）和日志（如，直接写入文本文件或写入控制台，没有花哨的日志记录，只需获取数据），异常的详细信息将给出你有更多的信息。

作为第二个建议，log4net是一个非常有用的日志库。它内置了一个滚动文件追加器。它易于配置和设置，并可通过NuGet获得。您可能需要考虑使用它。很多产品都有。

Answer 3

将您的pdbs与程序集一起部署，您将在堆栈跟踪中获得行号，这将使您能够指出抛出异常的位置。你确实生成了pdbs吗？

Answer 4

它不在旋转功能中。对我来说，看起来你正在进行旋转，同时你想要附加到文件（写日志），所以它崩溃。

但发布例外消息，这将清除所有内容

Answer 5

我认为你试图用一个线程在你的日志中写一些东西而另一个线程试图旋转日志......

只需更改代码就可以更好地了解正在发生的事情

static void LogMessage(string messageText)
{
    try
    {
        string ErrorLogFileName = @"C:\temp\dataTransferErrorLog.txt";

        using (StreamWriter Log = File.AppendText(ErrorLogFileName))
        {
            Log.WriteLine("{0}: {1}", dateStamp, messageText);
        }
     }
     catch(Exception) { throw; }
}

无论如何，您最好的选择是使用强大的日志库作为log4net或NLog。

他们的工作非常好

日志文件循环IOException

5 个答案: