Java 8 - Verschachtelt wenn innerhalb einer Schleife

Question

Also habe ich eine Liste und für jedes Element in der Liste muss ich "etwas tun" basierend auf jedem Listenelement.

Die Liste besteht aus Codes und es gibt insgesamt 5 Codes. Die Liste kann einige oder alle Codes enthalten.

Bisher habe ich die forEach verwendet und ich habe das, wenn die Bedingungen im Inneren wie unten geschrieben -

List<Category> categories = getCategories(); 

categories.stream().forEach(category -> { 
      if(category.equals(Category.A)) { 
       // do something 
      } else if(category.equals(Category.B)) { 
       // do something 
      } else if(category.equals(Category.C)) { 
       // do something 
      } else if(category.equals(Category.D)) { 
       // do something 
      } else if(category.equals(Category.E)) { 
       // do something 
      } 
     }); 

Ich bin bei Refactoring diese suchen. Kann jemand bitte schauen, wie das besser gemacht werden kann?

Answer 1

Zunächst nicht mehrzeiligen lambdas verwenden und neue Methode (mit Schalter) erstellen:

private void doSomethingBasedOnCategory(Category category) { 
    switch(category) { 
     case A: // do something 
       break; 
     case B: // do something 
       break; 
     case C: // do something 
       break; 
     case D: // do something 
       break; 
     case E: // do something 
       break; 
    } 
} 

Dann verwenden Sie es in Ihrem Lambda:

getCategories() 
    .stream() 
    .forEach(category -> doSomethingBasedOnCategory(category); 

Eine andere Möglichkeit ist statisch zu erstellen Karte vorgefüllt mit Schlüsseln (die Category.X sein werden) und Werten (die Funktionen bereit sein werden zu verwenden)

Answer 2

Das einzige, was ich verbessern würde, ist eine switch-Anweisung zu verwenden:

switch(category){  
    case Category.A: 
     // do Something 
    break; 
} 

Wie mentionend von luk2302 dies nur funktioniert, wenn Kategorie eine Enumeration ist.

Answer 3

Sie können der Category Klasse eine doSomething Klasse hinzufügen und sie einfach in der .forEach aufrufen.

Zum Beispiel:

public class Category { 

    // any other methods/variable/constructors 

    public void doSomething() { 
     //do something 
    } 
} 

Dann können Sie es so nennen:

categories.stream().forEach(Category::doSomething); 

Wenn die // do something kein gemeinsames Verhalten hat, können Sie das, wenn ein Teil im Inneren des doSomething Methode bewegen.