C++の単純なIO母音カウントプログラム

Question

char ch; 
//Get data from user 
cout << "Enter your sentence on one line followed by a # to end it: " << endl; 

while (cin >> character && character != '#') 
{ 
    cin.get(ch); 
    ch = static_cast<char>(toupper(ch)); 
    outFile << ch; 

    if (character == 'A' || character == 'E' || character == 'I' || character == 'O' 
       || character == 'U') 
    { 
     vowelCount ++; 

    } 
} 
outFile << "number of vowels: " << vowelCount << endl; 

文章を入力しようとしていますが、いくつの母音、空白、その他の文字が読み込まれていますか。しかし、vowelCountは正しいとは言えませんし、同じ文を出力ファイルに書き込むこともできません。何かヒント？

Answer 1

変数vowelCountの宣言/初期化が表示されていません。 int変数にはデフォルト値を持っていない、C++では

int vowelCount; // notice the variable is not initialized. 

：私はあなただけ宣言（および初期化されていない）、それはのような文を使用していると仮定します。あなたはこのようなコードを書かれている場合は、明示的のようなステートメントを使用して、その値を初期化することによって、それを修正することができます。また

int vowelCount = 0; 

、あなたのループは（2つの文字のうち1をスキップ）各反復で2つの文字を読み取り、あなたが不足しています母音はYです。 PMRのコメントは問題はあなたが各ループ反復で2つの文字で読みますが、最初のものだけをチェックしているということです、を示しているだけのように

//Get data from user 
cout << "Enter your sentence on one line followed by a # to end it: " << endl; 

int vowelCount = 0; 
while (cin >> character && character != '#') 
{ 
    character = toupper(character); 

    if (character == 'A' || character == 'E' || character == 'I' || character == 'O' 
       || character == 'U' || character == 'Y') 
    { 
     vowelCount ++; 

    } 
} 
outFile << "number of vowels: " << vowelCount << endl; 

Answer 2

：

修正例は次のようになります。あなたは既にcharacter` `に読んでいる

while (cin >> character && character != '#') 
{ 
    character = static_cast<char>(toupper(character));