Question

我有一个C ++ 11程序，它配置了许多可运行的对象，将它们放在std::vector中，然后在不同的线程中启动它们。不幸的是，当我迭代向量中的对象时，我只是为最后一个对象启动了线程。我已经在下面的测试代码中将问题提炼到了核心（使用clang++ -std=c++11 cpp_threadlaunch.cpp在OSX 10.9.5上使用clang 6.0编译）。

#include <iostream>
#include <thread>
#include <vector>
#include <unistd.h>

std::mutex  outputlock;

class agent {

public:
    agent(std::string name) : m_name(name) {};
    ~agent(void) {};

    void run(void) {
        while (1) {
            outputlock.lock();
            std::cout << "Agent: " << m_name << std::endl;
            outputlock.unlock();
            sleep(1);
        }
    }
    std::string     getName(void) { return m_name; }

private:

    std::string     m_name;
};

int main()
{
    std::vector<std::thread>        threads;
    std::vector<agent*>             agents;
    // std::string                  goal = "succeed";
    std::string                     goal = "fail";

    agents.push_back(new agent("A"));
    agents.push_back(new agent("B"));

    if (goal == "succeed") {

        threads.push_back(std::thread([&]() { agents.at(0)->run(); }));
        threads.push_back(std::thread([&]() { agents.at(1)->run(); }));

    }
    else {

        for (auto it = agents.begin(); it != agents.end(); it++) {

            agent* a = *it;

            std::cout << "Launching thread for " << a->getName() << std::endl;

            threads.push_back(std::thread([&]() { a->run(); }));
        }
    }

    for (auto it = threads.begin(); it != threads.end(); it++) {

        it->join();
    }

    exit(0);
}

当我使用goal = "succeed"时，我得到了希望的输出

Agent: A
Agent: B
Agent: A
Agent: B

当我使用goal = "fail"运行时，我只从一个对象获得两个输出副本，而不是每个对象的输出：

Launching thread for A
Launching thread for B
Agent: B
Agent: B
Agent: B
Agent: B

我怀疑我在这里遗漏了一些不足之处 - 如果有人能够解释发生了什么以及解决方案是什么，我会非常感激。谢谢 -

Answer 1

您在循环中传递给std::thread的lambda函数通过引用捕获a。由于a超出了范围，因此您有未定义的行为。按值捕获（使用[=]代替[&]）。

Answer 2

问题是您通过引用捕获'a'的值 [&]() { a->run() } 您看到最后一个值，因为循环已经完成，到其他线程运行时。如果按值捕获，使用[=]可以获得所需的效果。

从C ++ std :: vector中的线程中启动可运行对象

2 个答案: