验证 Java 中的可空字段

Question

假设我们有一个 class Product ，它有一些可以为 null 的可为空字段，例如， quantityAvailable和quantityReserved 。

我可能想验证这些字段并抛出错误，但由于它们被允许为null ，我首先要检查null ：

if (product.getQuantityAvailable() != null && product.getQuantityReserved() != null) {
  if (product.getQuantityAvailable() < product.getQuantityReserved()) {
    System.out.println("Error");
  }
}

当您有许多可为空的字段和许多null-checks时，这将变得乏味。 即使使用ofNullable也不会变得更好。

Optional.ofNullable(product.getQuantityAvailable())
    .ifPresent((available) -> {
          Optional.ofNullable(product.getQuantityReserved())
              .ifPresent((reserved) -> {
                    if (available < reserved) {
                      System.out.println("Error");
                    }
                  });
        });

理想情况下，会有一些注释可以检查空值并在找到时跳过，但显然这是行不通的。

if (@Nullable product.getQuantityAvailable() < @Nullable product.getQuantityReserved()) {
  System.out.println("Error");
}

有没有避免上述样板的解决方案？

Answer 1

我认为您应该为附加属性定义具有默认值的模型，而不是接受 null 值。 例如，使用 Integer 属性，默认值为 0，字符串属性为空字符串

Answer 2

即使持久数据库列使用 NULL，我也不会打折Optional 、 OptionalInt等。

确实会很麻烦：

product.getQuantityAvailable()
    .ifPresent((available) -> {
          product.getQuantityReserved()
              .ifPresent((reserved) -> {
                    if (available < reserved) {
                      System.out.println("Error");
                    }
                  });
        });

但是使用从 OptionalInt 到 IntStream 的 map：

if (product.getQuantityAvailable()
        .map().anyMatch(av -> product.getQuantityReserved()
            .map.anyMatch(res -> av < res))) {
    System.out.println("Error");
}

您可以单独存储谓词 lambda，例如在产品中。

该代码更具可读性，更不容易出错，尽管有点）））-ish。

Answer 3

首先，明确的null-checks没有任何问题。

其次， Optional.ofNullable()不用于验证。 通过使用它隐藏空检查，您会掩盖代码的作用并使其更难阅读。

以下是@StuartMarks 、Java 和 OpenJDK 开发人员的回答中关于Optional的用途的引述：

Optional的主要用途如下：

Optional 旨在为库方法返回类型提供有限的机制，其中明确需要表示“无结果”，并且使用 null 极有可能导致错误。

一个典型的代码异味是，它不是使用方法链接来处理从某个方法返回的 Optional 的代码，而是从可为null的东西创建一个Optional ，以便链接方法并避免条件。

还可以看看Stuart Marks的这个答案“应该将 Optional.ofNullable() 用于 null 检查吗？” .

使用Optional.ofNullable()来避免空检查是一种反模式。 您不应创建Optional以验证可以是 null 的null

我在您的问题的这个片段中没有看到任何可以被认为是“坏”的东西。 是的，它很冗长，这是允许可空字段的代价。

if (product.getQuantityAvailable() != null && product.getQuantityReserved() != null) {
    if (product.getQuantityAvailable() < product.getQuantityReserved()) {
        System.out.println("Error");
    }
}

如果您想降低嵌套级别，这里有一个使用Objects.compare()的空安全替代方案：

if (Objects.compare(product.getQuantityAvailable(), 
                    product.getQuantityReserved(), 
                    Comparator.nullsFirst(Comparator.naturalOrder())) < 0) {
    // perform the required action
    throw new MyCustomException("message helpful indebuging here");
}

如果您想减少代码中的空值检查次数，请分配默认值（ Integer 、 BigDecimal.ZERO 、空 collections 等为0 ），而不是将null引用的每个地方都保留 Z37A6259CC0C1DAE299A78664的应用程序。