Mengambil nilai tanpa harus membatalkan centang di Java

Banyak kali saya menemukan diri saya nol memeriksa ketika mengambil nilai dari beberapa hierarki data untuk menghindari NullPointerExceptions, yang saya temukan cenderung rentan terhadap kesalahan dan membutuhkan banyak boilerplate.

Saya telah menulis rutin yang sangat sederhana yang memungkinkan saya melewatkan pemeriksaan nol saat mengambil objek ...

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

Saya menggunakannya agak seperti ini ...

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

Di atas mengakibatkan saya mendapatkan objek Kamar atau nol, tanpa harus memeriksa semua level orangtua nol.

Apa pendapat Anda tentang hal di atas? Apakah saya membuat pola yang bermasalah? Apakah ada cara yang lebih baik untuk melakukan ini menurut Anda?

Solusi Anda sangat cerdas. Masalah yang saya lihat adalah kenyataan bahwa Anda tidak tahu mengapa Anda mendapatkannya null? Apakah itu karena rumah itu tidak memiliki kamar? Apakah karena kota itu tidak memiliki rumah? Apakah itu karena negara itu tidak memiliki kota? Apakah karena ada nulldi posisi 0 koleksi karena kesalahan bahkan ketika ada rumah di posisi 1 dan lebih besar?

Jika Anda menggunakan NonPEkelas secara ekstensif , Anda akan memiliki masalah debugging yang serius. Saya pikir lebih baik untuk mengetahui di mana tepatnya rantai itu rusak daripada diam-diam mendapatkan nullyang bisa menyembunyikan kesalahan yang lebih dalam.

Ini juga melanggar Hukum Demeter :country.getTown().getHouses().get(0).getLivingRoom() . Lebih sering daripada tidak, melanggar beberapa prinsip yang baik membuat Anda harus menerapkan solusi yang tidak lazim untuk memecahkan masalah yang disebabkan oleh melanggar prinsip tersebut.

Rekomendasi saya adalah bahwa Anda menggunakannya dengan hati-hati dan mencoba menyelesaikan cacat desain yang membuat Anda harus mengeluarkan antipattern kereta (jadi Anda tidak harus menggunakan di NonPEmana-mana). Kalau tidak, Anda mungkin memiliki bug yang akan sulit dideteksi.

Idenya baik-baik saja, sebenarnya sangat bagus. Karena Java 8 Optionaljenis ada, penjelasan rinci dapat ditemukan di tipe Java Opsional . Contoh dengan apa yang Anda poskan

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

Dan selanjutnya.

Metode Anda bekerja cukup baik untuk tujuan yang dimaksudkan, meskipun kembali nullketika Anda mendapatkan NullPointerExceptionsuara seperti desain yang buruk.

Coba hindari nullketika Anda bisa dan hanya melewati mereka ketika mereka mewakili sesuatu atau memiliki makna khusus dan hanya mengembalikan mereka ketika mereka mewakili / berarti sesuatu - jika tidak, Anda harus melempar NullPointerException. Ini menghindari bug dan kebingungan. Jika tidak Objectseharusnya null, NullPointerharus dilempar. Jika suatu objek bisa jadi nulltidak ada yang salah ketika dilewatkan. Jika tidak, metode Anda di atas berfungsi.

Saya bisa merasakan sakit Anda, tetapi solusi yang diusulkan adalah ide yang buruk.

• Jika salah satu getter melempar NPE karena alasan lain, Anda akan mengabaikannya.
• Ada risiko lambda dalam itu tumbuh menjadi kode yang mengerikan. Misalnya, jika ada persyaratan baru untuk mengembalikan konstanta khusus ketika tidak ada rumah di kota, programmer yang malas dapat memperpanjang lamda, meninggalkan semuanya terbungkus NoNPE.get.
• Seperti yang sudah disebutkan, Optional.mapadalah apa yang Anda cari.
• Hukuman untuk membuat instance baru NullPointerException seringkali signifikan. Ini banyak mikrodetik, terutama karena tumpukan panggilan Anda semakin besar. Sulit untuk memprediksi di mana utilitas Anda akan digunakan.

Sebagai catatan, NoNPEInterfaceadalah duplikat dari java.util.function.Supplier.

Dalam beberapa kasus, Anda dapat mempertimbangkan menggunakan utils evaluasi ekspresi yang ada di banyak kerangka kerja (misalnya: EL, SpEL):

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