C++循环意外提前退出求助——《Accelerated C++》练习代码排查

首先，你的程序出现不稳定行为的核心原因是整数溢出导致的未定义行为（Undefined Behavior），再加上对齐逻辑的不必要复杂，最终引发了无规律的异常表现。让我一步步拆解并解决：

1. 整数溢出的核心隐患 #

你的square函数返回int类型，但int的取值范围有限（通常32位int最大值为2147483647）。当maxInt足够大时，i*i的结果会超出这个范围，触发未定义行为——程序可能正常运行、突然崩溃、输出错误值，或者毫无规律地提前终止，这正是你遇到的“不稳定”现象。

即使你现在设置maxInt=1000没问题，但这种写法埋下了隐患，后续只要调整maxInt到足够大（比如46341），溢出就会立刻出现。

2. 对齐逻辑的复杂与潜在错误 #

你计算width的方式过于绕弯，虽然试图让总输出长度固定，但这种动态计算容易出错（比如如果溢出导致平方值变成负数，to_string会多一个负号，直接打乱宽度计算）。其实用固定列宽对齐会更简单可靠。

修复后的代码 #

我重写了你的代码，解决了上述问题，并优化了可读性：

#include <iostream>
#include <iomanip>
#include <string>

using std::cout;
using std::endl;
using std::setw;
using std::to_string;

// 使用long long避免整数溢出
long long square(const int n) {
    return static_cast<long long>(n) * n;
}

int main() {
    const int maxInt = 1000;
    // 计算两列需要的固定宽度
    const int numWidth = to_string(maxInt).length();
    const int squareWidth = to_string(square(maxInt)).length();

    // 可选：打印表头提升可读性
    cout << setw(numWidth) << "整数" << " " << setw(squareWidth) << "平方" << endl;
    cout << string(numWidth + squareWidth + 1, '-') << endl;

    for (int i = 1; i <= maxInt; ++i) {
        const long long j = square(i);
        // 固定列宽对齐，右对齐是默认行为
        cout << setw(numWidth) << i << " " << setw(squareWidth) << j << endl;
    }
    return 0;
}

关键改进： #

• 把square的返回类型改为long long，并在计算时先将n转为long long，彻底避免乘法溢出。
• 使用固定列宽numWidth和squareWidth对齐两列，逻辑更清晰，避免动态计算width的潜在问题。
• 添加了表头和分隔线，让输出结果更易读。

给C++新手的通用建议 #

• 警惕整数溢出：乘法、加法运算很容易超出类型范围，优先选择足够大的类型（如long long）存储结果，或者在编译期用constexpr验证范围。
• 保持代码简洁：复杂的逻辑不仅难写，更难维护。像对齐列这种需求，固定宽度比动态计算更可靠。
• 避免未定义行为：C++标准对未定义行为没有任何保证，一旦触发，程序的表现完全不可预测（崩溃、乱码、无规律终止都是可能的）。
• 善用标准库工具：<iomanip>里的setw、left/right等操纵符可以轻松实现格式化输出，不用自己造轮子。
• 养成良好的命名习惯：变量名numWidth比maxIntSize更直观，函数名square已经很清晰，继续保持。

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