IF条件付きステートメントのフォーマットが間違っていますか？

Question

は簡単なものです。私は3つのチェックボックスを持っており、どのチェックボックスが入力されたかに基づいて実行すべき多数の条件文を持っています。この場合、私は3つすべて「チェック」しているので、これらのステートメントのどれも実行する必要はありません。私はそれらを間違ってフォーマットしたように見えます。私は私が間違って何をしたか知ってみましょう...ありがとう

ロガー：3のうち2に該当する場合

// form.isOpenLevel() = true 
    // form.isPhyCompLevel() = true 
    // form.isFinCompLevel() = true 

これらの3つのステートメントのみを実行する必要があります。少なくとも私がやろうとしていたことだ。

if ((form.isOpenLevel() == true && form.isPhyCompLevel() == true) && (form.isFinCompLevel() != true)); 
     { 
      paramBean.addFilter(new DetFilterCriteriaBean("MSST_HEADER_DATA_MV.FIN_COMP_DATE","is","NULL")); 

     } 


     if ((form.isPhyCompLevel() == true && form.isFinCompLevel() == true) && (form.isOpenLevel() != true)); 
     { 
      paramBean.addFilter(new DetFilterCriteriaBean("MSST_HEADER_DATA_MV.PHYS_COMP_DATE","is","NOT NULL")); 

     } 

     if ((form.isOpenLevel() == true && form.isFinCompLevel() == true) && (form.isPhyCompLevel() != true)); 
     { 
      paramBean.addFilter(new DetFilterCriteriaBean("nvl2(MSST_HEADER_DATA_MV.PHYS_COMP_DATE,MSST_HEADER_DATA_MV.FIN_COMP_DATE,'X')","is","NOT NULL")); 

     } 

Answer 1

まず第一に、余分なparanthesisは冗長である：

((a && b) && c) 

はまた

(a && b && c) 

に相当します - 「3のうち2に該当する場合、これらの3つのステートメントのみ実行する必要があります手段

if (b && c) 

で十分です。

最後に、if後のセミコロンは、条件を終了さ：

if (b && c); 
    statement; 

は、あなたの文は常にステートメ理由です

if (b && c) 
{ 
} 
statement 

に相当します。

条件の後に;を削除します。

Answer 2

if文の最後にセミコロンを取り除きます。これは先制的にそれを閉じます。言い換えれば

、この：

if (a); { 
    //do something 
} 

がこれに相当します//do somethingは常に実行されるブロックフローティングになり、あなたのif文の条件部分

if (a) { 
} 

{ 
    //do something 
} 

た場合実行するコードはありません。

Answer 3

ifステートメントの最後にセミコロンがあります。削除してください。条件付きの権利を終了します。中括弧で囲まれたブロックは、の外側で実行されています。条件はです。

さらに、同等かどうかを確認する必要はありません。 IFライン上のセミコロンを取り除く

if (form.isOpenLevel() && form.isPhyCompLevel() && !form.isFinCompLevel()) 
{ 
    ... 
} 

Answer 4

：あなたの方法は、あなたが直接評価できるboolean値を返すされています。

セミコロンは基本的にJavaの「DO NOTHING」を意味します。

if (complex-logic) ; 

if (complex-logic) { /* DO NOTHING */ }