Question

我正在尝试制作一个程序来检查和列出一些学生的性别，m是fem，而我是男性。我不确定我的代码有什么问题，但是当我打印出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;
 }

Answer 1

该行

m,l=0;

不能按预期工作。查找comma operator，它会计算第一个操作数（在这种情况下只是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而不是固定大小的数组。

C ++ Char检查给出了无耻的价值

1 个答案: