Java'da null denetimi yapmak zorunda kalmadan bir değer getiriliyor

Çoğu zaman kendimi nullPointerExceptions önlemek için bazı veri hiyerarşisinden bir değer getirilirken null kontrol, hatalara eğilimli ve bir sürü boilerplate ihtiyacı buluyorum.

Bir nesneyi getirirken boş denetimi atlamama izin veren çok basit bir rutin yazdım ...

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();
    }
}

Biraz böyle kullanıyorum ...

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

Yukarıdakiler, tüm ebeveyn seviyelerini kontrol etmek zorunda kalmadan bir Room nesnesi veya null elde etmeme neden oldu.

Yukarıdakiler hakkında ne düşünüyorsun? Sorunlu bir kalıp oluşturuyor muyum? Sizce bunu yapmanın daha iyi bir yolu var mı?

Çözümünüz çok akıllı. Gördüğüm Sorun şu ki sen var neden bilmiyorum gerçektir null? Evde oda olmadığı için miydi? Kasabanın evi olmadığı için mi? Ülkenin kasabası olmadığı için mi? Bir yoktu çünkü miydi nullpozisyonları 1'den itibaren evler vardır bile bir hata nedeniyle toplama 0 pozisyonunda?

NonPESınıfı yaygın olarak kullanırsanız, ciddi hata ayıklama sorunlarınız olacaktır. Zincirin tam olarak nerede kırıldığını bilmek nulldaha derin bir hatayı gizleyebilecek bir sessizlikten daha iyi olduğunu düşünüyorum.

Ayrıca, bu ihlal Demeter'in Kanunu : country.getTown().getHouses().get(0).getLivingRoom(). Çoğu zaman, bazı iyi prensipleri ihlal etmek, bu prensibi ihlal etmenin neden olduğu sorunu çözmek için alışılmadık çözümler uygulamanıza neden olur.

Benim tavsiyem, dikkatli bir şekilde kullanmanız ve tren kazası antipatternerinde (yani NonPEher yerde kullanmak zorunda kalmamanız) gereken tasarım kusurunu çözmeye çalışmanızdır . Aksi takdirde tespit edilmesi zor olan hatalarınız olabilir.

Fikir iyi, aslında gerçekten iyi. Java 8 Optionaltürleri bulunduğundan, Java İsteğe bağlı türünde ayrıntılı bir açıklama bulunabilir . Gönderdiklerinize bir örnek

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

Ve dahası.

Metodunuz amaçlanan amaç için yeterince iyi çalışıyor, ancak kötü tasarım gibi nullbir NullPointerExceptionses aldığınızda geri dönüyor .

nullNe zaman yapabileceğinizden kaçınmaya çalışın ve sadece bir şeyi temsil ettiklerinde veya özel bir anlamı olduğunda onları geçirin ve sadece bir şeyi temsil ettikleri / ifade ettikleri zaman iade edin - aksi takdirde a atmalısınız NullPointerException. Bu, böcek ve karışıklığı önler. Bir Objectolmaması gerekiyorsa nulla NullPointeratılmalıdır. Bir nesne olabiliyorsa null, bir şey iletildiğinde hiçbir şey yanlış gitmez. Aksi takdirde yukarıdaki yöntem işe yarar.

Acınızı hissedebiliyorum, ancak önerilen çözüm kötü bir fikir.

• Eğer alıcılardan biri başka bir nedenden dolayı bir NPE fırlatırsa, bunu görmezden gelirsiniz.
• Bu iç lambda'nın korkunç bir koda dönüşmesi riski var. Örneğin, kasabada ev olmadığında özel bir sabiti iade etmek için yeni bir gereksinim varsa, tembel bir programcı lamda'yı genişletebilir ve her şeyi sarılmış olarak bırakabilir NoNPE.get.
• Daha önce de belirtildiği gibi, Optional.maparadığınız şey budur.
• Yeni bir NullPointerException örneği oluşturmanın cezası genellikle önemlidir. Bu, özellikle çağrı yığınınız büyüdükçe birçok mikrosaniye. Yardımcı programınızın nerede kullanılacağını tahmin etmek zor.

Yan not olarak, NoNPEInterfacebir kopyasıdır java.util.function.Supplier.

Bazı durumlarda, birçok çerçevede mevcut olan bir ifade değerlendirme aracı kullanmayı düşünebilirsiniz (örneğin: EL, SpEL):

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