Question

我正在尝试使用线程添加一个简单的for循环，但仍然有些东西没有用完。我检查了很多原因，但我找不到任何解决办法。

我有一个简单的类，有两个方法A()和B()。在其他课程中，我正在调用方法A()。这是它的外观：

void MyClass::A()
{
    std::vector<std::thread> threads;
    for(int i=0;i<2;i++)
    {
        threads.push_back(std::thread(&MyClass::B, this));
    }
    for(auto &t : threads)
    {
        if(t.joinable())
            t.join();
    }
}

void MyClass::B()
{
}

然而我仍然收到一些错误：

#0 ??   ?? () (??:??)
#1 00446D62 pthread_create_wrapper () (??:??)
#2 75327FB0 msvcrt!_cexit() (C:\Windows\SysWOW64\msvcrt.dll:??)
#3 040C8710 ?? () (??:??)
#4 753280F5 msvcrt!_beginthreadex() (C:\Windows\SysWOW64\msvcrt.dll:??)
#5 75B17C04 KERNEL32!BaseThreadInitThunk() (C:\Windows\SysWOW64\kernel32.dll:??)
#6 77ABAB8F ?? () (??:??)
#7 77ABAB5A ?? () (??:??)
#8 ??   ?? () (??:??)

有人知道出了什么问题吗？

再添一件事。这样：

void MyClass::A()
{
    std::thread t(&MyClass::B, this);
    if(t.joinable())
        t.join();
}

void MyClass::B()
{
}

没有任何问题。

Answer 1

我建议使用new和指针代替引用。

虽然引用很好，但注释的数量说明了在这种情况下引起的混乱。如何证明第三方库实现（例如std）将按照您期望的方式工作？和跨平台？

如果您使用：

std::vector<std::thread *> threads;

和

threads.push_back(new std::thread(&MyClass::B(), this));

当你完成线程时，当然会删除'删除'，然后就会消除混淆。

此外，代码正在将“A”类的SAME实例加载到所有线程的数组上。如果在函数“B”中使用A中的成员变量，而没有任何类型的多线程保护（例如信号量，关键部分等），则代码将永远无法正常工作。使用'new'进行线程实例创建可以避免复制构造函数或赋值运算符导致的向量类问题。

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

using namespace std;

class MyClass {
  public:
    void A() {
      cout << "starting\n";
      cout << "creating threads\n";
      vector<thread*> threads;
      for (int i = 0;i<4;i++) {
        thread * t = new thread(&MyClass::B, this);
        cout << "creating thread "<<t<<" id:"<<t->get_id()<<"\n";
        threads.push_back(t);
      }
      cout << "waiting for threads to complete\n";
      for (thread * t : threads) {
        if (t->joinable()) {
          t->join();
        }
        cout << "destroying "<<t<<"\n";
        delete t;
      }
      cout << "done\n";
   }

   void B() {
     // now it would be very bad if this function used 
     // member variables that are part of this instance of A
     cout << "hello from " << this << "\n";
   }
}; 

int main() {
  MyClass cls;
  cls.A();
  return 0;
}

for循环中多线程的错误

1 个答案: