需要进行空值检查的通用方法[重复]

Question

我需要一种通用的空值检查方法。我提出了一个要求。 这是我的类的 setter 方法：

placeHolderValues.setCurrentAddressCountry(communicationSO.getApplicationDetailSO().getApplicantDetails().get(0).getAddressDetail().get(0).getCountry());

我只想在值存在时在placeHolderValues.setCurrentAddressCountry() 中设置值。如果getApplicationDetailSO() 或getApplicantDetails() 或getAddressDetail() 或getCountry() 中的任何一个为NULL，那么我必须在placeHolderValues.setCurrentAddressCountry() 中设置NULL。我有将近 30 种这样的 setter。

我需要一个通用的方法来达到同样的效果。

Answer 1

这里有一个严重的不回答：不要那样做。

使用getA().getB().getC().getD()...getTheThingYouWant() 是非常糟糕的做法。 这明显违反了Law Of Demeter。

所以除了你遇到的问题（关于检查空值）之外，这样的构造实际上是邪恶的。你看，使用这段代码的类必须知道 A 有一个 B。B 有一个 C。以此类推。这直接将这个类与所有类 A、B、C ......等等结合起来。

真正的答案是避免这种变化。如果有的话，您可以使用getA().getMeWhatINeed()。

唯一的其他选择：在进入此类调用链之前，完全验证您的输入数据（在前期步骤中）。

Answer 2

你可以使用Optional.ofNullable和重复map将值转换成另一个，除非你在途中遇到null，然后得到最终值orElsenull。

var value = Optional.ofNullable(communicationSO)
        .map(x -> x.getApplicationDetailSO())
        .map(x -> x.getApplicantDetails())
        .map(x -> x.get(0))
        .map(x -> x.getAddressDetail())
        .map(x -> x.get(0))
        .map(x -> x.getCountry())
        .orElse(null);
placeHolderValues.setCurrentAddressCountry(value);

这仍然不是很漂亮，但至少你不必重复不同的方法调用。其中一些 lambdas (->) 可能也可以替换为适当的方法引用。 (::) 不过，这可能比将行包装到 try 块和 catching NullPointerExceptoin 中更好。