这是启动和停止线程的安全方法吗？

Question

假设我有一个名为foo的类，它在线程中执行一些任务。 我想按照我认为合适的方式开始和停止这项任务。 我的代码如下：

class foo
{

public:
    foo() {};
    ~foo() {};

    void start_thread() {
        thread_active = true;
        the_thread = std::thread(&foo::execute, this);
    }
    void stop_thread() {
        thread_active = false;
        the_thread.join();
    }

private:
    void execute() {
        while (thread_active)
            std::cout << "executing...\n";
    }

    std::thread the_thread;
    bool thread_active;
};

我这样称呼它：

int main()
{
    foo bar;
    bar.start_thread();
    Sleep(1000);
    bar.stop_thread();
}

这是一种安全的方法吗？

Answer 1

不，这不是一种安全的方式。

原因是thread_active成员在读取和写入时没有从多个线程同步访问，您应该声明它std::atomic_bool或使用某种其他形式的同步。