Tìm nạp một giá trị mà không phải kiểm tra null trong Java

Nhiều lần tôi thấy mình không kiểm tra khi tìm nạp một giá trị từ một số phân cấp dữ liệu để tránh NullPulumExceptions, điều mà tôi thấy dễ bị lỗi và cần rất nhiều bản tóm tắt.

Tôi đã viết một thói quen rất đơn giản cho phép tôi bỏ qua việc kiểm tra null khi tìm nạp một đối tượng ...

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

Tôi sử dụng nó một chút như thế này ...

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

Điều trên dẫn đến việc tôi nhận được một đối tượng Room hoặc null, mà không phải kiểm tra null tất cả các cấp độ cha.

Bạn nghĩ gì về những điều trên? Tôi đang tạo ra một mô hình có vấn đề? Có cách nào tốt hơn để làm điều này theo ý kiến ​​của bạn?

Giải pháp của bạn rất thông minh. Vấn đề tôi thấy là thực tế là bạn không biết tại sao bạn lại bị null? Có phải vì nhà không có phòng? Có phải vì thị trấn không có nhà? Có phải vì đất nước không có thị trấn? Có phải vì có một nullvị trí 0 trong bộ sưu tập vì có lỗi ngay cả khi có nhà ở vị trí 1 trở lên?

Nếu bạn sử dụng NonPElớp mở rộng , bạn sẽ gặp vấn đề gỡ lỗi nghiêm trọng. Tôi nghĩ rằng tốt hơn là biết chính xác chuỗi bị hỏng ở đâu hơn là âm thầm nhận được một nulllỗi có thể ẩn một lỗi sâu hơn.

Ngoài ra, điều này vi phạm Luật Demeter : country.getTown().getHouses().get(0).getLivingRoom(). Thường xuyên hơn không, vi phạm một số nguyên tắc tốt khiến bạn phải thực hiện các giải pháp không chính thống để giải quyết vấn đề gây ra bởi vi phạm nguyên tắc đó.

Lời khuyên của tôi là bạn nên sử dụng nó một cách thận trọng và thử giải quyết lỗ hổng thiết kế khiến bạn phải gánh chịu sự cố chống tàu hỏa (vì vậy bạn không phải sử dụng NonPEở mọi nơi). Nếu không, bạn có thể có lỗi sẽ khó phát hiện.

Ý tưởng là tốt, thực sự tốt trong thực tế. Vì Java 8 các Optionalloại tồn tại, nên có thể tìm thấy giải thích chi tiết tại loại Tùy chọn Java . Một ví dụ với những gì bạn đã đăng là

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

Và hơn thế nữa.

Phương pháp của bạn hoạt động đủ tốt cho mục đích dự định của nó, mặc dù trả về nulls khi bạn nhận được NullPointerExceptionâm thanh như thiết kế xấu.

Cố gắng tránh nulls khi bạn có thể và chỉ vượt qua chúng khi chúng đại diện cho một cái gì đó hoặc có ý nghĩa đặc biệt và chỉ trả lại chúng khi chúng đại diện / có nghĩa là một cái gì đó - nếu không bạn nên ném một NullPointerException. Điều này tránh lỗi và nhầm lẫn. Nếu Objectkhông nên null, NullPointernên ném. Nếu một đối tượng có thể nullthì sẽ không có gì sai khi một đối tượng được truyền vào. Nếu không thì phương thức của bạn ở trên hoạt động.

Tôi có thể cảm thấy nỗi đau của bạn, nhưng giải pháp đề xuất là một ý tưởng tồi.

• Nếu một trong các getters ném NPE vì một số lý do khác, bạn sẽ bỏ qua nó.
• Có nguy cơ cho lambda bên trong đó phát triển thành mã khủng khiếp. Ví dụ, nếu có một yêu cầu mới để trả về một hằng số đặc biệt khi không có nhà trong thị trấn, một lập trình viên lười biếng có thể mở rộng lamda, bỏ lại mọi thứ NoNPE.get.
• Như đã đề cập, Optional.maplà những gì bạn đang tìm kiếm.
• Hình phạt của việc tạo một phiên bản mới của NullPulumException thường rất đáng kể. Đó là nhiều micro giây, đặc biệt là khi ngăn xếp cuộc gọi của bạn ngày càng lớn hơn. Thật khó để dự đoán nơi tiện ích của bạn sẽ được sử dụng.

Như một lưu ý phụ, NoNPEInterfacelà một bản sao của java.util.function.Supplier.

Trong một số trường hợp, bạn có thể cân nhắc sử dụng các tiện ích đánh giá biểu thức có trong nhiều khung (ví dụ: EL, SpEL):

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