Java：如何避免三元运算符中的 NPE，通常如何优雅地执行空检查？ [复制]

Question

昨天我不得不写一段丑陋的代码，为了对一个对象的字段执行许多空检查，以避免来自三元运算符构造的 NPE。

有问题的代码：

ResourceThresholds rt = getThresholdsFromConfig();
Thresholds defaultPerContainer = getDefaultThresholds();
    
return new Thresholds((!rt.getCpu().equals("")) ? Long.parseLong(rt.getCpu()) : defaultPerContainer.getCpu(),
           (!rt.getMemory().equals("")) ? Long.parseLong(rt.getMemory())  : defaultPerContainer.getMemory(),/*omitted for brevity*/);

我在 defaultPerContainer.getCpu() 上获得了 NPE，因为字段 cpu = null。 这很好，Java 按照它的工作方式工作。 为什么我不只是默认字段 Long cpu = 0L; ？因为我需要null 值作为我们不设置任何值的指标。

这段特定代码的最终功能变体是：

        Long cpuVal;
        if (!rt.getCpu().equals("")) {
            cpuVal = Long.parseLong(rt.getCpu());
        } else {
            cpuVal = defaultPerContainer.getCpu();
        }
        Long memory;
        if (!rt.getMemory().equals("")) {
            memory = Long.parseLong(rt.getMemory());
        } else {
           memory = defaultPerContainer.getMemory();
        }
        //... many similar if-elses that give me the desired value;
        //which is really ugly, and I believe I am not the only one hitting this.
        return new Thresholds(cpuVal, memory..);

这段代码可以按我的需要工作，但它很丑！

Q1：有人可以提示我是否可以找到使用Optional<T> 的方法来解决带有三元运算符的第一个变体中的 NPE？因为这个 sn-p 有效：!rt.getCpu().equals("")) ? Long.parseLong(rt.getCpu()) : null 即如果我明确地将null 作为一个值，我会在满足条件时得到null。

一般来说，有没有优雅的 Java 8+ 方法来处理这个问题？

Q2：您如何优化出色的 if-else 构造以进行空值检查？

Answer 1

1. 代码 sn-p 中没有 null 检查。
2. 最好实现一个简单的实用方法，在设置cpu和memory时使用，如果val是null，使用Joda条件"".equals(val)防止NPE
3. 为避免拆箱，请使用Long.valueOf 而不是返回原始long 的Long.parseLong：

public static Long getValue(String val, Long defaultValue) {
    return "".equals(val) ? defaultValue : Long.valueOf(val);
}

Long cpuVal = getValue(rt.getCpu(), defaultPerContainer.getCpu());
Long memory = getValue(rt.getMemory(), defaultPerContainer.getMemory());

---

也可以使用参数提供者提供重载实用方法，然后将方法引用传递给它：

public static Long getValue(Supplier<String> str, Supplier<Long> defVal) {
    return getValue(str.get(), defVal.get()); // calling implementation above 
}

Long cpuVal = getValue(rt::getCpu, defaultPerContainer::getCpu);
Long memory = getValue(rt::getMemory, defaultPerContainer::getMemory);

Answer 2

问题是在三元表达式A ? B : C中，如果B和C都是兼容的数值类型，但是一个是boxed object，另一个是原语，大多数人会认为结果是通过自动装箱原语来装箱的。

事实并非如此。相反，三元运算符将对象拆箱，因此它们都是原语，结果是原语。

这意味着以下内容相同：

long B = ...;
Long C = ...;

Long R = ... ? B : C;

Long R = (Long) (... ? B : (long) C);

结果是如果C 为空，则得到NPE。

解决它的一种方法是强制B 自动装箱：

Long R = ... ? (Long) B : C;

通过该更改，一个空的C 值将简单地设置R = null。

在问题的情况下，B 是 Long.parseLong(rt.getCpu())，因此不要添加强制转换来强制自动装箱，请使用 long.valueOf(String s)。

另外，不相关的，使用isEmpty() 而不是equals("")，并且A 周围不需要括号。

将代码改为：

return new Thresholds(!rt.getCpu().isEmpty() ? Long.valueOf(rt.getCpu()) : defaultPerContainer.getCpu(),
                      !rt.getMemory().isEmpty() ? Long.valueOf(rt.getMemory())  : defaultPerContainer.getMemory(),
                      /*omitted for brevity*/);