1

伙计们，我是线程和日志记录的初学者。顺便说一句，我不是以英语为母语的人，如果我的英语有任何错误，请原谅我。

我创建了一个多线程软件，其中每个线程使用如下日志模块：每个线程使用不同的日志文件，所以我认为发生数据冲突的机会是 0。

__inline void print_logW(int _level,const wchar_t *domain,const wchar_t *msg)
{
    wchar_t mess[200] = _T("");

    if(_level<=traceLevel)
    {
        __time64_t timer;
        struct tm t_st;
        _time64(&timer);

        localtime_s(&t_st,&timer);

        if (domain == NULL)
        {
            domain = _T("");
        }
        if (msg != NULL)
        {
            if (showTimeStampFlag == true)
            {
                swprintf_s(mess,200,_T("%s : %ld"),msg,GetTickCount());
            }
            else
            {
                wcscpy_s(mess,200,msg);
            }
        }

        if(oldTime.tm_year != t_st.tm_year || oldTime.tm_mon != t_st.tm_mon || oldTime.tm_mday != t_st.tm_mday) 
        {
            oldTime = t_st;
            print_log_preparebyDateW();
        }


        FILE* fp;
        errno_t err = _wfopen_s(&fp, this->m_pathW, _T("at+, ccs=UTF-8"));
        if (err != 0)
        {
            // error
            return;
        }

        fwprintf_s(fp, m_logFormatW,
            _level,
            1900 + t_st.tm_year, t_st.tm_mon + 1, t_st.tm_mday,
            t_st.tm_hour, t_st.tm_min, t_st.tm_sec,
            domain, mess
            );
        fflush(fp);
        fclose(fp);
    }
}

当我看到我制作的软件的日志时，我发现了一个问题，有时线程进程变得如此缓慢（一个进程（例如获取图像的指针）通常只需要16 ms max，将需要0.2 秒或更多完成）。我仍在调查此问题的原因，但首先，我想知道日志记录模块是否已经是线程安全的。

顺便说一句，对于参数，
"_level"是打印或取消打印进程的详细信息的日志级别，
我"domain"用来显示执行日志记录的类是日志
"msg"的内容（例如“进程 1 启动”

）m_logFormatW,

m_logFormatW = _T("[%.2d][%.4d-%.2d-%.2dT%.2d:%.2d:%.2d][%s] %s\n");

如果有任何问题或不清楚的地方，请随时提问。

4

2 回答 2

1

只要您链接到多线程运行时库并且oldTime不是全局或静态变量，您的日志函数将是线程安全的。如果oldTime是全局变量或静态变量，您在访问或修改它时需要序列化对它的访问，否则您将面临竞争条件的风险。唯一可能不是线程安全的另一件事是，print_log_preparebyDateW但很难说，因为您没有包含它的代码。只要oldTime不是全局或静态的，并且您使用的所有运行时库函数都被标记为线程安全或者是 MSDN 中标记为线程安全的库的一部分，您就可以了。

我能看到的唯一另一个问题是打开文件时。如果文件已经打开并且另一个线程尝试记录信息，则打开调用将失败，从而导致信息丢失。这是因为_wfopen_s在没有任何共享模式的情况下打开文件。您可以通过在文件打开时使用std::mutex并锁定它并在文件关闭后解锁它来解决此问题。

您的工作线程执行时间比预期长的一个可能原因是打开日志文件、写入信息、刷新文件并关闭它可能需要一些额外的时间。即使在涉及缓存的情况下，只要发生文件 I/O，就会发生这种情况。通常，您可以通过打开日志文件一次然后在应用程序终止时将其关闭来减少时间。

另一种减少工作线程执行时间的可能解决方案是使用管道。在这种情况下，您将日志文本写入管道，并有一个额外的线程从管道读取并写入日志文件。这将消除工作线程记录信息时可能发生的任何磁盘 I/O。您可能会遇到一些情况，如果管道已满，日志记录会花费一些额外的时间，但这种情况不会经常发生。

于 2013-06-17T04:58:27.360 回答
0

您的方法适用于单线程应用程序，并且在多线程环境中不起作用，因为您没有序列化日志消息请求。

最好在开源中查看一些编写良好的记录器类，例如

a) AsynchronousAndSynchronouslogger - http://www.codeproject.com/Articles/288827/g2log-An-efficient-asynchronous-logger-using-Cplus

b) Simplethreadsafe - http://cpplogging.codeplex.com/

c) Log4Cpp - http://log4cpp.sourceforge.net/

于 2013-06-17T03:41:50.813 回答