检查方法链中的最后一个 getter 是否不为空

Question

在代码中我们有很多链式方法，例如obj.getA().getB().getC().getD()。我想创建帮助类来检查方法getD() 是否不为空，但在此之前我需要检查所有以前的getter。我可以这样做：

try {
    obj.getA().getB().getC().getD();
}
catch (NullPointerException e) {
    // some getter is null
}

或（这是“傻”）

if (obj!null && obj.getA()!=null && obj.getA().getB()!=null && ...) {
    obj.getA().getB().getC().getD();
}
else {
    // some getter is null
}

我不想每次在我的代码中使用try{} catch() 来检查它。为此目的的最佳解决方案是什么？

我认为最好的是：

1. obj.getA().getB().getC().getD().isNull() - 为此我需要更改我所有的 getter，例如实现一些包含 isNull() 方法的接口。
2. NullObjectHelper.isNull(obj.getA().getB().getC().getD()); - 这将是最好的（我认为是这样）但是如何实现呢？

Answer 1

您可以使用Option 模式获得所需的结果。这会强制您更改方法签名，但基本上如果您的方法返回某个类型 T，它保证它有一些非空值，如果它返回Option<T>，则意味着它具有值 T 或 null。

Java 7 有一些称为空安全的特性，但它已从最终版本中删除。你可以这样做：

obj?.getA()?.getB()?.getC()?.getD()

此外，Java 8 将添加一个名为 Optional 的功能，因此您可以安全地进行操作。

事实上，如果你现在真的想使用它，试试Null Object 模式。这意味着您可以返回某种不会触发 NullPointerException 的 default 值，而不是返回普通的 null。不过，您需要对 getter 进行一些更改

class Object {
   A getA() {
     // ...
     return a == null ? A.NULL : a;
   }
}

class A {
   static A NULL = new A(); // some default behaviour
   B getB() {
     if (this == NULL) return B.NULL;
     // ...
     return b == null ? B.NULL : b;
   }
}

编辑：如果您希望实用程序执行此操作，您可以将其包装在某个功能接口中，然后调用它。

static boolean isNullResult(Callable call) throws Exception {
    try {
        return call.call() == null;
    } catch (NullPointerException npe) {
        return true;
    }
}

用法如下：

isNullResult(new Callable<Integer>() {
    @Override
    public Integer call() throws Exception {
        return new A().getB().getC().getInt();
    }
});

它不需要您更改现有功能

Answer 2

如前所述，真正的解决方案是重构。

与此同时，您可以将您的第一个解决方法封装在一个函数中：

static D getD(MyClass obj) {

    try {
        return obj.getA().getB().getC().getD();
    }
    catch (NullPointerException e) {
        return null; // Or even better, some default D
    }
}

在调用方站点：

D d = getD(obj);

至少您不必使用 try-catch 块来丢弃调用者。你仍然需要以某种方式处理错误，当一些中间 getX() 调用返回 null 并且所以 d 变为 null. 最好是在包装函数中返回一些默认的 D。

---

如果中间 getX() 中的任何一个返回 null，我看不出您在问题末尾列出的两个选项有什么帮助；你会得到一个NullPointerException。

Answer 3

从 Java 8 开始，您可以使用 Optional.isPresent 和 Optional.orElse 之类的方法来处理 getter 链中的 null：

boolean dNotNull = Optional.ofNullable(obj)
              .map(Obj::getA)
              .map(A::getB)
              .map(B::getC)
              .map(C::getD)
              .isPresent();

虽然这比捕获 NullPointerException 更可取，但这种方法的缺点是 Optional 实例的对象分配。

可以编写自己的静态方法来执行类似的操作，而不会产生这种开销：

boolean dNotNull = Nulls.isNotNull(obj, Obj::getA, A::getB, B::getC, C::getD);

有关示例实现，请参阅 Nullifier 类型 here。

没有任何方法可能比嵌套的 if-not-null 检查具有更高的运行时效率。