C++线程Lambda捕获循环临时变量是否安全？

std::vector<std::string> adapterList = getAdapterList();
std::vector<std::thread> captureThreads;

for (auto adapterName : adapterList) {
    captureThreads.push_back(std::thread([&] {
        // do something with adapterName. Safe?
    }));
}

for (auto& thread : captureThreads) {
    if (thread.joinable())
        thread.join();
}

这段代码不安全，原因如下：

• 循环里的adapterName是值拷贝生成的变量，但它的生命周期只覆盖当前循环迭代（C18及以后）或整个循环周期（C17及更早）。当线程延迟执行时，adapterName已经被销毁，Lambda里的引用会指向一个已失效的对象，触发未定义行为（比如程序崩溃、数据乱码）。
• 哪怕线程刚好在迭代过程中执行，C++17及更早标准里的adapterName是循环共用的实例，线程访问时可能已经被下一次迭代覆盖，拿到的不是预期的值。

正确的处理方式 #

• 值捕获变量：把Lambda捕获列表改成[adapterName]或[=]，让线程持有一份独立的adapterName拷贝，生命周期和线程绑定，访问完全安全：

  for (auto adapterName : adapterList) {
      captureThreads.push_back(std::thread([adapterName] {
          // 使用adapterName，安全
      }));
  }
  

• 移动捕获优化：如果不需要保留原变量的值，用移动语义减少拷贝开销：

  for (auto adapterName : adapterList) {
      captureThreads.push_back(std::thread([name = std::move(adapterName)] {
          // 使用name，安全且高效
      }));
  }
  

内容的提问来源于stack exchange，提问作者user5266221