C++字符检查给出了惊人的值

C++ Char checking gives outrageous value

本文关键字：字符 检查 C++      更新时间：2023-10-16

我正在尝试制作一个程序来检查和列出一些学生的性别，m是fem，l是男性。我不确定我的代码出了什么问题，但当我打印出m和l变量时，它们要么非常大。几个小时以来一直在努力解决这个问题，非常感谢您的帮助，干杯。

第页。对不起，我的英语不好。

#include<iostream>
#include<string>
using namespace std;
main()
{
char gender[20];
int jlh,i,j,m,l;
cin>>jlh;
system("cls");
for(i=0;i<jlh;i++)
{   cout<<"Data "<<i+1<<endl;
cout<<"Enter your gender - "<<endl;
cin>>gender[i];
}
m,l=0;
for(i=0;i<jlh;i++){
if(gender[i]=='p'){
m=m+1;
}
else if(gender[i]=='l'){
l=l+1;
}
}
cout<<endl<<l<<endl;
cout<<m;
}

行

m,l=0;

不会像你期望的那样工作。查找逗号运算符，它计算第一个操作数(在本例中仅为m)，丢弃结果，然后计算并返回第二个操作数。因此只有l被设置为零。我建议将声明移到这一行，并一次性初始化变量，就像一样

int m=0, l=0;
for (int i=0; i<jlh; i++) 
...

我还将把i等变量的声明移到需要它们的地方，如上所示；没有必要将所有声明都放在函数的开头。

然后输出

cout<<endl<<l<<endl;
cout<<m;

将endl放在第一个变量之前和之后，但不放在第二个变量之后。您应该在输出的最后一行后面有一个endl，否则控制台提示就在您的值后面。有这样的东西会提高可读性：

std::cout << "Number of females: " << m << std::endl; 
std::cout << "Number of males:   " << l << std::endl;

您还应该确保输入的值不超过20个，因为您的数组具有此大小。但甚至不需要这样做(可能在您的真实代码中有，但在MCVE中没有)：您可以在读取输入时增加变量，而无需将其存储在数组中。这就摆脱了这个任意的限制。如果您确实需要这些值，则应该使用std::vector，而不是固定大小的数组。

相关文章：