std::cout无法打印到终端问题排查：代码异常原因分析

问题描述 #

使用std::cout向终端打印内容时无任何输出，仅保留打印语句时终端能正常输出，尝试过std::flush也无法解决。相关代码如下：

#include <iostream>
#include <vector>
#include <string>
using namespace std;

std::string multiplystr(std::string string, int mult) {
    for (int i = 1; i < mult; i++)     {
        string = string + string;
    }
    return string;
}

class entry {
public:

    entry(int width) {
        int frameWidth = width;
    }

    int frameWidth;
    std::string framechar = "-";

    std::string frame = multiplystr(framechar, (frameWidth + 4));
    std::string justout = frame + '\n';
};

int main() {
    entry x1(15);
    std::string out1 = x1.frame;
    std::cout.flush();
    cout << "out1" << std::endl;
}

问题原因 #

• 变量遮蔽导致成员未初始化：entry类的构造函数中，声明了局部变量int frameWidth = width;，这会遮蔽类的成员变量frameWidth，导致类成员frameWidth始终是未初始化的垃圾值。
• 未初始化值引发程序异常：类成员frame在初始化阶段调用multiplystr时，传入的frameWidth + 4是基于未初始化的垃圾值计算的，会得到一个不可控的超大数值。multiplystr函数中不断加倍字符串的操作会瞬间占用巨量内存，导致程序崩溃或进入无响应状态，根本无法执行到后续的cout打印语句。

修复方案 #

1. 修复构造函数的变量遮蔽问题 #

去掉构造函数中的局部变量声明，直接给类成员赋值，或使用初始化列表（更规范）：

// 方法1：直接赋值
entry(int width) {
    frameWidth = width;
}

// 方法2：初始化列表
entry(int width) : frameWidth(width) {}

2. 修正multiplystr函数逻辑 #

原函数逻辑是将字符串反复加倍，最终得到的是原字符串的2^(mult-1)倍，而非预期的mult倍。如果需要生成重复mult次的字符串，修改为：

std::string multiplystr(const std::string& str, int mult) {
    std::string result;
    result.reserve(str.size() * mult); // 预分配内存提升效率
    for (int i = 0; i < mult; ++i) {
        result += str;
    }
    return result;
}

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