ดึงค่าโดยไม่ต้องตรวจสอบเป็นโมฆะใน Java

หลายครั้งที่ฉันตรวจสอบตัวเองเป็นโมฆะเมื่อดึงค่าจากลำดับชั้นข้อมูลบางอย่างเพื่อหลีกเลี่ยง NullPointerExceptions ซึ่งฉันพบว่ามีแนวโน้มที่จะเกิดข้อผิดพลาดและต้องมีสำเร็จรูปจำนวนมาก

ฉันได้เขียนกิจวัตรที่ง่ายมากซึ่งทำให้ฉันสามารถข้ามการตรวจสอบค่าว่างเมื่อดึงวัตถุ ...

public final class NoNPE {

    public static <T> T get(NoNPEInterface<T> in) {
        try {
            return in.get();
        } catch (NullPointerException e) {
            return null;
        }
    }

    public interface NoNPEInterface<T> {
        T get();
    }
}

ฉันใช้มันแบบนี้ ...

Room room = NoNPE.get(() -> country.getTown().getHouses().get(0).getLivingRoom());

ด้านบนทำให้ฉันได้รับวัตถุห้องหรือโมฆะโดยไม่ต้องตรวจสอบระดับผู้ปกครองทั้งหมด

คุณคิดอย่างไรกับข้างต้น? ฉันกำลังสร้างรูปแบบที่มีปัญหาหรือไม่? มีวิธีที่ดีกว่านี้ในความคิดของคุณ?

ทางออกของคุณฉลาดมาก ปัญหาที่ฉันเห็นคือความจริงที่ว่าคุณไม่รู้ว่าทำไมคุณถึงได้null? เป็นเพราะบ้านไม่มีที่ว่างใช่ไหม มันเป็นเพราะเมืองไม่มีบ้านหรือไม่? เป็นเพราะประเทศไม่มีเมืองหรือ เป็นเพราะมีnull0 ในตำแหน่งของคอลเลกชันเนื่องจากข้อผิดพลาดแม้ว่าจะมีบ้านในตำแหน่ง 1 และมากกว่า?

ถ้าคุณใช้NonPEคลาสอย่างเต็มที่คุณจะมีปัญหาในการดีบักอย่างรุนแรง ฉันคิดว่าเป็นการดีกว่าที่จะทราบว่าโซ่ถูกทำลายไปกว่าการรับเสียงเงียบ ๆnullซึ่งอาจซ่อนข้อผิดพลาดได้ลึกกว่า

นอกจากนี้ละเมิดกฎหมายของ Demetercountry.getTown().getHouses().get(0).getLivingRoom() : บ่อยครั้งที่การละเมิดหลักการที่ดีบางอย่างทำให้คุณต้องใช้วิธีแก้ปัญหานอกรีตเพื่อแก้ไขปัญหาที่เกิดจากการละเมิดหลักการดังกล่าว

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

ความคิดนั้นดีจริง ๆ แล้วดีมาก ตั้งแต่ Java 8 Optionalประเภทที่มีอยู่คำอธิบายรายละเอียดสามารถพบได้ที่ประเภท Java ตัวเลือก ตัวอย่างของสิ่งที่คุณโพสต์คือ

Optional.ofNullable(country)
    .map(Country::getTown)
    .map(Town::Houses);

และเพิ่มเติมเกี่ยวกับ

วิธีการของคุณทำงานได้ดีพอสำหรับวัตถุประสงค์ที่ต้องการแม้ว่าจะส่งคืนnullเมื่อคุณได้รับNullPointerExceptionเสียงเช่นการออกแบบที่ไม่ดี

พยายามหลีกเลี่ยงnullเมื่อคุณสามารถและมีเพียงผ่านพวกเขาเมื่อพวกเขาเป็นตัวแทนของอะไรบางอย่างหรือมีความหมายพิเศษและมีเพียงพวกเขากลับมาเมื่อพวกเขาเป็นตัวแทน / สิ่งที่มีค่าเฉลี่ย - NullPointerExceptionมิฉะนั้นคุณควรโยน สิ่งนี้จะหลีกเลี่ยงข้อบกพร่องและความสับสน หากObjectไม่ควรจะnullเป็นNullPointerควรจะโยน ถ้าวัตถุnullนั้นไม่มีอะไรจะผิดพลาดเมื่อวัตถุถูกส่งเข้ามามิฉะนั้นวิธีการของคุณจะใช้ได้

ฉันรู้สึกถึงความเจ็บปวดของคุณ แต่วิธีแก้ปัญหาที่เสนอนั้นเป็นแนวคิดที่ไม่ดี

• หากหนึ่งใน getters โยน NPE ด้วยเหตุผลอื่นคุณจะไม่สนใจมัน
• แลมบ์ดานั้นมีความเสี่ยงที่จะเติบโตเป็นโค้ดที่น่ากลัว ตัวอย่างเช่นถ้ามีความต้องการที่จะกลับมาใหม่อย่างต่อเนื่องเป็นพิเศษเมื่อมีบ้านในเมืองไม่มีโปรแกรมเมอร์ขี้เกียจสามารถขยาย Lamda NoNPE.getทิ้งทุกอย่างที่อยู่ในห่อ
• ตามที่ระบุไว้แล้วOptional.mapคือสิ่งที่คุณกำลังมองหา
• บทลงโทษของการสร้างตัวอย่างใหม่ของ NullPointerException นั้นมีความสำคัญ เป็นไมโครวินาทีจำนวนมากโดยเฉพาะอย่างยิ่งเมื่อสแตกการโทรของคุณเริ่มใหญ่ขึ้น เป็นการยากที่จะคาดการณ์ว่ายูทิลิตี้ของคุณจะถูกใช้งานที่ไหน

ตามบันทึกข้าง, ซ้ำNoNPEInterfacejava.util.function.Supplier

ในบางกรณีคุณอาจพิจารณาใช้การประเมินผลนิพจน์ที่มีอยู่ในหลาย ๆ กรอบ (ตัวอย่าง: EL, SpEL):

evaluateProperty(country, "town.houses[0].livingRoom")