计数元音c ++

Question

当用户输入 10 个字母时，程序应该知道其中有多少是元音。我写了这段代码：

while (count<10)
    {
        cin >> n;
        count++;
        if (n == 'A' || n == 'a' && n == 'E' || n == 'e' && n == 'I' || n == 'i' && n == 'O' || n == 'o' && n == 'U' || n == 'u')
            {
                total++;
            }
    }

cout <<  total << endl;

即使用户输入了元音，这也会产生 0 的输出。有什么问题吗？

Answer 1

让我们从稍微减少条件开始，只看a和e

if (n == 'A' || n == 'a' && n == 'E' || n == 'e')

然后为了简单起见只考虑小写字母（但保留了问题）

if (n == 'a' && n == 'e')

如果你大声读出来，它会说“如果n 是'a' 并且n 是'e'”。 “AND”（来自&& 运算符）意味着两个 条件都必须为真。你在这里创造了一个不可能的条件，如果n 是'a'，那么它不是'e'，所以你得到if (true && false) - 这是错误的。如果n 是'e'，那么它就不是'a'，所以你得到if (false && true)。

如果至少有一个相等比较为真，只需将所有&&（和）运算符替换为||（或）运算符即可使条件为真。

if (n == 'A' || n == 'a' || n == 'E' || n == 'e' 
    || n == 'I' || n == 'i' || n == 'O' || n == 'o' 
    || n == 'U' || n == 'u')

---

有一些方法可以简化条件。

一种是加一个#include <cctype>，用std::tolower将n转为小写，那么只需要和小写字符进行比较即可。

n = std::tolower(n);
if (n == 'a' || n == 'e' || n == 'i' || n == 'o' || n == 'u')

另一种重复性较低的方法是创建一个包含所有元音的std::string，然后在顶部创建see if it contains n、#include <string>。

std::string vowels = "aeiouAEIOU";
while (/*...*/) {
   // ...
   if (vowels.find(n) != std::string::npos) {
     ++total;
   }
}

正如n314159 所指出的，如果您使用的是C++ 17 或更高版本，则可以改用std::string_view，这样更便宜。 #include <string_view>

static constexpr std::string_view vowels = "aeiouAEIOU";   
while (/*...*/) {
   // ...
   if (vowels.find(n) != std::string_view::npos) {
     ++total;
   }
}

Answer 2

我建议使用带有直通的switch 语句，这样更具可读性但也可能更有效（可以通过跳转表实现）。

int count = 10;
while (count--) {
    switch(std::tolower(n)) {
        case 'a': case 'e': 
        case 'i': case 'o': 
        case 'u': total ++; break;
        default:;
    }
}