2017-05-25 20 views
-2

次のコードスニペットをチェックし、その後、すべての3例では、それは同じことん:同じアクションで条件チェックを減らすには?条件の

if (TEMP_UUID_1.equals(fsTempUUID) && "P1".equals(siv.getParentUuid())) { 
     iRating.setFinSalesAmount(value); 
} else if (TEMP_UUID_2.equals(fsTempUUID) && "P2".equals(siv.getParentUuid())) { 
     iRating.setFinSalesAmount(value); 
} else if (TEMP_UUID_3.equals(fsTempUUID) && "P3".equals(siv.getParentUuid())) { 
     iRating.setFinSalesAmount(value); 
} 

が、私は条件をチェックした後、それは同じことをしているので、何とかこれを変換することができますか?

+2

なぜ '||'を使用しないのですか? – Mritunjay

+0

use "または"( '||')? –

+0

TEMP_UUID_1とは何ですか?それが文字列の場合は、switch文を使用できます。 – matt

答えて

2

は、単一if文を使用してORオペレータ(||)で、これらの条件を分ける:&&||よりも優先順位が高いとあなたが括弧で初期条件を囲む必要はありません

if (TEMP_UUID_1.equals(fsTempUUID) && "P1".equals(siv.getParentUuid() 
    || 
    TEMP_UUID_2.equals(fsTempUUID) && "P2".equals(siv.getParentUuid() 
    || 
    TEMP_UUID_3.equals(fsTempUUID) && "P3".equals(siv.getParentUuid()){ 

    iRating.setFinSalesAmount(value); 
}