技术问询：C++中for内嵌while循环仅执行一次的问题排查

问题根源 #

你代码里的核心问题是全局变量rep在第一次进入while循环后被递减到0，后续外层for循环的其他迭代中，rep已经是0，while循环直接跳过，所以只有第一行被重复输出了指定次数，剩下的行都只执行了一次换行操作，没有进入内层的输出逻辑。

修复方案 #

解决思路是每次处理金字塔的一行时，使用临时变量保存原始的重复次数，避免修改全局的rep值，这样每一行都能重复输出指定次数。

修正后的代码 #

#include<iostream>
using namespace std;

int main() {
    int h, rep = 0; // 把变量移到main内，避免全局变量带来的意外问题
    cout << "Please enter pyramid's height: ";
    cin >> h;
    cout << "Please enter Repetition Number: ";
    cin >> rep;
    
    for(int i = 0; i <= h; i++) {
        // 每次循环创建临时变量，保存当前需要重复的次数
        int current_rep = rep;
        while(current_rep > 0) {
            for(int j = 0; j <= h; j++) {
                if(i >= j)
                    cout << "x ";
                else
                    cout << "  ";
            }
            current_rep--; // 递减临时变量，不影响原始rep
        }
        cout << endl;
    }
}

额外优化建议 #

• 外层for循环的条件i <= h会生成h+1行内容（比如h=3时，输出0、1、2、3共4行），如果你的需求是生成h行金字塔，可以把条件改成i < h。
• 尽量避免使用全局变量，像原来的h和rep放在main函数内部更安全，减少变量污染和意外修改的风险。

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