c++ - 我的日志记录模块线程安全吗？

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，需要 0.2 秒 或更长时间才能完成)。我仍在调查这个问题的原因，但首先，我想知道日志记录模块是否已经是线程安全的。

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

至于 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。您可能会遇到一些情况，如果管道已满，日志记录会花费一些额外的时间，但这种情况不会经常发生。