检查字符串中的小写字符或空格.布尔值不起作用

Checking for a lowercase character or space in a string. Boolean not working

本文关键字：空格 布尔值 不起作用 字符 字符串 检查      更新时间：2023-10-16

这应该检查输入的字符串是否是有效的许可证号。有效数字不能包含任何小写字母或空格。如果我输入"cat"，它应该看到字符"c"更低，所以它应该将bool isValid设置为false，从循环中断开，然后打印"cat不是有效的许可证号"。然而，它没有，它只是使布尔值对我在开始时设置的任何内容都有效，这是真的。因此，如果我将其初始化为 false 并且输入"CAT"是有效的，则仍然是 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;
}

问题就在这里：

bool isValid = false;
break;

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

除此之外，您还可以删除此部分

else
bool isValid = true;

因为当到达这部分代码时，无论如何isValidtrue。您也可以简单地写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;

另外，如果您有时间，请查看为什么是"使用命名空间 std;"被认为是不好的做法？。

你只需要从 for 循环中删除bool类型(在if和else条件下( - 它定义了一个新的局部变量isValid它隐藏了循环外部的isValid定义(在注释// Initialize boolean下(，一旦命令流离开定义它的块，这个局部定义就不复存在了。

在这个块之外，定义isValid = true再次发挥作用(处理逻辑中保持不变(。

相关文章：