Java：如何減少無法合併它們的if-else語句的數量

Question

儘管我的代碼能夠按照我的要求完美工作，但是在我的代碼中有很多if-else語句，所以我想問問是否有可能減少在此代碼if-else語句

if (nightPost(timePicker1.getValue().toString(), timePicker4.getValue().toString())) { 
    if (!teToTime.isAfter(LocalTime.parse("08:30",format))) { 
     textField4.setStyle(""); 
     if(getDifference(timeKA, timeKB)<121 && getDifference(timeKA, timeKB)>59) { 
      timePicker3.setStyle(""); 
      if (range(timePicker1.getValue().toString(), timePicker2.getValue().toString(), 
       timePicker3.getValue().toString(), timePicker4.getValue().toString())) { 
       timePicker2.setStyle(""); 
       timePicker3.setStyle(""); 
       if(increaseTxt.isBefore(LocalTime.parse("11:01",format))) 
       { 
        textField5.setStyle(""); 
        return true; 
       } else { 
        textField5.setStyle("-fx-border-color: red"); 
        return false; 
       } 
      } else { 
       timePicker2.setStyle("-fx-border-color: orange"); 
       return false; 
      }        
     } 
     else { 
      timePicker3.setStyle("-fx-border-color: red"); 
      return false; 
     } 
    } else { 
     textField4.setStyle("-fx-border-color: red"); 
     return false; 
    } 
} 
return false; 

Answer 1

替換嵌套條件與 Guard Clauses

if (!nightPost(timePicker1.getValue().toString(), timePicker4.getValue().toString())) { 
    return false; 
} 
if (teToTime.isAfter(LocalTime.parse("08:30",format))) { 
    textField4.setStyle("-fx-border-color: red"); 
    return false; 
} 
textField4.setStyle(""); 
if (!(getDifference(timeKA, timeKB) < 121 && getDifference(timeKA, timeKB) > 59)) { 
    timePicker3.setStyle("-fx-border-color: red"); 
    return false; 
} 
timePicker3.setStyle(""); 
if (!range(timePicker1.getValue().toString(), timePicker2.getValue().toString(), 
      timePicker3.getValue().toString(), timePicker4.getValue().toString())) 
{ 
    timePicker2.setStyle("-fx-border-color: orange"); 
    return false; 
} 
timePicker2.setStyle(""); 
timePicker3.setStyle(""); 
if (!increaseTxt.isBefore(LocalTime.parse("11:01",format))) { 
    textField5.setStyle("-fx-border-color: red"); 
    return false; 
} 
textField5.setStyle(""); 
return true; 

Answer 2

我並不完全相信你的代碼做什麼，但之後快速查看（和@matoni已經證明），你實際上可以刪除很多你的'別的因爲你在大多數'真實'的情況下返回。這是否使得它更具可讀性是主觀的。

也@Matoni顯示，你可以反轉很多你的if語句！操作員，這樣可以真正減少嵌套。

編輯：看看我的回答和我現在回家的代碼。道歉@AxelH，似乎我誤解了matoni的答案。這裏有一個更新的答案：

我已經倒過去了，如前所述刪除了Elses，但我也將一些代碼移出到它自己的私有方法中，以獲得更多的可讀性，因爲我假設這就是爲什麼你想要重構。

public bool yourMethod(your params){ 

    if (!nightPost(timePicker1.getValue().toString(), timePicker4.getValue().toString())) return false; 

    if (teToTime.isAfter(LocalTime.parse("08:30",format ||))) return redBorder(); 
    textField4.setStyle(""); 

    if(!(getDifference(timeKA, timeKB)<121 && getDifference(timeKA, timeKB)>59)) return redTimeBorder(); 
    timePicker3.setStyle(""); 

    // assuming range returns a boolean value 
    if (!range(timePicker1.getValue().toString(), timePicker2.getValue().toString(), 
    timePicker3.getValue().toString(), timePicker4.getValue().toString())) return orangeTime2Border(); 

    timePicker2.setStyle(""); 
    timePicker3.setStyle(""); 

    if(!increaseTxt.isBefore(LocalTime.parse("11:01",format))) return redText5Border(); 

    textField5.setStyle(""); 
    return true; 
}        

private boolean redText4Border(){ 
    textField4.setStyle("-fx-border-color: red"); 
    return false; 
} 

private boolean redTime3Border(){ 
    timePicker3.setStyle("-fx-border-color: red"); 
    return false; 
} 

private boolean orangeTime2Border(){ 
    timePicker2.setStyle("-fx-border-color: orange"); 
    return false; 
} 

private boolean redText5Border(){ 
    textField5.setStyle("-fx-border-color: red"); 
    return false; 
}