Java - 引数で渡されたnull値を処理する方法は？

Question

私はこの簡単な関数を書いた：

private String getOperatorForCardinality(String op) 
{ 
    String operator =""; 
    if(op!=null) 
    { 
     if(op.equals(">=")) 
     { 
      operator = ">="; 
     } 
     else if (op.equals("<=")) 
     { 
      operator = "<="; 
     } 
    } 
    else 
    { 
     operator = "empty"; 
    } 

    return operator; 
} 

文字列を返します。

この関数を呼び出すメインプログラムでは、引数がnullの場合、コンパイラはNullPointerExceptionのエラーを表示します。 理由ははっきりしていますが、引数を渡すとnull値を処理する方法がわかりません。

Answer 1

これは、nullポインタ例外をスローすることはできません

if (op==null) 
operator = "empty"; 

Answer 2

トライ反対：

private String getOperatorForCardinality(String op) 
{ 
    String operator = ""; 
    if(">=".equals(op)) 
    { 
     operator = ">="; 
    } 
    else if ("<=".equals(op)) 
    { 
     operator = "<="; 
    } else { 
     operator = "empty"; 
    } 

    return operator; 
} 

Answer 3

それはあなたがNPEをスローするように掲載のコードのために不可能です。エラーは別の場所にあるか、またはあなたが思っているコードを実行していない（つまり、再コンパイルしていないなど）。

private static List<String> OPS = Arrays.asList("<=", ">="); // can add more valid ops 

private static String getOperatorForCardinality(String op) { 
    if (op == null) 
     return "empty"; 
    return OPS.contains(op) ? op : ""; 
} 

それとも、ネストされた三元系を気にしない場合：

private static String getOperatorForCardinality(String op) { 
    return OPS.contains(op) ? op : op == null ? "empty" : ""; 
} 

少ないコードは、通常は明確コードであり、かつにはあまり場所を残して、あなたの方法をに簡素化することができる、と述べた

バグが潜んでいる。

Answer 4

それは防御的プログラミングと呼ばれ、あなたが何か行う必要があります。

private String getOperatorForCardinality(String op) { 
    if(null == op) { 
     //return null; 
     //throw new NullPointerException("..."); 
    } 
    .... 
} 

をあなたの方法は、反応パラメータがnullの場合はnullを返す必要があるか、例外をスローする必要がありますどのように考えるべきか。一般的に、パラメータが決してnullにならないようにして、常にチェックして行動を取らなければなりません。