การจัดการกับแอตทริบิวต์ว่างโดยใช้สตรีม Java 8 และการเรียงลำดับโดยใช้นิพจน์แลมบ์ดา

คำตอบทั้งหมดที่นี่เกี่ยวข้องกับ "กำจัดองค์ประกอบที่ไม่ดีออกไป ซึ่งมี getParentId() เป็นโมฆะ" นั่นอาจเป็นคำตอบหากพวกเขาแย่จริงๆ แต่มีอีกทางเลือกหนึ่ง: Comparators.nullsFirst (หรือสุดท้าย) ซึ่งช่วยให้คุณสามารถเปรียบเทียบสิ่งต่าง ๆ ที่ใช้ค่าว่างว่าน้อยกว่า (หรือมากกว่า) ค่าที่ไม่ใช่ค่าว่างทั้งหมด ดังนั้นคุณจึงไม่จำเป็นต้องโยนองค์ประกอบด้วย parentId เป็นโมฆะ ห่างออกไป.

Comparator<Entity> cmp = nullsLast(comparing(Entity::getParentId));
List<Entity> list = list.stream().sorted(cmp).collect(toList());

คุณสามารถทำสิ่งที่คล้ายกันในการกรอง กำหนดภาคแสดงของคุณเป็น:

Predicate<Entity> predicate = e -> e.getParentId() != null 
                                       && e.getParentId() < 100;

ดูเหมือนว่าคุณกำลังมองหาบางสิ่งเช่น:

list.sort(Comparator.comparing(Entity::getParent, 
                               Comparator.nullsLast(Integer::compareTo)));

องค์ประกอบทั้งหมดที่มีพาเรนต์ null จะถูกใส่ไว้ตอนท้าย และส่วนที่เหลือจะถูกจัดเรียงตามพาเรนต์

ลองกรองล่วงหน้าสำหรับค่า parentId ที่ไม่ใช่ค่าว่างเท่านั้น:

result = list.stream().filter(e -> e.getParentId() != null).filter(predicate).collect(Collectors.toList());

[แก้ไข] เพิ่งเห็นว่าแอตทริบิวต์ (e.parentId) ดูเหมือนเป็น null ในกรณีนั้น อย่างที่สอง การคัดแยกก็แตกหัก คุณกำลังเรียงลำดับรายการดั้งเดิม ไม่ใช่รายการที่ถูกกรอง ลอง result.sort(comp) แล้วคุณควรหลีกเลี่ยง NPE

คุณสามารถทำได้ทั้งหมดใน Stream ไปป์ไลน์เดียว:

List<Entity> result =  
    list.stream()
        .filter(e -> e.getParentId()!=null) // it's not clear if the null
                                            // is the Entity object itself
                                            // (in which case it should be e!=null)
                                            // or just the ParentId member
        .filter(predicate)
        .sorted(comp)
        .collect(Collectors.toList());

BTW ตามข้อความคำถามของคุณ ภาคแสดงดั้งเดิมควรเป็น:

Predicate<Entity> predicate = e -> e.getParentId() > 100; // not < 100

ควบคุมพลังของการอ้างอิงวิธีการเพื่อทำให้โค้ดมีขนาดกะทัดรัดยิ่งขึ้น:

   List<Entity> result =
            list.stream()
                    .filter(Objects::nonNull) 
                    .filter(predicate)
                    .sorted(comp)
                    .collect(Collectors.toList());

คุณทำได้:

Predicate<Entity> predicate = e -> e.getParentId() < 100;
Predicate<Entity> nonNull = e -> e != null;
result = list.stream().filter(nonNull)
                      .filter(predicate)
                      .collect(Collectors.toList());

แน่นอน คุณสามารถรวมภาคแสดงให้เป็นภาคแสดงเดียวได้ แต่ฉันแนะนำภาคแสดงที่สองเพื่อให้อ่านง่ายขึ้น นอกจากนี้ คุณยังเปลี่ยนภาคแสดง nonNull เพื่อตรวจสอบเอนทิตีทั้งหมดหรือเฉพาะแอตทริบิวต์ id ได้อย่างอิสระ (หรือทั้งสองอย่าง) แค่เล่นกับมันสักหน่อย :)