C++チャーチェックはひどい値を与えます

Question

私は数多くの学生の性別をチェックし、リストするプログラムを作ろうとしています。 私のコードに何が間違っているのかよく分かりませんが、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; 
} 

Answer 1

あなたが期待するように、ライン

m,l=0; 

は動作しません。 comma operatorを参照して、最初のオペランド（この場合はm）を評価し、結果を破棄し、2番目のオペランドを評価して返します。したがって、lだけがゼロに設定されます。私は、上に示したように

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

ように私はまた、彼らが必要とされる場所にiような変数の宣言を移動するだろう、この行に宣言を移動すると、一度に変数を初期化することをお勧めします。すべての宣言を関数の先頭に置く必要はありません。その後

出力endl最初の変数の前と後ではなく秒後

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

場所。出力の最後の行の後ろにendlがなければなりません。さもなければ、あなたの値の直後にコンソールプロンプトが表示されます。あなたの配列がこのサイズを持っているとして、あなたはまた、20以下の値が入力されていることを確認する必要があります

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

：それはこのような何かを持って読みやすさを改善するだろう。しかし、これが必要なことさえありません（実際のコードにはありますが、MCVEにはないかもしれません）。入力を読み込むときに変数をインクリメントするだけで、配列に格納する必要はありません。これは、この任意の制限を取り除きます。実際に値が必要な場合は、固定サイズの配列の代わりにstd::vectorを使用する必要があります。