作者热门文章
- html - 出于某种原因,IE8 对我的 Sass 文件中继承的 html5 CSS 不友好?
- JMeter 在响应断言中使用 span 标签的问题
- html - 在 :hover and :active? 上具有不同效果的 CSS 动画
- html - 相对于居中的 html 内容固定的 CSS 重复背景?
我正在尝试制作一个程序来检查和列出一些学生的性别,m 是女性,l 是男性。我不确定我的代码有什么问题,但是当我打印出 m 和 l 变量时,它们要么非常大。几个小时以来一直在努力解决这个问题,非常感谢您的帮助,干杯。
P.S 抱歉我的英语不好。
#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;
没有像您预期的那样工作。查找 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++ 字符检查给出了离谱的值,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/46690574/
我是一名优秀的程序员,十分优秀!