c++ - 检查字符串中的小写字符或空格。 bool 值不起作用

Question

这应该检查输入的字符串是否是有效的许可证号。有效数字不能包含任何小写字母或空格。如果我输入“cat”，它应该看到字符“c”较低，因此它应该将 bool isValid 设置为 false，从循环中跳出，然后打印“cat is not a valid license number”。但是它没有，它只是使 bool isValid 对我一开始设置的任何值都是正确的。因此，如果我将其初始化为 false 并输入“CAT”，isValid 仍然是 false。

int main()
{
    // Input
    cout << "Enter a valid license number: ";
    string license_number;
    getline(cin, license_number);
    cout << endl;

    // Initialize boolean
    bool isValid = true;

    // Loop to check for space or lowercase
    for (int i = 0; i < license_number.size(); i++)
    {
        if (isspace(license_number[i]) || islower(license_number[i]))
        {
            bool isValid = false;
            break;
        }

        else
            bool isValid = true;
    }

    // Output
    if (isValid == true)
        cout << license_number << " is a valid license number." << endl;

    else
        cout << license_number << " is not a valid license number." << endl;

    return 0;
}

Answer 1

问题出在这里:

bool isValid = false;
    break;

您没有更改您的 isValid 变量。相反，您正在创建一个新的 isValid 来隐藏原始 isValid，并且该新变量在其之后超出范围时立即被丢弃。因此，您原来的 isValid 不受影响。删除此行中的 bool 即可。

---

除此之外你还可以删除这部分

else
    bool isValid = true;

因为 isValid 在到达这部分代码时无论如何都是 true 。此外，您可以简单地编写 if (isValid) 而不是 if (isValid == true)。您甚至可以像这样简化代码:

// Loop to check for space or lowercase
for (int i = 0; i < license_number.size(); i++)
{
    if (isspace(license_number[i]) || islower(license_number[i]) )
    {
        cout << license_number << " is not a valid license number." << endl;
        return 0;
    }
}

cout << license_number << " is a valid license number." << endl;

return 0;

另外，如果你有时间，看看Why is "using namespace std;" considered bad practice? .