Mein Programm zählt die Vokale nicht

Question

Ich habe eine Hausaufgabe, die die Vokale in der Zeichenfolge zählen muss. Mein Programm läuft, aber Konsonanten und Leerzeichen werden nicht ignoriert. Gibt es etwas, das mir im Code fehlt?

static void main(String[] args) { 
    // TODO Auto-generated method stub 
    boolean stopProgram = false; 
    while(!stopProgram == true) { 
     System.out.println("Enter a Word or \"q\" to end the program: "); // prompt the user for input 
     Scanner wordScanner = new Scanner(System.in); 
     String word = wordScanner.nextLine(); 
     if(!word.equals("q")) { 
      String vowels[] = {"a","e","i","o","u","y","A","E","I","O","U","Y"}; //define upper and lower case vowels 
      int count = 0; 
      for(int i = 0; i < word.length(); i++) { 
       if(String.valueOf(word.charAt(i)).equals(vowels[0]) || 
         String.valueOf(word.charAt(i)).equals(vowels[1])); 
         String.valueOf(word.charAt(i)).equals(vowels[2]); 
         String.valueOf(word.charAt(i)).equals(vowels[3]); 
         String.valueOf(word.charAt(i)).equals(vowels[4]); 
         String.valueOf(word.charAt(i)).equals(vowels[5]); 
         String.valueOf(word.charAt(i)).equals(vowels[6]); 
         String.valueOf(word.charAt(i)).equals(vowels[7]); 
         String.valueOf(word.charAt(i)).equals(vowels[8]); 
         String.valueOf(word.charAt(i)).equals(vowels[9]); 
         String.valueOf(word.charAt(i)).equals(vowels[10]); 
String.valueOf(word.charAt(i)).equals(vowels[11]); 

         count++; 
       } 
      System.out.println(("The Vowel Count is: ") + count); 
      } 


    } 
} 

}

Answer 1

Das ist Ihre if-Anweisung:

if(String.valueOf(word.charAt(i)).equals(vowels[0]) || 
        String.valueOf(word.charAt(i)).equals(vowels[1])); 

Sie brechen die if-Anweisung durch die ; ganz am Ende.

Danach gibt es eine Menge von Aussagen (deren Ergebnis ignoriert wird)

String.valueOf(word.charAt(i)).equals(vowels[1])); 

, die sich nicht mehr in der if-Anweisung sind.

Dann inkrementieren Sie die Zählung mit jeder Iteration - unabhängig von der if, die durch ; unterbrochen wird.

Answer 2

Zuerst sollten Sie Ihren Code ein wenig sauberer machen: Erstellen Sie eine Liste von Vokalen außerhalb Ihrer while, so dass Sie nicht bei jeder Iteration erstellen, dann verwenden Sie vielleicht eine doppelte for-Schleife durch Ihre Registerkarte von Vokalen:

static void main(String[] args) { 
    // TODO Auto-generated method stub 
    boolean stopProgram = false; 
    String vowels[] = {"a","e","i","o","u","y","A","E","I","O","U","Y"}; //define upper and lower case vowels 
    while(!stopProgram == true) { 
     System.out.println("Enter a Word or \"q\" to end the program: "); // prompt the user for input 
     Scanner wordScanner = new Scanner(System.in); 
     String word = wordScanner.nextLine(); 
     if(!word.equals("q")) { 
      int count = 0; 
      for(int i = 0; i < word.length(); i++) { 
       for(int j = 0; j < 12; ++j){ 
        if(String.valueOf(word.charAt(i)).equals(vowels[j]){ 
         ++count; 
         break; 
        } 
       } 
      } 
     System.out.println(("The Vowel Count is: ") + count); 
     } 
    } 
}