Question

我有一个名为Task的类，它在内部拥有一个成员std :: thread。一般的想法是创建一个在处理请求时保持活跃的线程。

class Task
{
public:
    Task();
    ~Task();

    void start();
    // some funny stuff here
protected:
    Task(const Task& ref);
    void main_function();

    std::thread m_thread;
    // more funny stuff like queues, mutexes, etc
}

在函数start（）中我做：

void Task::start()
{
    m_thread = std::thread(std::bind(&Task::main_function, this));
}

问题是该行使用运行时错误R6010调用abort（）。我在某处读过这可能是因为m_thread的析构函数在没有先前连接的情况下被调用，但由于线程尚未启动，我无法加入它。

我正在使用Visual Studio 2012运行它

更新

所以我尝试了一个测试示例，无法复制错误。然后我将有问题的函数中的start函数替换为sugested：

void Task::start()
{
    assert(!m_thread.joinable());
    m_thread = std::thread(&Task::main_function,this);
}

但我仍然得到错误R6010。调用堆栈是：

msvcr110d.dll!_NMSG_WRITE(int rterrnum) Line 226    C
msvcr110d.dll!abort() Line 62   C
msvcr110d.dll!terminate() Line 97   C++
msvcp110d.dll!_Call_func(void * _Data) Line 63  C++

msvcr110d.dll!_callthreadstartex() Line 354 C
msvcr110d.dll!_threadstartex(void * ptd) Line 337   C

UPDATE2： 最后可以复制这个问题。代码如下。在主函数中调用foo（）。

class Task 
{
public:
    Task() : m_exitFlag(false) 
    { 
        std::cout << "constructor called" << std::endl;
    }

    ~Task()
    {
        m_lock.lock();
        m_exitFlag = true;
        m_condlock.notify_all();
        m_lock.unlock();

        if (m_thread.joinable()) m_thread.join();
        std::cout << "destructor called" << std::endl;
     }

     void start()
     {
         std::cout << "Task start" << std::endl;
         assert(!m_thread.joinable());
         m_thread = std::thread(&Task::main_function, this);
     }
protected:
     void main_function()
     {
         std::cout << "thread started" << std::endl;
         while(1)
         {
             m_lock.lock();
             while(m_queue.empty() && !m_exitFlag)
                 m_condlock.wait(std::unique_lock<std::mutex>(m_lock));

             if (m_exitFlag)
             {
                 m_lock.unlock();
                 std::cout << "thread exiting" << std::endl;
                 return;
             }

             std::function<void()> f;
             if (!m_queue.empty()) f = m_queue.front();

             m_lock.unlock;
             if (f != nullptr) f();
         }
     }
     Task(const Task&ref) { }

     Task& operator=(const Task& ref) {
         return *this;
     }
};
void foo() {
    Task tk;
    tk.start();
}

我猜这里有一个竞争条件，因为它有时会崩溃而有些则没有。一个线程在~Task（）中的关键区域内，另一个线程在Update1中作为堆栈。

Answer 1

永远不要直接锁定互斥锁。 C ++提供lock_guard，unique_lock等。有一个原因。

特别是，这一部分存在问题：

m_lock.lock();
while(m_queue.empty() && !m_exitFlag)
    m_condlock.wait(std::unique_lock<std::mutex>(m_lock));

新构建的unique_lock将尝试锁定已锁定的互斥锁m_lock。如果互斥锁为std::mutex，则为will cause undefined behavior，如果互斥锁为std::recursive_mutex，则为{{}}}。另请注意，在绑定未命名的unique_lock to a non-const reference when calling wait时，此行依赖于非标准编译器扩展。

因此，首先要做的是将锁定为命名变量。然后将std::adopt_lock传递给锁的构造函数，或者更好但永远不会直接锁定互斥锁，但始终将其包装在适当的锁管理类中。

例如，

m_lock.lock();
m_exitFlag = true;
m_condlock.notify_all();
m_lock.unlock();

变为

{
    std::lock_guard<std::mutex> lk(m_lock);
    m_exitFlag = true;
    m_condlock.notify_all();
} // mutex is unlocked automatically as the lock_guard goes out of scope

如果在关键部分内引发异常，这会带来额外的好处，即您不会泄漏锁。

Answer 2

似乎从队列的front（）获取一个函数，然后在解锁之后运行该函数，而不首先使用pop（）将其从队列中删除。这是你的意图吗？在这种情况下，下一个随机线程也可以获取相同的功能并同时运行它。这些功能是线程安全的吗？另外，你检查了queue.empty（）但是在执行后队列被清空的位置在哪里？

std :: thread导致应用程序中止，错误R6010

2 个答案: