Question

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

我创建了一个多线程软件，其中每个线程使用日志记录模块，如下所示：每个线程使用不同的日志文件，因此我认为发生数据冲突的可能性为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 < / strong>，需要 0.2秒或更多才能完成）。我仍在调查这个问题的原因，但首先，我想知道日志记录模块是否已经是线程安全的。

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

至于m_logFormatW，

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

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

Answer 1

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

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

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

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

Answer 2

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

你可以更好地研究一下开源中一些写得很好的logger类，比如

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/

我的日志模块线程安全吗？

2 个答案: