Java: Wie reduziere ich die Anzahl der if-else-Anweisungen, wo sie nicht zusammengeführt werden können

Question

Obwohl mein Code funktioniert perfekt wie ich will, aber es gibt viele if-else-Anweisungen in meinem Code, also möchte ich fragen, ist es möglich if-else-Anweisungen in diesem Code

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

ersetzen Nested Conditional mit 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

ich bin nicht ganz sicher zu reduzieren, was Ihr Code tut, aber nach einem kurzen Blick über (und als @matoni hat gezeigt), du könntest tatsächlich viel von deinem 'sonst' entfernen s, da du in den meisten "echten" Fällen zurückkehrst. Ob das etwas lesbarer ist, ist subjektiv.

Auch wie @Matoni gezeigt hat, können Sie viele Ihrer If-Anweisungen mit invertieren! auf diese Weise können Sie die Verschachtelung wirklich reduzieren.

Edit: Hatte einen Blick über meine Antwort und den Code jetzt, dass ich wieder zu Hause bin. Entschuldigung @AxelH, es scheint, als hätte ich Matonis Antwort missverstanden. Hier ist eine aktualisierte Antwort:

Ich habe die If invertiert und entfernt Elses wie bereits erwähnt, aber ich habe auch einige der Code in seine eigenen privaten Methoden für eine bessere Lesbarkeit verschoben, da ich davon ausgehen, warum Sie wollten umgestalten.

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; 
}