超短路

Question

必须为这个类调试某段代码，我无法弄清楚为什么这个JS会计算所有字母而不仅仅是元音。

var text, i, sLength, myChar;
var count;
var text = prompt("Type a phrase please"); //put var in front of text and   fixed " in place of ' in front of type
count = 0;
for (i = 1; i <= text.length; i+= 1){
    myChar = text[i];
    if (myChar == 'a' || 'o' || 'e' || 'u'){ //switched to the proper vowels
        count += 1;
        console.log('Vowel:', myChar);   
    }
    console.log(myChar, count);
}
alert (count); //put the count in () instead of alert

Answer 1

进行比较的线路未正确进行检查。它应该是：

if (myChar == 'a' || myChar == 'o' || myChar == 'e' || myChar == 'u')

Answer 2

正确的JS就是这个。

var text, i, sLength, myChar;
var count;
var text = prompt("Type a phrase please"); //put var in front of text and   fixed " in place of ' in front of type
count = 0;
for (i = 1; i <= text.length; i+= 1){
    myChar = text[i];
    if (myChar == 'a' || myChar == 'o' || myChar =='e' || myChar == 'u'){ //switched to the proper vowels
        count += 1;
        console.log('Vowel:', myChar);   
    }
    console.log(myChar, count);
}
alert (count)

你的工作不起作用的原因是因为当你说表达式为||时，它会询问“o”是否为真'o'，因为字符串文字不是假值（0，“”，null，undefined，false），所以它对于循环的每次迭代都是正确的。

Answer 3

超短路

function countVowels (string) {
    return string.match(/a|e|i|o|u/g).length;
}

用作：

alert(countVowels(prompt("Type a phrase please")));

<小时/> 您的if条件无效。如果您希望快速检查值是否是某些选项之一，请尝试以下操作：

if (['a', 'e', 'i', 'o', 'u'].indexOf(myChar) > -1) {
}

<小时/>

功能！

function hasOptions (c) {
    return arguments.splice(1).indexOf(c);
}

然后

if (hasOptions(myChar, 'a', 'e', 'i', 'o', 'u'))

为什么这段代码不仅算上元音？

3 个答案:

超短路

功能！