バリデーションで実際にnullをチェックする必要はありますか？

Question

私は、JSF 2.0の時点で、valueパラメータがnullの条件を適切に処理するためにバリデータ・ルーチンが必要であると読んでいます。 selectBooleanCheckbox：

public void validateTandC(FacesContext context, UIComponent source, Object value) { 
    if (value == null || !(value instanceof Boolean)) { 
     FacesMessage fm = new FacesMessage(FacesMessage.SEVERITY_ERROR, 
       "Terms and Conditions Not Accepted", "Value Not Present"); 
     throw new ValidatorException(fm); 
    } 
    if (((Boolean) value) != true) { 
     FacesMessage fm = new FacesMessage(FacesMessage.SEVERITY_ERROR, 
       "Terms and Condistions Not Accepted", 
       "You must check the box that confirms you accept the Terms and Condtions "); 
     throw new ValidatorException(fm); 
    } 
} 

これは、このメソッドはONLY関連するページにのみ単一hから参照されることを考えると、かなり退屈な構造である。だから、僕はこのようになります。このvalidateメソッドを書きました。その場合、最初のはのステートメントが真実に解決された場合、なぜですか？私はそれを持っていない場合、NPEは現実に起こるだろうか？ valueをnull場合

Answer 1

を実際問題として、 <h:selectBooleanCheckbox>コンポーネントにバインドされたプロパティはnullになることはありません。チェックボックスをオフにすると、falseになり、その逆もあります。したがって、Validatorのnullを確認する必要はありません。

はまた、他のUIInputコンポーネントについて、私はあなたが単純に自動的に処理されますtrueとnullケースに自分のrequired属性を設定することができると信じて。例えば、ユーザは、上記のテキストボックスを逃した

<h:inputText id="myInput" required="true" requiredMessage="Hey yo! Type something here." /> 
<h:message for="myInput" /> 

場合は、彼が提出クリックした後Hey yo! Type something here.メッセージが表示されます。

Answer 2

、次いで!(value instanceof Boolean)もtrue（nullがBooleanのインスタンスではない）が返されます。

だから、機能を変更したり、NPEを心配することなく、あなたの小切手のvalue == null部分を排除するために、これを簡素化することができます - あなたの「退屈な建設を」排除：

if (!(value instanceof Boolean)) {