c++ - 如何在 else-if 语句中正确使用 or 语句 (||)

Question

考虑这段代码:

string GameExit;
bool GameChoiceGo = true;

while (GameChoiceGo == true)
{
    system("cls"); 
    cout << "\n  Are you sure you want to exit? (Yes or No)     ";
    cin >>  GameExit;
    if (GameExit == "y" || "Y" || "yes" || "Yes" || "YES")
    {
        cout << "User typed Yes";
        Sleep(3000);
        system("cls");
        break;
    }
    if (GameExit == "n" || "N" || "no" || "No" || "NO")
    {
        cout << "User typed No";
        Sleep(3000);
        system("cls");
        GameChoiceGo = false;
    }
    else
    {
        cout << "\nI'm sorry but, " << GameExit << " is not a acceptable choice. Type: Yes or No.\n\n\n";
        Sleep(3000);
        system("cls");
    }
}
break;

这里，只有第一条语句被激活。即使用户键入 "No" 或任何其他内容，它也会输出 "user typed yes"。

else-if 语句 如果我只用一个语句(即 "y" 和 "替换 or 语句，则该语句起作用n").唯一的问题是，我想要用户可能在代码中键入的任何可能版本的 yes 和 no。

代码无法正常工作的任何想法？

Answer 1

很抱歉，您必须为要检查的每个条件编写 GameExit ==:

if (GameExit == "y" || GameExit == "Y" || GameExit == "yes" || GameExit == "Yes" || GameExit == "YES")

如果你写 if ("y")(这基本上就是你正在做的，只是有更多的语句)，const char[] 将衰减为const char*，该指针将与 0 进行比较。现在，该指针将永远为空，因为总会为字符串文字分配内存。

更好的解决方案是 (1) 创建一个包含所有选项的数组，以便检查条件成为一个简单的搜索或 (2) 例如将输入转换为全小写，然后进行比较。

// 1)
std::vector<std::string> options = { "y", "Y", "yes", "Yes", "YES" };

if (std::find(options.begin(), options.end(), GameExit) != options.end());
// or
if (std::any_of(options.begin(), options.end(), [&GameExit](const auto& value) { 
        return GameExit == value;
   }); 

// 2)
std::transform(GameExit.begin(), GameExit.end(), GameExit.begin(), ::tolower);

if (GameExit == "y" || GameExit == "yes");

如果您不知道函数的作用，可以查找函数:)。