C++实现数字转字母与倍数序列输出的代码问题排查

核心错误清单 #

• 函数参数类型错误：secretCode的参数声明为char类型，无法正确存储输入的大数字（比如样例中的23），且函数没有覆盖所有输入情况的返回值分支，存在未定义行为。
• 函数调用时机错误：printSequence用于输出10个数字拼接的完整字母串，原代码在每输入1个数字时就调用一次，此时数组未完全赋值，未赋值的元素是随机值，会输出重复内容+乱码。
• 输出格式缺失：倍数序列没有输出数字:的前缀，且每个序列输出完成后没有换行，所有输出内容会粘连在一起不符合要求。
• 冗余无效代码：for (int y = number[x]; y <= number[x]; ++y) 循环完全无意义，仅执行1次可以直接删除；if (z < 1) 判断永远不会触发（z从1开始循环），属于无效代码。

修复后的完整代码 #

#include <iostream>
using namespace std;

char secretCode(int number)
{
    // 按需求1对应a、2对应b的规则直接转换
    return static_cast<char>('a' - 1 + number);
}

void printSequence(int number[10])
{
    for (int i = 0; i < 10; i++)
    {
        cout << secretCode(number[i]);
    }
    // 字母串输出完换行分隔
    cout << endl << endl;
}

int main()
{
    int number[10];
    // 先读入全部10个数字
    for (int x = 0; x < 10; ++x)
    {
        cin >> number[x];
    }
    // 输出拼接后的字母串
    printSequence(number);
    // 遍历每个数字输出对应的倍数序列
    for (int x = 0; x < 10; ++x)
    {
        int cur = number[x];
        cout << cur << ":";
        for (int z = 1; z <= cur; ++z)
        {
            if (z > 1)
            {
                cout << "-";
            }
            cout << cur * z;
        }
        // 每个序列输出完换行分隔
        cout << endl << endl;
    }
    return 0;
}

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