Ich versuche, eine Funktion zu schreiben, die die Anzahl der Vokale in einem String zurückgibt

Question

Das ist mein Code, aber ich bin immer in der Konsole

function numOfVowels(string) { 
 
     let count = 0; 
 
    
 
     string = string.split(''); 
 
    
 
     const vowels = function(string) { 
 
     for (let i = 0; i > string.length; i++) { 
 
      if (string[i].toLowerCase() === 'a' || string[i].toLowerCase() === 'e' || string[i].toLowerCase() === 'i' || string[i].toLowerCase() === 'o' || string[i].toLowerCase() === 'u'){ 
 
      count+= 1; 
 
      } 
 
      return count; 
 
     } 
 
     }; 
 
    } 
 
    console.log(numOfVowels('Yellow Submarine'));

Answer 1

Sie zurückkehren, werden keinen Wert undefiniert von numOfVowels, und es gibt auch keine Notwendigkeit, die Zeichenfolge zu teilen, können Sie über sie iterieren, einen Blick auf den Code unten nehmen:

function numOfVowels(string) { 
 
     let count = 0; 
 
    
 
     for (let i = 0; i < string.length; i++) { 
 
     if (string[i].toLowerCase() === 'a' || string[i].toLowerCase() === 'e' || string[i].toLowerCase() === 'i' || string[i].toLowerCase() === 'o' || string[i].toLowerCase() === 'u'){ 
 
      count+= 1; 
 
     } 
 
     } 
 
     
 
     return count; 
 
} 
 

 
console.log(numOfVowels('Yellow Submarine'));

Answer 2

Hier ist Code korrigiert

erstes Problem war mit i > string.length sollte es i < string.length in for Schleife zweites Problem bei return count; war sollte es sein, nachdem die Schleife beendet wird und das dritte Problem war mit vowels Funktion es war nie innerhalb numOfVowels aufgerufen werden.

function numOfVowels(string) { 

    let count = 0; 

    string = string.split(''); 

    const vowels = function(string) { 

    for (let i = 0; i < string.length; i++) { 
     if (string[i].toLowerCase() === 'a' || string[i].toLowerCase() === 'e' || string[i].toLowerCase() === 'i' || string[i].toLowerCase() === 'o' || string[i].toLowerCase() === 'u'){ 
     count+= 1; 
     } 
    } 
     return count; 
    }; 
    return vowels(string); 
} 

console.log(numOfVowels('Yellow Submarine')); 

Answer 3

Es gibt einige Probleme hier:

1. Sie erklären aber nicht nennen die Funktion vowels.
2. Ihre Return-Anweisung return count; innerhalb Ihrer Schleife und wird auf der ersten Iteration zurückkehren.
3. Ihre bedingten im for Schleife verwendet > und es sollte < sonst Ihr Codeblock innerhalb der for Schleife als i nie größer sein würde laufen würde nie verwenden als string.length.

Einige alternative Implementierungen von Vokal mit regex Zählen & es2015:

var s = "Yellow Submarine"; 
 

 
// es2015/es6 
 
// using reduce 
 
let count1 = s.split('').reduce((count,char) => (/[aeiou]/i.test(char)) ? count + 1 : count, 0); 
 

 
// using filter 
 
let count2 = s.split('').filter(char => /[aeiou]/i.test(char)).length; 
 

 
console.log(count1); 
 
console.log(count2);

Answer 4

Wenn Sie ein bisschen kürzer Implementierung wollen (nur sayin):

countvowels=word=>word.split("").reduce((c,l)=>(["a","e","i","o","u"].indexOf(l.toLowerCase())+1)?++c:c,0); 

http://jsbin.com/durizarobu/edit?console