このコードは正しく実行されません。誰かがここで私を助けることができますか？

Question

/* File name: IsConsonant.java 
* -------------------------------------- 
* Tests alphabets for consonants 
*/ 

import acm.program.*; 

public class IsConsonant extends ConsoleProgram { 

    public void run() { 
     String ch = readLine ("Enter an alphabet: "); 
     isEnglishConsonant(ch.charAt(0)); 
     if (true){ 
      println("True"); 
     } else { 
      println("False");} 
     } 

    private boolean isEnglishConsonant(char ch){ 
     switch (Character.toUpperCase(ch)){ 
      case 'A': 
      case 'E': 
      case 'I': 
      case 'O': 
      case 'U': 
       return false; 
     default: 
       return true; 
     } 
    }  
} 

Answer 1

if (true) 

は常にtrueとテストされます。あなたは代わりに行う必要があります

if (isEnglishConsonant(ch.charAt(0))) 

Answer 2

コードisEnglishConsonant(ch.charAt(0));の3行目は、変数に格納されていないブール値を返すようです。私が間違っていないかどうか確認してください。

Answer 3

if (true)常に返信atrueです。

むしろやる：あなたのcase文でbreakを追加するのを忘れ

boolean consonant = isEnglishConsonant(ch.charAt(0)); 
if (consonant) { 
     println("True"); 
} else { 
     println("False"); 
} 

。まあ

switch (Character.toUpperCase(ch)){ 
    case 'A': case 'E': case 'I': case 'O': case 'U': 
      return false; 
    break; 

    default: 
     return true; 
} 

Answer 4

public void run() { 
    String ch = readLine ("Enter an alphabet: ");   
    if (isEnglishConsonant(ch.charAt(0))){ 
     println("True"); 
    } else { 
     println("False"); 
    } 
} 

private boolean isEnglishConsonant(char ch){ 
     switch (Character.toUpperCase(ch)){ 
      case 'A': case 'E': case 'I': case 'O': case 'U': 
      return false; 
     default: 
      return true; 
     } 
    } 

Answer 5

真がtrueの場合、あなたのチェックは、それは常にケースになる場合！

if (true){ 

それはあなたがすべての後にチェックしたいものだ、isEnglishConsonant(ch.charAt(0))代わりのtrueを入れて！

Answer 6

ここでは、真偽値を返すメソッドを直接比較していません。そう、それは働かない。好きなように変更してください

if(isEnglishConsonant(ch.charAt(0))) 
{ 
println("True"); 
} 
else 
{ 
println("False"); 
}