プライベートメソッドを呼び出すことで複数のパブリックメソッドを短くしました

Question

私は、割引をチェックする複数のパブリックメソッドを持っています。コードを何度も繰り返さずにパブリックメソッドがアクセスできるプライベートメソッドのコードを大量に持ちます。ここにコードがあります。これは可能ですか？

public static boolean isE1(WebOrder order) 
    { 
    if(order.getPromos().stream().filter(i -> E1.contains(i.toUpperCase())).findFirst().isPresent()) 
    { 
    return true; 
    } 
    else 
    { 
    List<Discount> discounts = new LinkedList<>(); 

    order.getItems().forEach(i -> discounts.addAll(i.getDiscounts())); 

    if(!discounts.isEmpty()) 
    { 
     return discounts.stream() 
      .filter(i -> E1.contains(i.getCode().toUpperCase())) 
      .findFirst() 
      .isPresent(); 
    } 
    else 
    { 
     return false; 
    } 
    } 
} 
    public static boolean isE2(WebOrder order) 
    { 
    if(order.getPromos().stream().filter(i -> E2.contains(i.toUpperCase())).findFirst().isPresent()) 
    { 
    return true; 
    } 
    else 
    { 
    List<Discount> discounts = new LinkedList<>(); 

    order.getItems().forEach(i -> discounts.addAll(i.getDiscounts())); 

    if(!discounts.isEmpty()) 
    { 
     return discounts.stream() 
       .filter(i -> E2.contains(i.getCode().toUpperCase())) 
       .findFirst() 
       .isPresent(); 
    } 
    else 
    { 
     return false; 
    } 
    } 

}

Answer 1

はいそれは可能です：

だけプライベート静的でメソッドを起動します。

public static boolean isE2(WebOrder order) 
{ 
    yourMethod(order); 
} 

private static yourMethod(WebOrder order){ 
    //your code 

    if(order.getPromos().stream().filter(i -> E1.contains(i.toUpperCase())).findFirst().isPresent()) 
{ 
    return true; 
} 
else 
{ 
    List<Discount> discounts = new LinkedList<>(); 

order.getItems().forEach(i -> discounts.addAll(i.getDiscounts())); 

if(!discounts.isEmpty()) 
{ 
    return discounts.stream() 
     .filter(i -> E1.contains(i.getCode().toUpperCase())) 
     .findFirst() 
     .isPresent(); 
} 
else 
{ 
    return false; 
    } 
} 
}