2013-01-01 170 views
-2
public void cariData(){ 
    if(!= dataPasienview.getTextCari().getText().trim().isEmpty()){ 
     dataPasienimpl.getCariData(dataPasienview.getTextCari().getText()); 
     isiTableCariData(); 
    } else{ 
     JOptionPane.showMessageDialog(dataPasienview, "silahkan pilih data"); 
    } 
} 
+0

'如果(!= ...)'你缺少變量... – alfasin

+0

@alfasin作爲合適的表情是一個布爾(的isEmpty()),我猜想=應該簡單地刪除。 –

+1

@dystroy我想你是對的! (這是最糟糕的,如果條件我看過一段時間...)+1 – alfasin

回答

2

取出=!

if(!dataPasienview.getTextCari().getText().trim().isEmpty()){ 
+0

好的非常感謝你, – user1940743

0

正如其他人所說,改變=只!

您還應該改變測試。 之類的語句

if (!x) { 
    doSomething(); 
} else { 
    doErrorThing(); 
} 

會絆倒靜態代碼分析工具爲混亂。它更容易閱讀的

if (x) { 
    doErrorThing(); 
} else { 
    doSomething(); 
} 
+0

我不同意。首先放置正常路徑是非常自然的,即非空文本。 –

+0

PMD(一種非常常見的靜態分析工具)不同意你的觀點:)請參閱http://www.ing.iac.es/~docs/external/java/pmd/rules/design.html上的「ConfusingTernary」條目 –

相關問題