我使用QtConcurrent在后台线程中执行发现过程:
// Start the discover process in the background thread so it doesn't block the gui
*m_Future = QtConcurrent::run(this, &Controller::StartDiscover);
m_Watcher->setFuture(*m_Future);
我之前没有使用Mutex来保护该函数内部的共享变量,这些变量可以在后台线程运行时访问。这在我的“扫描”操作期间造成了一些随机的锁定。
我现在已经实现了互斥锁,即在该函数的开头创建一个QMutexLocker。
int Controller::StartDiscover() {
// Lock mutex
QMutexLocker ml(&m_Mutex);
// Zero
m_NumberBoundDevices = 0;
// Update to scanning
m_Status = Scanning;
// Discover slaves
int numberAttachedSlaves = m_Client->DiscoverSlaves();
m_Client->setTimeout(20000); // 20 Second Timeout
if ( numberAttachedSlaves > 0 ) {
int cnt = 0;
while ( cnt < 3 ) {
for (int slave = 1 ; slave <= numberAttachedSlaves ; slave++ ) {
// Get information about this slave
QDBusPendingReply<uchar> reply = m_Client->SlaveService(slave,m_Packet);
reply.waitForFinished(); // Wait for it to complete
if (!reply.isValid()) {
m_Client->SlaveService(slave,m_Packet);
reply.waitForFinished();
}
if ( reply.isError() ) {
QString errorMsg = reply.reply().errorMessage();
}
}
// Increment counter
cnt++;
}
}
// Update
m_NumberBoundDevices = numberAttachedSlaves;
// Return the number of devices discovered
return numberAttachedSlaves;
}
这会保护函数中的所有共享变量吗?或者,我应该在每个m_ *变量周围明确使用QMutex.lock()和unlock()吗?
谢谢
答案 0 :(得分:2)
考虑一下: -
char buffer[]
void ReadFromBuffer()
{
QMutexLocker ml(&m_Mutex);
// read contents of buffer
}
void WriteToBuffer()
{
QMutexLocker ml(&m_Mutex);
// write to buffer
}
如果ReadFromBuffer在与WriteToBuffer的单独线程中运行,则需要确保不会同时发生读取和写入。
如果一个线程在WriteToBuffer中执行并锁定互斥锁,当另一个线程进入ReadFromBuffer并执行其互斥锁时,它会检查该互斥锁是否被锁定。如果它被锁定,则线程暂停并等待它被解锁。当QMutexLocker对象超出锁定互斥锁的第一个函数的范围时,会发生这种情况。
这就是为什么围绕访问共享对象的所有代码都需要QMutexLocker。