重構if語句中的語句

Question

我需要一些幫助來重構這個if語句。我想過把百分比聲明爲常量。我也想做一個包含if方括號內的代碼的方法。我還可以做些什麼？

if(totalReceiptsAmount >= getIncome() && totalReceiptsAmount < 0.20 * getIncome()) 
     setTaxIncrease(getBasicTax() + 0.05 * getBasicTax()); 
    if(totalReceiptsAmount >= 0.20 * getIncome() && totalReceiptsAmount < 0.40 * getIncome()) 
     setTaxIncrease(getBasicTax() - 0.05 * getBasicTax()); 
    if(totalReceiptsAmount >= 0.40 * getIncome() && totalReceiptsAmount < 0.60 * getIncome()) 
     setTaxIncrease(getBasicTax() - 0.10 * getBasicTax()); 
    if(totalReceiptsAmount >= 0.60 * getIncome()) 
     setTaxIncrease(getBasicTax() - 0.15 * getBasicTax()); 

Answer 1

在你的代碼的主要問題是可能的代碼的複製，這意味着如果你想例如，改變的情況下，你可能在所有四個條件採用同樣的變化。因此，您可以嘗試將常用功能分解出來，就像您對條件已經提出的那樣。所以，你可以定義一個方法

private boolean receiptsAmountIsBetweenFactorOfXAndYOfIncome(double x, double y){ 
    return totalReceiptsAmount >= x * getIncome() && totalReceiptsAmount < y * getIncome(); 
} 

，並相應地更新您的if語句：

if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0, 0.2)) 
    setTaxIncrease(getBasicTax() + 0.05 * getBasicTax()); 
if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0.2, 0.4)) 
    setTaxIncrease(getBasicTax() - 0.05 * getBasicTax()); 
if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0.4, 0.6)) 
    setTaxIncrease(getBasicTax() - 0.10 * getBasicTax()); 
if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0.6, 1)) 
    setTaxIncrease(getBasicTax() - 0.15 * getBasicTax()); 

---

現在，仍然有重複的if語句的主體。再次

private void increaseTaxByFactorOfX(double x){ 
    setTaxIncrease(getBasicTax() + x * getBasicTax()); 
} 

和更新的if語句：所以，你可以引入其他方法

if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0, 0.2)) 
    increaseTaxByFactorOfX(0.05); 
if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0.2, 0.4)) 
    increaseTaxByFactorOfX(-0.05); 
if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0.4, 0.6)) 
    increaseTaxByFactorOfX(-0.10); 
if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(0.6, 1)) 
    increaseTaxByFactorOfX(-0.15); 

---

現在，如果你願意，你可以在使用numericals檢測模式，或只需硬編碼數組或列表中的數字，並使用循環代替多個類似的if語句：

double[] factorOfIncome = {0, 0.2, 0.4, 0.6, 1}; 
double[] taxIncreaseFactor = {0.05, -0.05, -0.10, -0.15}; 

for(int i = 0; i<taxIncreaseFactor.length; i++) 
    if(receiptsAmountIsBetweenFactorOfXAndYOfIncome(factorOfIncome[i], factorOfIncome[i+1])) 
     increaseTaxByFactorOfX(taxIncreaseFactor[i]); 

This la st重構步驟完全擺脫了重複，但在我看來使代碼有點不太可以理解。

編輯：注意，我認爲第一個條件應該是

if(totalReceiptsAmount >= 0 * getIncome() && //... 

，因爲它確實看起來這是你打算寫什麼。如果情況並非如此，那麼第一個條件將需要分開處理。