C++中先cout输出其他字符串后vector内容无输出的原因是什么

核心错误原因 #

你的代码触发了C++的未定义行为，根源在于execv函数返回了局部对象的悬空引用：

• execv函数内部定义的vector<string> result是栈上的局部变量，它的生命周期仅存在于execv函数执行期间。
• 函数执行结束返回时，这个局部vector会被自动销毁，对应的内存空间会被标记为可复用。你返回的引用本质是指向已经失效的内存地址，属于悬空引用。

表现不稳定的原因 #

未定义行为的典型特征就是运行结果不可预测：

• 当你注释掉cout那行代码时，execv退出后栈内存还没有被其他操作覆盖，此时读取悬空引用指向的地址，还能凑巧读到之前vector存储的残留数据，看起来运行正常。
• 当你打开cout那行代码时，cout的执行逻辑会使用栈内存，刚好覆盖了之前vector占用的栈空间，原来的数据被破坏，自然无法输出正确的vector内容。

修复方案 #

直接修改execv函数的返回值类型，改为按值返回vector即可，C++的返回值优化（NRVO）会自动省略不必要的拷贝，不会带来性能损耗：

修复后的关键代码 #

#include <iostream>
#include <cstdio>
#include <memory>
#include <stdexcept>
#include <string>
#include <array>
#include <vector>

using namespace std;

// 修改函数声明，返回值改为vector<string>而非引用
vector<string> execv(const char* cmd);
void output(const vector<string>& vec);


int main() {
    // 直接接收返回值，也可以用const引用绑定，生命周期会自动延长
    const vector<string> result = execv("net user %USERNAME%");
    int _numlines = result.size();
    // 现在取消注释这行也不会有问题
    cout << _numlines << " lines had been found: " << endl;
    output(result);
    return 0;
}

void output(const vector<string>& vec)
{
    for(const string& line : vec)
        cout << line;
}

// 函数实现修改返回类型，直接返回局部result即可
vector<string> execv(const char* cmd)
{
    array<char, 128> buffer;
    vector<string> result;
    unique_ptr<FILE, decltype(&pclose)> pipe(_popen(cmd, "r"), _pclose);
    if(!pipe)
    {
        throw runtime_error("_popen() failed!");
    }
    while (fgets(buffer.data(), buffer.size(), pipe.get()) != nullptr)
    {
        result.push_back(buffer.data());
    }
    // 直接返回即可，不需要额外绑定引用
    return result;
}

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