编程新手求助：为何我的C++程序返回原始变量值？

嘿，别担心，作为编程新手遇到这种问题太正常了！我帮你拆解一下代码里的问题：

核心问题分析 #

• 重复声明函数原型：你已经在文件最开头声明了int whichNumber(int x);，但在main函数里又重复写了一遍，这完全没必要，反而容易造成混淆，直接删掉main里的这句就行。
• 函数缺少返回语句：这是导致程序返回原始变量值的关键！你的whichNumber函数声明的返回类型是int，但函数体里根本没有return语句。在C++中，这种情况属于未定义行为——程序会随机返回一个内存里的垃圾值，刚好你看到的是原始变量x的值，这只是巧合而已。

另外还有两个小细节要注意：

1. 你用到了cout、cin和string，必须在代码开头包含对应的头文件：#include <iostream>和#include <string>。
2. 如果不想每次都写std::cout这种前缀，可以加上using namespace std;（新手阶段这样用会方便很多）。

修复后的代码示例 #

根据你的需求，我给你两种可行的修复方案：

方案一：让函数返回对应的字符串 #

这个方案更符合函数的职责（输入数字，返回对应的英文表述）：

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

string whichNumber(int x); // 修改返回类型为string

int main() {
    int x;
    cout << "Enter a number in the range 1-5: ";
    cin >> x;
    cout << "\nYou entered the number " << whichNumber(x) << "\n";
    return 0;
}

string whichNumber(int x){
    string numbers[] = {"one.", "two.", "three.", "four.", "five."};
    // 数组索引从0开始，所以1对应索引0，要减1
    if (x >= 1 && x <= 5) {
        return numbers[x - 1];
    } else {
        return "invalid number."; // 处理输入不在1-5的情况
    }
}

方案二：让函数直接输出内容（返回类型为void） #

如果更希望函数直接完成输出工作，可以把返回类型改成void：

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

void whichNumber(int x); // 返回类型改为void

int main() {
    int x;
    cout << "Enter a number in the range 1-5: ";
    cin >> x;
    cout << "\nYou entered the number ";
    whichNumber(x);
    cout << "\n";
    return 0;
}

void whichNumber(int x){
    string numbers[] = {"one.", "two.", "three.", "four.", "five."};
    switch(x) { // 用switch比多个if更简洁
        case 1: cout << numbers[0]; break;
        case 2: cout << numbers[1]; break;
        case 3: cout << numbers[2]; break;
        case 4: cout << numbers[3]; break;
        case 5: cout << numbers[4]; break;
        default: cout << "invalid number.";
    }
}

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