Javaエラー：java.lang.NullPointerException

Question

ファイルからいくつかのレコードを読み込み、特別な方法で並べ替えるコードを書いています。私はこのようなコードを試してみました：

public class Main { 

    static class judgement implements Comparable<judgement> { 
     public int q; 
     public int d; 
     public int r; 

     public int compareTo(judgement j) { 
      int k = ((judgement) j).q; 
      return 0; 
     } 
    } 

    public static void method() throws Exception { 
     judgement[] judgements; 
     judgements = new judgement[18425]; 
     try { 
      // fill the "judgements" array 
     } finally { 
      Arrays.sort(judgements); 
     } 
    } 

    public static void main(String[] args) throws Exception { 
     method(); 
    } 

}

をしかし、私は、関数のcompareToにエラーNullPointerExceptionがを取得します。誰も私にこの問題を手伝うことができますか？

Answer 1

null値で配列を初期化しています。

judgements = new judgement[18425]; 

あなたはnullと比較して値をチェックしていません。 if文を使用する必要があります。

public int compareTo(judgement j) { 
      int k =0; 
    if(j!=null){ 
    k = ((judgement) j).q; 
    } 

       return 0; 
      } 

Answer 2

これは、配列がヌルポインタで埋められているためです。コンパレータはヌルオブジェクトを互いに比較することはできません。

アレイを作成しても、アレイの各要素は初期化されません。

Answer 3

compareTo（）の実装に関しては、配列にnull以外の要素のみが含まれていることを確認してください。 それ以外の場合、関数compareTo（）では、既存の判断で「q」属性を呼び出すと、NullPointerExceptionが発生します。

さらに、タブを避け、コレクションを使用することをおすすめします。 あなたの場合、配列の代わりにTreeSetオブジェクトを使用することができます。なぜなら内部的に要素のcompareTo（）メソッドを使用するからです。

Answer 4

おそらく起こっていることは、配列の一部の要素がnullであることです。

compareTo機能にヌルチェックを含めることをお勧めします。 また、現在compareTo関数は機能していません。常に0を返します。これは、すべての項目が等しいと言っているようなものです。

もう1つの問題は、使用しないでください、try{}finally{}を使用しているようです。