Re: [問題] Lambda語法

[JustinHere (良葛格)]

方向不太對，仔細看原本的 for 迴圈，其實主要是在 filter 出想要的 Order
，才從後續留下的 Order 取得細節，因此，比較好的寫法是：

orders.stream()
      .filter(order -> order.getOrdertails().getAmount() > 5 && order.getOrderDetail().getProduct() != null)
      .forEach(order -> {
          OrderDetail orderDetail = order.getOrdertails()
          String s = String.format("%s Buy %s * %d",
              order.getId(),
              orderDetail.getProduct().getName(),
              orderDetail.getAmount());
          System.out.println(s);
      });

關鍵在於，原本的 for 迴圈用了兩個 if，其實就是 && 兩個情況的意思 …

當然，如果想要更讀性更好一些，可以將 filter 與 forEach 中的邏輯提取至方法，
然後用方法參考：

orders.stream()
      .filter(OrderUtil::amountGTFiveAndHasProduct)
      .forEach(OrderUtil::printIdNameAmount);



public class OrderUtil {
    public static boolean amountGTFiveAndHasProduct(Order order) {
        return order.getOrdertails().getAmount() > 5 &&
               order.getOrderDetail().getProduct() != null;
    }

    public static void printIdNameAmount(Order order) {
        OrderDetail orderDetail = order.getOrdertails()
        String s = String.format("%s Buy %s * %d",
              order.getId(),
              orderDetail.getProduct().getName(),
              orderDetail.getAmount());
        System.out.println(s);
    }
}

※ 引述《kojilin (阿~~小達達)》之銘言：
: ※ 引述《Argosde ()》之銘言：
: : 最近在學Java8 Lambda語法
: : 自己在練習兩層for迴圈的重構
: : 遇到一個狀況，不知道是我使用觀念錯誤還是有不知道的方法
: : 兩層For迴圈版本
: : for(Order order : orders){
: :   for(OrderDetail orderDetail : order.getOrdertails()){
: :     if(orderDetail.getAmount()>5){
: :       Product product = orderDetail.getProduct();
: :       if(product!=null){
: :         String s = String.format("%s Buy %s * %d", order.getId(), product.getName(), orderDetail.getAmount());
: :         System.out.println(s);
: :       }
: :     }
: :   }
: : }
: : Lambda版本
: : orders.stream()
: :   .flatMap(order -> order.getOrdertails().stream())
: :   .filter(orderDetail -> ordertail.getAmount()>5)
: :   .map(orderDetail1 -> orderDetail1.getProduct())
: :   .forEach(product -> {
: :     ?
: :   });
: : 在?的地方沒辦法呼叫到上層資料，不知道是不是Lambda不適用這種狀況還是有其他方法
: : 第一次發問，請多指教...
: 這種情況下我就不一定會用 stream, 但你真要做可以像這樣，建一個容器
: class Pair<T1, T2>
: 然後
: orders.stream()
:   .flatMap(order -> order.getOrdertails().stream()
:        .map(detail -> new Pair<>(order, detail)))
:   .filter(orderDetail -> orderDetail.getT2().getAmount() > 5)
:   .map(orderDetail -> new Pair<>(orderDetail, orderDetail.getT2().getProduct()))
:   .forEach(product -> {
:   });
: 另一種就是醜一點，巢狀 stream，但程式碼就更難讀@@...
: orders.stream()
:         .forEach(order -> order.getOrdertails()
:             .stream()
:             .filter(orderDetail -> orderDetail.getAmount() > 5)
:             .forEach(orderDetail -> {
:             }));
: koji

--
良葛格學習筆記
http://openhome.cc

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 101.12.85.15
※ 文章網址: https://www.ptt.cc/bbs/java/M.1444785384.A.1BD.html
※ 編輯: JustinHere (101.12.85.15), 10/14/2015 09:21:21
※ 編輯: JustinHere (101.12.85.15), 10/14/2015 09:23:31
※ 編輯: JustinHere (101.12.85.15), 10/14/2015 09:24:22
※ 編輯: JustinHere (101.12.85.15), 10/14/2015 10:57:54
推 james732: 推良葛格
→ swpoker: 其實都lambda 當然函式也要匿名阿
推 Argosde: order 1-n* orderDetail 1-1* product
→ Argosde: filter會變成 getOrderdetails().stream().anyMatch(...)
→ Argosde: 這樣filter會篩選到orderDetails嗎?