C++での文字列配列のテストインデックス？

Question

#include<iostream> 
#include<cstdio> 
using namespace std; 

int main() 
{ 

    char str[80]; 
    int i=0; 
    cout<<"Enter the String "; 
    gets(str); 
    for (int j=0;str[j]!='\0';j++) 
    if (str[i]=='A'||'E'||'I'||'O'||'U') 
    i++; 
    cout<<"Number of vowels is: "<<i; 



} 

ここで私はVowelのStringの要素をチェックしていますが、誰でもこの代替方法を提案できますか？私は文字列中の母音の数を数える必要があります。

このコードは私にとって完璧に機能していますが、別の方法を見つける必要があります。これは、あまりにも多くの "||" 「a」と「A」の違いがあります。

Answer 1

if (str[i]=='A'||'E'||'I'||'O'||'U') 

これは間違っ

では次のようにする必要があります：

if(str[i]=='A' || str[i]=='E'||str[i]=='I'||str[i]=='O'||str[i]=='U') 

ともケースの世話を'E'は常にtrueであるため

Answer 2

str[i]=='A'||'E'||'I'||'O'||'U' 

は常に、trueを返します敏感。あなたは探しています

strchr("AEIOU", str[j]) 

あなたはループ変数が間違っていることに注意してください。

（また、C++で、あなたの代わりにcstdioとfgetsのiostreamsとgetlineを使用することをお勧めします。）

Answer 3

ウェイ？または構文？

私は...それはあなたがより良い方法について尋ねている場合は、私はすべてのがあると思いません

if(str[i]=='A' || str[i]=='E'||str[i]=='I'||str[i]=='O'||str[i]=='U')

ようにする必要があり、その構文エラーだと思います..あなたはOでやっている （n）複雑さ、それは十分に私が推測する良いです！あなたはハッシュを使用する場合でも、あなたは、文字列内のすべてのn個の文字をハッシュする必要があります。..

Answer 4

よりC++ - っぽいソリューション：

std::string vowels("aAeEiIoOuU"); 
for (int j=0;str[j]!='\0';j++) 
{ 
    if (vowels.find(str[j]) != std::string::npos) 
    i++; 
} 

Answer 5

またはテーブルを使用します。

#include <iostream> 
#include <string> 

int main() 
{ 
    char list_v[256] = { 0 }; 
    list_v['a'] = list_v['e'] = list_v['i'] = list_v['o'] = list_v['u'] = 1; 
    list_v['A'] = list_v['E'] = list_v['I'] = list_v['O'] = list_v['U'] = 1; 
    std::string str = "a sentence here"; 
    uint32_t cnt = 0; 
    for (uint32_t i = 0; i < str.length(); i++) 
    cnt += list_v[str[i]]; 
    std::cout << "found " << cnt << std::endl; 
    return 0; 
} 

または同じ目的のマップを使用するか、またはC++の配列/ベクトルでは、人々はここでc配列を好きではありません。これは当然ASCIIのためだけに機能します。

Answer 6

inline bool is_vowel(char a) 
{ 
    a=std::tolower(a); 
    switch(a) 
    { 
    case 'a':  case 'e': 
    case 'i':  case 'o': 
    case 'u': 
     return true; 
    } 
    return false; 
} 

int main() 
{ 
    std::string line; 
    std::cout << "enter text" << std::endl; 
    std::getline(std::cin, line); 
    int vowel=std::count_if(line.begin(), line.end(), is_vowel); 
    std::cout << "Number of vowels: " << vowel << std::endl; 
}