在将 java 流应用到列表之前对列表进行空检查

Question

我有以下代码

item.getSubCategory().stream().map(category -> {
              return subCategoriesList.add(new SubCategoryViewModel(
                      category.get(String.format("_%s",categoryProperties[0])).toString(),
                      category.get(categoryProperties[2]).toString(),
                      category.get(categoryProperties[3]).toString()));
            }).collect(Collectors.toList());

item.getSubCategory() 是一个类别模型列表。现在，如果 getSubCategory 上出现 null，我们会得到一个空指针异常，并且不能对其应用 Steam。在将流应用到列表之前，是否有更好的方法来处理对列表的空检查。

我不想使用IF 语句来检查getSubCategory 上的空值。 java steam API有什么更好的方法吗？

Answer 1

返回列表的方法永远不应该返回null - 这是一种非常糟糕的做法。只需在构造函数中初始化列表，如果没有添加任何对象，则返回一个空列表。这样一来，您就可以在使用 getSubCategory 时消除所有需要的空检查。

可以说你最终得到了一个占用内存的未使用对象。然而，除非你真的有很多 item 对象，否则拥有一些“备用”列表不会伤害任何人。请记住，在大多数情况下，代码可读性是第一位的 - 当您看到实际问题时修复性能。

编辑：按照 Holger 的建议，如果您担心这些空列表的内存成本，您可以使用 Collections.emptyList()。在您的item 中，您保留所有内容，仅在需要时创建列表等。但是，在getSubCategory 内部而不是返回null，您应该检查列表是否已初始化，如果未初始化，则返回Collections.emptyList() 代替null。根据文档，Collections.emptyList() 返回一个不可变的列表实例，多次调用将返回同一个实例 - 只会创建一个。

Answer 2

从 Java 9 开始，你也可以使用Objects.requireNonNullElse()：

Objects.requireNonNullElse(item.getSubCategory(), Collections.emptyList())
    .stream()
    .map(...);

这可能类似于：

Optional.ofNullable(item.getSubCategory()).orElse(Collections.emptyList())
    .stream()
    .map(...);

但方式不同，requireNonNullElse() 将直接返回非空List，而Optional 变体将初始List 包装在Optional 中，然后用orElse() 展开

Answer 3

也许这应该是一个很好的评论候选者，但我怀疑一个足以解释我的观点。

---

我经常看到有人建议使用空列表（或任何空集合）而不是 null。在我看来，这是两种完全不同的结果类型：empty list 和 absent list。也许有时一个空列表就可以了，但并非在所有情况下都如此。

我同意返回 null 是一种糟糕的做法，会导致许多进一步的问题。但是null 并不是表达没有结果的唯一方式。 我宁愿使用 Optional.empty() 而不是空列表作为默认结果。

这是一个证明我的观点的例子。

假设我们有一个从电表获取消耗记录列表的方法。有时设备可能处于离线状态，因此您无法获取任何内容，稍后将重试。 现在看看这两种方法：

List<ConsumptionAmount> getConsumptionData(Date from, Date to); 

Optional<List<ConsumptionAmount>> getConsumptionData(Date from, Date to); 

---

第一个实现非常棘手。返回空列表非常令人困惑（因此说根本没有能源消耗）。

List<ConsumptionAmount> consumptionData = getConsumptionData(from, to);
// report 'zero consumption' to the billing service

另一种选择是传播异常，这也可能导致代码中出现try-catch 混乱（调用者必须了解异常类型并做出相应的行为，否则异常会进一步传播，导致更大的混乱） ：

try {
    List<ConsumptionAmount> consumptionData = getConsumptionData(from, to);
    // report consumption to the billing service
} catch (WhateverException ex) {
    // handle missing data here
}

---

相比之下，第二个实现没有提供有关导致数据缺失的问题的任何细节，但它也不会混淆调用者。

getConsumptionData(from, to).ifPresent( list -> /* report consumption here */);

如果您确实关心丢失的结果，那么只需提供另一种方法参考：

getConsumptionData(from, to).ifPresentOrElse( 
    list -> /* report consumption here    */,
    ()   -> /* handle missing data here */
);

---

对于此类用例（Either 和 Try），还有更有趣的功能方法，但不幸的是，Java 并没有提供开箱即用的方法。您可以检查第三方框架，例如this one

Answer 4

之前有没有更好的方法来处理对列表的空检查 对其应用流。

我只能想到一种方法。使项目 item.getSubCategory() 返回 Optional<YourObject> 而不是实际的对象。

我不想使用 IF 语句在 getSubCategory 上检查 null。是 java steam API有什么更好的方法吗？

您的担忧与流 API 无关。你问的是如何处理一个可能为空的对象，这样我就不必使用 If，else 检查。