在Java中返回錯誤？

Question

我得到的代碼中的錯誤從我這部分代碼：

public boolean findCustomer(String inPersonalNumber){ 

    // check if personal number already exist 
    for (int i=0; i<customerList.size();i++) { 
     if(customerList.get(i).getCustomerPersonalNumber().equals(inPersonalNumber)){ 
      return true; 
     } 
    } 
    return true;  
} 

當我刪除了第一個真正的回報，而是到最後返回true，它沒有得到錯誤在我的Eclipse代碼，但爲什麼我不能擁有第一名，這是否會一樣？謝謝！

編輯：來自eclipse的錯誤消息說：此方法必須返回布爾類型的結果。我很困惑，因爲那不是我所做的？！

是的，休息必須在代碼

我可以寫一些其他的方式方法？

編輯編號2
爲什麼不能使用此代碼？

public boolean findCustomer(String inPersonalNumber){ 

// check if personal number already exist 
for (int i=0; i<customerList.size();i++) { 
    if(customerList.get(i).getCustomerPersonalNumber().equals(inPersonalNumber)){ 
      return true; 
     } 
     else { 
      return false; 
     } 
    } 
} 

此方法返回一個布爾值，所以我不明白爲什麼我得到一個錯誤！？代碼看起來適合我嗎？

Answer 1

您的編輯＃2無法編譯，因爲您的代碼有可能不會進入for-loop。這將是情況下，如果customerList.size()爲0。要解決這個問題，你只需要後添加return語句for循環，以及：

// check if personal number already exist 
for (int i=0; i<customerList.size();i++) { 
    if(customerList.get(i).getCustomerPersonalNumber().equals(inPersonalNumber)){ 
     return true; 
    } 
    else { 
     return false; 
    } 
} 
return false; 

這裏還有一點是，這個代碼沒有按」 t 邏輯上有很大意義：它只會只有根據您的列表中的第一個項返回true或false。這可能是而不是你想要什麼。所以仔細看看其他幾個答案，其中很多答案都是你如何做到這一點的好例子。

Answer 2

看起來像你已經關閉了自動構建 eclipse的功能。它可能會抱怨當你還沒有完全輸入代碼時曾經存在的錯誤！如果您出於某種原因退訂了系統，也會發生這種情況。

另外，如果條件不滿足，你不應該返回false嗎？

public boolean findCustomer(String inPersonalNumber) { 

    // check if personal number already exist 
    for (int i = 0; i < customerList.size(); i++) { 
     if (customerList.get(i).getCustomerPersonalNumber().equals(inPersonalNumber)) { 
      return true; 
     } 
    } 

    return false; 
} 

Answer 3

因爲無論如何你的for循環都會失去意義。
如果您想停止循環使用break;而不是第一次返回。

Answer 4

public boolean findCustomer(String inPersonalNumber){ 
    boolean result = false; 
    // check if personal number already exist 
    for (int i=0; i<customerList.size();i++) { 
     if(customerList.get(i).getCustomerPersonalNumber().equals(inPersonalNumber)){ 
      result = true; 
      break; 
     } 
    } 
    return result ;  
} 

Answer 5

首先回歸將返回只有在滿足了所有條件的情況下，但這種方法應該返回boolean按代碼。它也希望在失敗的情況下回歸。

刪除第一個返回不會影響編譯，因爲它有第二個返回，它將在沒有任何條件的情況下工作。

編輯：回答你的第二個問題

此代碼有兩個返回的，但如果你的customerList是什麼大小爲0，在這種情況下也，方法必須返回boolean值。對？爲此，編譯器問。

順便說一句，代碼沒有空檢查。

您的最終代碼可能是這樣的。在代碼中保留多個返回語句並不是一個好習慣。

public boolean findCustomer(String inPersonalNumber) { 
    boolean retVal = false; 
    if (!(inPersonalNumber == null || inPersonalNumber.trim().equals("") 
     || customerList == null || customerList.size() == 0)) { // inputs are valid to run this check 
     // check if personal number already exist 
     for (int i = 0; i < customerList.size(); i++) { 
      if (inPersonalNumber.equals(customerList.get(i).getCustomerPersonalNumber()) { // to avoid NPE, kept inPersonalNumber in check 
       retVal = true; 
       break; 
      } 
     } 
    } 
    return retVal;  
} 

Answer 6

When I remove the first return true and instead to the last return true, it don't get the error in my eclipse code, but why can't I have the first place and would this be the same?

如果您刪除第二個return語句代碼將能夠運行，並沒有返回值 - 這是不可能的，因爲你定義爲具有布爾返回類型的方法。所以它無論如何都必須始終返回一個值。

只要將第二個return語句更改爲false，應該做你想做的事。