C++简易考试评分程序无法触发FalseInput函数问题求解与改进咨询

你代码的核心问题是判断逻辑顺序错误：当cin >> Grades输入非法值时，C++11及之后标准会将int类型的Grades默认置为0，同时将cin流置为失败状态。但你的main函数中先判断了Grades >= MinGrade，再判断Grades < MinGrade，最后才校验cin.fail()状态。0明显小于50，会直接命中Grades < MinGrade分支输出「you failed」，完全不会走到触发FalseInput的校验分支。

首先调整所有输入场景下的判断顺序：先校验cin流状态是否正常，再处理分数数值逻辑。同时修复代码中的拼写错误、补充必要的头文件，修复后完整代码如下：

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

string Name;
int Grades;
const int MinGrade = 50;

void FalseInput() {
    cout << "please enter the number again : ";
    cin >> Grades;
    // 先判断流状态，再判断数值
    if (cin.fail()) {
        cout << "it's not a valid number\n";
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        cout << endl;
        FalseInput();
    } else if (Grades >= MinGrade) {
        cout << Name << " : " << "you passed\n";
        cout << Grades;
    } else {
        cout << "you failed\n";
    }
}

int main() {
    cout << "Please Type Your Name : ";
    getline(cin, Name);
    cout << "Please Insert The Grade : ";
    cin >> Grades;
    // 先判断流状态，再判断数值
    if (cin.fail()) {
        cout << "it's not a valid number\n";
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        cout << endl;
        FalseInput();
    } else if (Grades >= MinGrade) {
        cout << Name << " : " << "you passed\n";
        cout << "The Grade Achieved : " << Grades << "%";
    } else {
        cout << "you failed\n";
    }

    return 0;
}

• 移除不必要的全局变量：可以将Name、Grades改为main函数内的局部变量，需要在FalseInput中使用的话通过传参实现，降低代码耦合。
• 用循环替代递归实现重试：递归校验输入如果连续失败次数过多会触发栈溢出，改为while循环实现重试逻辑更稳定。
• 增加分数合法性校验：可以补充判断分数是否在0~100的合理区间内，避免输入比如120、-10这类合法但不符合业务规则的数值。
• 避免使用using namespace std：大型项目中容易引发命名冲突，建议直接用std::前缀标注标准库成员。

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