Тестирование индекса строкового массива в 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; 



} 

Здесь я проверяю элемент в String for Vowel, может ли кто-нибудь предложить альтернативный метод для этого? Мне нужно подсчитать количество гласных в строке.

Этот код работает идеально для меня, просто нужно найти альтернативный метод, когда мне не нужно набирать слишком много «||» и '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') 

, а также заботиться о случае чуткости

Answer 2

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

всегда возвращает истину, потому что 'E' всегда верно. Вы ищете

strchr("AEIOU", str[j]) 

Обратите внимание на j, вы получили переменную неправильно цикла.

(Кроме того, в C++, вы будете хотеть использовать iostreams и getline вместо cstdio и fgets.)

Answer 3

путь? или синтаксис?

я думаю, что его ошибка синтаксиса .. она должна быть как

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

Если вы спрашиваете о лучшем способе, я не думаю, что есть какие-либо .. Вы делаете в O (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++ array/vector, людям не нравятся 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; 
}