特定の値を持つ次のアイテムが指定されたリストのアイテムの値を設定します。

Question

私には、非アクティブとアクティブの両方のSalesOrdersのリストがあります。このリスト全体をループし、各SalesOrderがアクティブであった時間を設定したいと思います。以下のように（リストはすでにのStartTimeが発注される）

私のループは次のとおりです。

for(int i = 0; i < salesOrders.size(); i++) { 
    if(salesOrders.get(i).getStatus() == RecordStatus.ACTIVE) { 
     ELDDutyLog activeSalesOrder = salesOrders.get(i); 

     for(int j = i + 1; j < salesOrders.size(); j++) { 
      if(salesOrders.get(j).getStatus() == RecordStatus.ACTIVE) { 
       ELDDutyLog nextActiveSalesOrder = salesOrders.get(j); 

       salesOrders.get(i).setActiveDurationMinutes(
        Minutes.minutesBetween(
         activeSalesOrder.getStartTime(), 
         nextActiveSalesOrder.getStartTime() 
        ).getMinutes() 
       ); 
       break; 
      } 
     } 
    } 
} 
return salesOrders; 

このアプローチは動作しますが、私は多分Javaの8を使用することにより、これを行うのより良い方法がなければならないと感じ？誰も私の周りの任意の方向を提供することはできますか？

Answer 1

無効な受注をスキップするたびに、リスト全体を再度ループする必要はありません。代わりに、アクティブな受注を一度フィルタリングして、次にアクティブな受注がフィルタリングされた一覧の次の注文であることを常に知ることができます。このような

何か（テストしていません）：

List<ELDDutyLog> activeSalesOrders = salesOrders.stream() 
     .filter(so -> so.getStatus() == RecordStatus.ACTIVE) 
     .collect(Collectors.toList()); 

for (int i = 0; i < activeSalesOrders.size() - 1; i++) { 
    activeSalesOrders.get(i).setActiveDurationMinutes(
        Minutes.minutesBetween(
         activeSalesOrders.get(i).getStartTime(), 
         activeSalesOrders.get(i+1).getStartTime() 
        ).getMinutes() 
       ); 
} 

Answer 2

私は私が正しく理解してほしいです。 （テストしていない）試してみてください。

salesOrders.stream() 
    .filter(salesOrder -> salesOrder.getStatus() == RecordStatus.ACTIVE) 
    .reduce((current, next) -> { 
    current.setActiveDurationMinutes(
     Minutes.minutesBetween(
     current.getStartTime(), 
     next.getStartTime() 
    ).getMinutes() 
    ); 
    return next; 
    }); 

正直に言うと...それはreduceのための奇妙な使用のビットですが、私はそれがうまくいくと思います。

Answer 3

2つのループをネストする必要はありません。 setActiveDurationMinutesコールの引数を計算するために検索したnextActiveSalesOrderは、外側ループの次の繰り返しで検索するactiveSalesOrderとなる同じオブジェクトです。それ以外にも、activeSalesOrderにその値が格納されているにもかかわらず、内部操作でsalesOrders.get(i)を不必要に呼び出しています。そのほかに

ELDDutyLog activeSalesOrder = null; 
for(ELDDutyLog nextActiveSalesOrder: salesOrders) { 
    if(nextActiveSalesOrder.getStatus() != RecordStatus.ACTIVE) continue; 
    if(activeSalesOrder != null) 
     activeSalesOrder.setActiveDurationMinutes(
      Minutes.minutesBetween(
       activeSalesOrder.getStartTime(), 
       nextActiveSalesOrder.getStartTime() 
      ).getMinutes() 
     ); 
    activeSalesOrder = nextActiveSalesOrder; 
} 

... APIの種類Minutes.minutesBetween(a, b).getMinutes()ような何かを書くためにあなたが必要ですか！ APIが何回多くの時間あなたに分がほしいと言われる必要がありますか？
がjava.timeと比較：

1. ChronoUnit.MINUTES.between(a, b)は（DITO、まだ冗長性）