关于 c :创建阻塞队列 | 珊瑚贝

Creating a Blocking Queue

有时这种BlockingQueue 的实现和执行是有效的。有时它会出现段错误。知道为什么吗？

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75

#include <thread>
using std::thread;
#include <mutex>
using std::mutex;
#include <iostream>
using std::cout;
using std::endl;
#include <queue>
using std::queue;
#include <string>
using std::string;
using std::to_string;
#include <functional>
using std::ref;

template <typename T>
class BlockingQueue {
private:
mutex mutex_;
queue< T > queue_;
public:
T pop() {
this–>mutex_.lock();
T value = this–>queue_.front();
this–>queue_.pop();
this–>mutex_.unlock();
return value;
}

void push(T value) {
this–>mutex_.lock();
this–>queue_.push(value);
this–>mutex_.unlock();
}

bool empty() {
this–>mutex_.lock();
bool check = this–>queue_.empty();
this–>mutex_.unlock();
return check;
}
};

void fillWorkQueue(BlockingQueue<string>& workQueue) {
int size = 40000;
for(int i = 0; i < size; i++)
workQueue.push(to_string(i));
}

void doWork(BlockingQueue<string>& workQueue) {
while(!workQueue.empty()) {
workQueue.pop();
}
}

void multiThreaded() {
BlockingQueue<string> workQueue;
fillWorkQueue(workQueue);
thread t1(doWork, ref(workQueue));
thread t2(doWork, ref(workQueue));
t1.join();
t2.join();
cout <<“done\
“;
}

int main() {
cout << endl;

// Multi Threaded
cout <<“multiThreaded\
“;
multiThreaded();
cout << endl;
}

相关讨论

如果它出现段错误，我假设你可以得到它发生的代码行？知道可能会有所帮助…
如果你检查 itemQueue 是否为空，然后让其他线程做一些工作，然后 pop() 一个项目会发生什么？
这个问题有足够的代码，任何人都可以尝试一下，自己看看问题出在哪里。也没有太多多余的代码，因此它与教科书 SSCCE 相距不远，而且肯定可以回答。
@Flexo 我很难相信”这个程序的错误在哪里？”可能永远是一个好问题。
如果您愿意，很高兴在元数据或聊天中进一步解释我对此的想法，但请避免在评论中聊天。

请看这里：

我从空 std 容器的 front() 中得到什么？

如果你在一个空容器上调用 .front() 会发生不好的事情，最好先检查 .empty()。

试试：

1
2
3
4
5
6
7
8
9
10
11
12

T pop() {
this–>mutex_.lock();
T value;
if( !this–>queue_.empty() )
{
value = this–>queue_.front(); // undefined behavior if queue_ is empty
// may segfault, may throw, etc.
this–>queue_.pop();
}
this–>mutex_.unlock();
return value;
}

注意：由于原子操作对这种队列很重要，我建议更改 API：

1	bool pop(T &t); // returns false if there was nothing to read.

更好的是，如果您确实在重要的地方使用它，您可能希望在删除之前标记正在使用的项目以防失败。

1
2
3

bool peekAndMark(T &t); // allows one”marked” item per thread
void deleteMarked(); // if an item is marked correctly, pops it.
void unmark(); // abandons the mark. (rollback)

Creating a Blocking Queue

猜你喜欢