向线程传递引用是否为不良实践？C++代码场景分析

先看你给出的代码：

void edit_document(std::string const& filename) {
    open_document_and_display_gui(filename);
    while(!done_editing()) {
        user_command cmd=get_user_input();
        if(cmd.type==open_new_document) {
            std::string const new_name=get_filename_from_user();
            std::thread t(edit_document,new_name);
            t.detach();
        } else {
            process_user_input(cmd);
        }
    }
}

咱们来逐个解答你的问题：

1. 此场景下用const std::string&传参是否错误？ #

完全没问题！

这里的核心是std::thread的参数处理逻辑：当你把new_name传给线程构造函数时，std::thread会自动把这个局部变量拷贝一份到线程自身的内存空间中，而非直接传递原变量的引用。之后线程调用edit_document时，会把这份拷贝的对象的引用绑定到函数的const std::string& filename参数上。

因为const引用允许绑定临时/拷贝对象，而且这份拷贝的生命周期和线程完全绑定——只要线程还在运行，它就会一直存在，所以这个引用是安全有效的。只有当你需要让线程直接操作原变量（而非拷贝）时，才需要显式使用std::cref(new_name)来传递真正的引用，显然这段代码不需要这么做。

2. 新线程阻塞，new_name销毁后是否会读取到垃圾值？ #

绝对不会！

线程里的filename引用的是std::thread内部拷贝的那份字符串，和原new_name变量完全是两个独立的对象。原new_name是if块内的局部变量，当if块执行完毕就会被销毁，但这丝毫不影响线程里的拷贝——这份拷贝会独立存在，直到线程运行结束才会被释放。哪怕线程因为阻塞长时间运行，它访问的都是自己专属的那份有效数据，不会出现读取垃圾值的情况。

最后提个小题外话：这段代码使用了t.detach()，这意味着主线程彻底失去了对新线程的控制权。如果程序退出时这些detach的线程还在运行，可能会引发未定义行为（比如资源泄漏、程序崩溃），这是个需要注意的潜在问题，但和你问的传参问题无关。

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