基于函数内switch的repeat变量导致死循环问题求助

核心问题拆解 #

1. 变量作用域导致repeat值无法传递 #

你在DayCheck函数里定义的bool repeat是局部变量，和main函数里的repeat完全是两个独立的变量——函数内部修改这个局部repeat的值，根本不会影响到main里的同名变量，这就是你修改了repeat但外部毫无反应的原因。

2. do-while循环条件的致命错误 #

do { ... } while (repeat = false); 这里用的是赋值运算符=而非比较运算符==，而且main里的repeat一开始未初始化，值是不确定的。更关键的是，这个条件每次都会把repeat强制设为false，但结合前面的变量作用域问题，不管怎样，repeat的取值都没被正确更新，直接导致循环无法终止。

3. 日期校验逻辑不完整 #

你需要校验的是MO、TU、WE、TH、FR、SA、SU格式的日期代码，但当前代码只判断了首字符，既没处理完整的两位输入，逻辑也完全颠倒：只有输入不符合首字符时才设置repeat=false，但正确输入时没有设置终止循环的标记，完全达不到校验目的。

修正后的代码 #

#include <iostream>
#include <string> // 处理两位日期输入，用string更合适

using namespace std;

// 用返回值直接传递校验结果，避免作用域问题
bool DayCheck(const string& day) {
    // 校验所有合法的日期代码
    if (day == "MO" || day == "TU" || day == "WE" || day == "TH" || 
        day == "FR" || day == "SA" || day == "SU") {
        return true; // 输入合法，无需重复输入
    } else {
        cout << "\nThat was an invalid input.";
        return false; // 输入非法，需要重新输入
    }
}

int main() {
    string day;
    bool repeat;
    do {
        cout << "\nEnter day: ";
        cin >> day;
        repeat = !DayCheck(day); // 合法则repeat为false终止循环，非法则为true继续循环
    } while (repeat);
    cout << "Valid day entered: " << day;
    return 0;
}

关键修改说明 #

• 解决变量传递问题：将DayCheck改为返回bool值，直接把校验结果返回给main，彻底避免局部变量无法传递的问题。
• 修正循环条件：用repeat = !DayCheck(day)关联校验结果，逻辑清晰，确保合法输入时终止循环，非法输入时继续。
• 完善校验逻辑：改用string接收输入，直接校验完整的两位日期代码，完全符合需求中的格式要求。
• 消除未初始化风险：main里的repeat会被DayCheck的返回值赋值，避免了未初始化导致的不确定值问题。

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