我正在尝试使用Qt的并发线程构造创建并发队列。
#ifndef CONCURRENTQUEUE_H
#define CONCURRENTQUEUE_H
#include <QMutex>
#include <QWaitCondition>
#include <queue>
template<typename Data>
class ConcurrentQueue
{
private:
std::queue<Data> the_queue;
QMutex the_mutex;
QWaitCondition the_condition_variable;
bool closed;
public:
void setClosed(bool state)
{
QMutexLocker locker(&the_mutex);
closed = state;
}
bool getClosed()
{
QMutexLocker locker(&the_mutex);
return closed;
}
void push(Data const& data)
{
QMutexLocker locker(&the_mutex);
the_queue.push(data);
the_condition_variable.wakeOne();
}
bool empty()
{
QMutexLocker locker(&the_mutex);
return the_queue.empty();
}
bool try_pop(Data& popped_value)
{
QMutexLocker locker(&the_mutex);
if(the_queue.empty())
{
return false;
}
popped_value = the_queue.front();
the_queue.pop();
return true;
}
void wait_and_pop(Data& popped_value)
{
QMutexLocker locker(&the_mutex);
while(the_queue.empty())
{
the_condition_variable.wait(&the_mutex);
}
popped_value = the_queue.front();
the_queue.pop();
the_condition_variable.wakeOne();
}
//created to allow for a limited queue size
void wait_and_push(Data const& data, const int max_size)
{
QMutexLocker locker(&the_mutex);
while(the_queue.size() >= max_size)
{
the_condition_variable.wait(&the_mutex);
}
the_queue.push(data);
the_condition_variable.wakeOne();
}
};
#endif // CONCURRENTQUEUE_H
我的生产者线程使用wait_and_push方法将数据推入队列,我一直试图让我的消费者使用try_pop从队列中读取
while(!tiles->empty() || !tiles->getClosed())
{
if(!tiles->try_pop(tile))
continue;
//do stuff with the tile
}
但是,有时会出现这种僵局。生产者将关闭的布尔值设置为消费者线程的标志,表明它已完成加载队列。我的消费者只有这样才能知道队列是否正在加载,仍在进行中,还是尚未启动。
生产者有“wait_and_push”而不是使用正常推送的原因是因为我希望能够制作该线程块,直到处理了一些项目以避免占用这么多内存,并且做了不必要的磁盘I / O操作。
有人能指出我出了什么问题吗?
答案 0 :(得分:3)
您忘了添加
the_condition_variable.wakeOne();
try_pop
中的。
如果有多个生产者/消费者访问您的队列,您应该为生产者和消费者分别设置QWaitCondition
,否则wakeOne
可能无法唤醒正确的线程。
编辑:
如果有多个生产者/消费者,那么您应该有notFullCondvar
和notEmptyCondvar
。
try_pop
方法会唤醒notFullCondvar
。wait_and_pop
方法等待notEmptyCondvar
,但会唤醒notFullCondvar
。push
方法会唤醒notEmptyCondvar
。wait_and_push
方法等待notFullCondvar
,但会唤醒notEmptyCondvar
。我希望这是有道理的。