母音と子音Java

Question

文字列の母音と子音の数を数えることについてこのコードを終了するのに問題があります。 文字列をクラスに送信すると、getVowelCountメソッドとgetConstCountメソッドの両方がエラーになることが判明しました。問題は、両方のメソッドのforループにあり、isLetterメソッドを文字列ですが、わかりません。私はスレッドで

例外を取得し、 "メイン" のjava.lang.NullPointerException/DCMvowelsAndConsonants.getVowelCount（DCMvowelsAndConsonants.java:30）で/ DCMvowelsAndConsonantsDriver.main（DCMvowelsAndConsonantsDriver.java:40）で

Iこれはすでに大きいので、私はドライバを投稿する必要があるか分からない。

public class DCMvowelsAndConsonants 
{ 
    String line; 

    public DCMvowelsAndConsonants() 
    { 
     String line = " "; 
    } 

    public DCMvowelsAndConsonants(String l) 
    { 
     String line = l; 
    } 

    public static boolean isLetter(char i) 
    { 
     return i == 'a' || i == 'A' || i == 'e' || i == 'E' || i == 'i' || i == 'I' || i == 'o' || i == 'O' || i == 'u' || i == 'U'; 
    } 

    public int getVowelCount() 
    { 
     int vowelCount = 0; 

     for(int i = 0; i < line.length(); i++) 
     { 
     if(isLetter(line.charAt(i))) 
      vowelCount++; 
     } 
     return vowelCount; 
    } 

    public int getConstCount() 
    { 
     int constCount = 0; 

     for(int i = 0; i < line.length(); i++) 
     { 
     if(!isLetter(line.charAt(i))) 
      constCount++; 
     } 
     return constCount; 
    } 
} 

Answer 1

「文字列」部分を削除するようにコンストラクタを変更します。これは、メンバー変数に代入する代わりに、ローカル変数を宣言して代入しています。

だから、彼らは次のようになります。

public DCMvowelsAndConsonants() 
    { 
     line = " "; 
    } 

    public DCMvowelsAndConsonants(String l) 
    { 
     line = l; 
    } 

Answer 2

あなたの問題は、あなたのコンストラクタです。コンストラクタで新しい変数を作成しています。あなたのコンストラクタを置き換えます

@TimBiegeleisen

public DCMvowelsAndConsonants() { 
     this.line = " "; 
} 

public DCMvowelsAndConsonants(String l) { 
     this.line = l; 
}