ไม่มีการตรวจสอบในการปรับปรุงสำหรับวง

วิธีที่ดีที่สุดในการป้องกัน null ใน for for loop ใน Java คืออะไร

ดูเหมือนว่าน่าเกลียด:

if (someList != null) {
    for (Object object : someList) {
        // do whatever
    }
}

หรือ

if (someList == null) {
    return; // Or throw ex
}
for (Object object : someList) {
    // do whatever
}

อาจไม่มีวิธีอื่นใด พวกเขาควรจะวางมันลงในforโครงสร้างของตัวเองหรือไม่ถ้ามันเป็นโมฆะอย่าเรียกใช้ลูป?

คุณควรตรวจสอบว่าคุณได้รับรายชื่อจากที่ใด

รายการที่ว่างเปล่าเป็นสิ่งที่คุณต้องการเพราะรายการที่ว่างเปล่าจะไม่ล้มเหลว

หากคุณได้รับรายชื่อนี้จากที่อื่นและไม่ทราบว่าใช้ได้หรือไม่คุณสามารถสร้างวิธีการอรรถประโยชน์และใช้มันได้ดังนี้:

for( Object o : safe( list ) ) {
   // do whatever 
 }

และแน่นอนsafeจะเป็น:

public static List safe( List other ) {
    return other == null ? Collections.EMPTY_LIST : other;
}

คุณสามารถเขียนเมธอดตัวช่วยซึ่งส่งคืนลำดับว่างถ้าคุณผ่านเป็นโมฆะ:

public static <T> Iterable<T> emptyIfNull(Iterable<T> iterable) {
    return iterable == null ? Collections.<T>emptyList() : iterable;
}

จากนั้นใช้:

for (Object object : emptyIfNull(someList)) {
}

ฉันไม่คิดว่าฉันจะทำอย่างนั้นจริง ๆ - ฉันมักจะใช้แบบฟอร์มที่สองของคุณ โดยเฉพาะอย่างยิ่ง "หรือการโยนทิ้ง" เป็นสิ่งสำคัญ - หากไม่ควรเป็นโมฆะจริงๆคุณควรโยนข้อยกเว้น คุณรู้ว่ามีบางอย่างผิดปกติ แต่คุณไม่ทราบขอบเขตความเสียหาย ยกเลิกก่อน

ตอนนี้เป็นปี 2560 แล้วและตอนนี้คุณสามารถใช้ได้ Apache Commons Collections4 ได้แล้ว

การใช้งาน:

for(Object obj : ListUtils.emptyIfNull(list1)){
    // Do your stuff
}

คุณสามารถทำเช็ค null ปลอดภัยเดียวกันกับชั้นเรียนอื่น ๆ CollectionUtils.emptyIfNullที่มีการเก็บ

ด้วย Java 8 Optional:

for (Object object : Optional.ofNullable(someList).orElse(Collections.emptyList())) {
    // do whatever
}

ใช้ArrayUtils.nullToEmptyจากcommons-langห้องสมุดสำหรับอาร์เรย์

for( Object o : ArrayUtils.nullToEmpty(list) ) {
   // do whatever 
}

ฟังก์ชันนี้มีอยู่ในcommons-langไลบรารีซึ่งรวมอยู่ในโครงการ Java ส่วนใหญ่

// ArrayUtils.nullToEmpty source code 
public static Object[] nullToEmpty(final Object[] array) {
    if (isEmpty(array)) {
        return EMPTY_OBJECT_ARRAY;
    }
    return array;
}

// ArrayUtils.isEmpty source code
public static boolean isEmpty(final Object[] array) {
    return array == null || array.length == 0;
}

นี่คือคำตอบเดียวกับที่ @OscarRyz แต่เพื่อประโยชน์ของDRYมนต์ฉันเชื่อว่ามันเป็นที่น่าสังเกต ดูหน้าโครงการคอมมอนส์ lang นี่คือเอกสารnullToEmpty API และแหล่งที่มา

รายการ Maven ที่จะรวมcommons-langในโครงการของคุณหากยังไม่ได้ดำเนินการ

<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-lang3</artifactId>
    <version>3.4</version>
</dependency>

น่าเสียดายที่commons-langไม่มีฟังก์ชั่นนี้สำหรับListประเภท ในกรณีนี้คุณจะต้องใช้วิธีการช่วยเหลือตามที่ระบุไว้ก่อนหน้านี้

public static <E> List<E> nullToEmpty(List<E> list)
{
    if(list == null || list.isEmpty())
    {
        return Collections.emptyList();
    }
    return list;
}

หากคุณได้รับสิ่งนั้นListจากการเรียกเมธอดที่คุณนำมาใช้อย่าส่งnullคืนค่าว่างListกลับมาที่ว่างเปล่า

หากคุณไม่สามารถเปลี่ยนการใช้งานได้แสดงว่าคุณติดอยู่กับการnullตรวจสอบ หากไม่เป็นnullเช่นนั้นก็ให้ทำการยกเว้น

ฉันจะไม่ไปหาผู้ช่วยที่คืนค่ารายการว่างเปล่าเพราะมันอาจมีประโยชน์บางครั้ง แต่คุณจะคุ้นเคยกับการเรียกมันในทุก ๆ วงที่คุณอาจซ่อนข้อผิดพลาดบางอย่าง

ฉันได้แก้ไขคำตอบข้างต้นดังนั้นคุณไม่จำเป็นต้องส่งจาก Object

public static <T> List<T> safeClient( List<T> other ) {
            return other == null ? Collections.EMPTY_LIST : other;
}

แล้วก็เรียกรายการโดย

for (MyOwnObject ownObject : safeClient(someList)) {
    // do whatever
}

คำอธิบาย: MyOwnObject: ถ้าเป็นเช่นList<Integer>นั้น MyOwnObject จะเป็นจำนวนเต็มในกรณีนี้

วิธีการได้อย่างมีประสิทธิภาพป้องกัน null ในห่วงอีกประการหนึ่งคือการตัดคอลเลกชันของคุณกับ Google ฝรั่งของเช่นนี้หนึ่งความหวังทำให้เป็นไปได้ของคอลเลกชันที่ว่างเปล่าได้อย่างมีประสิทธิภาพที่ชัดเจนตั้งแต่ลูกค้าจะคาดว่าจะตรวจสอบว่าเป็นของสะสมที่มีอยู่ด้วยOptional<T>Optional.isPresent()

สำหรับทุกคนที่ไม่สนใจในการเขียนวิธีการเพื่อความปลอดภัยคง null ของตัวเองคุณสามารถใช้: org.apache.commons.lang.ObjectUtils.defaultIfNull(Object, Object)คอมมอน-lang ตัวอย่างเช่น:

    for (final String item : 
    (List<String>)ObjectUtils.defaultIfNull(items, Collections.emptyList())) { ... }

ObjectUtils.defaultIfNull JavaDoc

ใช้CollectionUtils.isEmpty(Collection coll)วิธีการตรวจสอบว่าเป็นโมฆะปลอดภัยถ้าคอลเลกชันที่ระบุว่างเปล่า

สำหรับสิ่งนี้ import org.apache.commons.collections.CollectionUtilsสำหรับเรื่องนี้

Maven พึ่งพา

<dependency>
    <groupId>org.apache.commons</groupId>
    <artifactId>commons-collections4</artifactId>
    <version>4.0</version>
</dependency>

for (Object object : someList) {

   // do whatever
}  throws the null pointer exception.