Question

我正在实施一个并发包装器，由Herb Sutter在他的演讲＆＃34; C ++和Beyond 2012＆＃34;中提出。

template <typename T>
class ConcurrentWrapper {

  private:

    std::deque<std::unique_ptr<std::function<void()>>> _tasks;
    std::mutex _mutex;
    std::condition_variable _cond;

    T _object;
    std::thread _worker;
    std::atomic<bool> _done {false};

  public:

    template <typename... ArgsT>
    ConcurrentWrapper(ArgsT&&... args) :
      _object {std::forward<ArgsT>(args)...},
      _worker {
        [&]() {
          typename decltype(_tasks)::value_type task;
          while(!_done) {
            { 
              std::unique_lock<std::mutex> lock(_mutex);
              while(_tasks.empty()) {
                _cond.wait(lock);
              }
              task = std::move(_tasks.front());
              _tasks.pop_front();
            } 
            (*task)();
          }
        }
      } {
    }

    ~ConcurrentWrapper() {

      {
        std::unique_lock<std::mutex> lock(_mutex);
        _tasks.push_back(std::make_unique<std::function<void()>>(
          [&](){_done = true;}
        ));
      }
      _cond.notify_one();
      _worker.join();           
    }

    template <typename F, typename R = std::result_of_t<F(T&)>>
    std::future<R> operator()(F&& f) {

      std::packaged_task<R(T&)> task(std::forward<F>(f)); 
      auto fu = task.get_future();

      {
        std::unique_lock<std::mutex> lock(_mutex);
        _tasks.push_back(std::make_unique<std::function<void()>>(
          [this, task=MoveOnCopy<decltype(task)>(std::move(task))]() { 
            task.object(this->_object); 
          }
        ));
      }
      _cond.notify_one();

      return fu;
    }
};

基本上，我们的想法是使用操作()包装对象并以FIFO顺序提供线程安全访问。但是，在某些运行中（并非总是如此），以下程序被挂起：

ConcurrentWrapper<std::vector<int>> results;

results(
  [&](std::vector<T>& data) {
    std::cout << "sorting...\n";
    std::sort(data.begin(), data.end());
    std::cout << "done ...\n";
    EXPECT_EQ(data, golden);
  }   
).get();

但是，该程序无需显式调用get()方法即可正常工作。

results(
  [&](std::vector<T>& data) {
    std::cout << "sorting...\n";
    std::sort(data.begin(), data.end());
    std::cout << "done ...\n";
    EXPECT_EQ(data, golden);
  }    
);    // Function correctly without calling get

有什么问题？我做错了吗？我注意到有一个帖子here说＆＃34;在调用f.get（）之前需要调用packaged_task，否则程序会冻结，因为将来永远不会准备就绪。＆＃34;这是真的？如果是，我怎样才能解决这个问题？

我正在使用-std=c++1z -pthread和G++ 6.1

编译代码

C ++ packaged_task在Concurrent Wrapper中挂起

0 个答案: