Java 8 - ループ内にネストされている場合

Question

リストがあり、リスト内の各項目について、各リスト要素に基づいて「何か」する必要があります。

リストはコードで構成され、合計5つのコードがあります。リストには、任意のコードまたはすべてのコードを含めることができます。

これまでのところ私はforEachのを使用しましたし、以下のように私はそれの内側であれば条件を書いた - 私はこれをリファクタリングで探しています

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

。誰かがこれをいかにうまくいくか見てみることができますか？私は改善するだろう

Answer 1

、複数行のラムダを使用し、（スイッチで）新しいメソッドを作成しないでください：

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

次に、あなたのラムダでそれを使用します。

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

もう一つの方法は、静的を作成することですマップキー（これはCategory.Xになります）と値（使用準備ができている機能）

Answer 2

唯一のことは、スイッチ・ステートメントを使用することです：

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

カテゴリーが列挙型である場合にのみ動作しますluk2302によってmentionend通り。まず

Answer 3

CategoryクラスにdoSomethingメソッドを追加し、単に.forEachで呼び出すことができます。例えば

：

public class Category { 

    // any other methods/variable/constructors 

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

次に、あなたがこのようにそれを呼び出すことができます。// do somethingが共通の振る舞いをしていない

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

場合は、doSomethingメソッド内であれば一部を移動することができます。