前几天上一个网站,看到一篇很好的技术文章。首先简单的介绍一下这个程序。这个程序主要是用来可以在多线程环境中,使用Edit控件显示Log信息。
有一个class CEditLog,封装了windows下的Edit控件,其中包含了一个成员变量用来保存字符串,类型是std::string类型。本来我以为MFC中的CString也有类似的问题,因此打算将它替换成源代码比较容易理解的CString。毕竟谁能看懂windows下STL的源代码?
后来仔细看了一下CString的源代码,结果它还真没有这个问题。惭愧!
class CEditLog : public CEdit
{
void AddLog(const char * pszLog);
LRESUTL OnAddLog(WPARAM, LPARAM);
private:
std::string m_strStore; //表示当前需要插入的Log信息。
CMutex m_mutex; //由于在多线程环境下使用,因此需要保护m_strStore
}
//AddLog的大概实现,多线程环境中使用
void CEditLog::AddLog(const char * pszLog)
{
m_mutex.Lock();
m_strStore += pszLog;
PostMessage(CWM_TEXTCHANGED); //发一个自定义的消息通知主线程有新的Log了。
m_mutex.Unlock();
}
//CWM_TEXTCHANGED的处理函数,在主线程中被调用
LRESULT CEditLog::OnAddLog(WPARAM, LPARAM)
{
std::string strLog;
m_mutex.Lock();
strLog = m_strStore; //为了尽快释放锁,拷贝一个临时对象
m_mutex.Unlock();
//然后处理,为了简单起见,假设我们调用了
.....
SetWindowText(strLog.c_str());
.....
}
结果上述代码中,实际上隐含了一个多线程的Bug,聪明的你发现了吗?