即使为空，如何干净地处理 java 8 流“findFirst()”结果

Question

我经常发现与 java 8 流混淆的一个方面是中间结果可能为空，如果它为空或不为空，您需要采用备用路径。

例如，如果我有这样的代码：

String pymtRef = defaultValue;
Optional<PaymentTender> paymentTender = paymentTenders.stream()
        .filter(pt -> (pt.getFlag() == Flag.N || pt.getFlag() == null)).findFirst();
if (paymentTender.isPresent()) {
    pymtRef = paymentTender.get().getId();
}
return pymtRef;

我想弄清楚如何删除条件块并在单个流中执行此操作。

如果我只是在过滤结果上调用“.map”，如果它找到匹配的条目就可以工作。如果没有，我会收到 NoSuchElementException。

我可能会改用“ifPresent()”，但它的返回类型是“void”。

有什么方法可以让这个更干净吗？

更新：

使用“orElse()”的解决方案效果很好。

整个方法现在看起来像这样：

public String getPaymentReference(OrderContext orderContext) {
    List<PaymentTender> paymentTenders = getPaymentTenders(orderContext);
    if (paymentTenders.size() == 1) {
        return paymentTenders.get(0).getId();
    }
    return paymentTenders.stream()
            .filter(pt -> (pt.getAutoBill() == AutoBill.N || pt.getAutoBill() == null))
            .findFirst().map(pt -> pt.getId()).orElse(DEFAULT_VALUE);
}

你能想出一种方法在流中包含第一个条件而不使其更复杂吗？

Answer 1

在 map 之后直接调用 get() 如果 Optional 的状态为空，则会产生异常，而是在 map 之后调用 orElse 并提供默认值：

paymentTenders.stream()
            .filter(pt -> (pt.getFlag() == Flag.N || pt.getFlag() == null))
            .findFirst()
            .map(PaymentTender::getId)
            .orElse(someDefaultValue);

编辑：

至于：

你能想出一种方法在流中包含第一个条件吗 不让它变得更复杂？

不，这比你做的更好。它更易读，更容易理解。

引入任何类型的逻辑以使其成为一个管道（如果可能）只会变得复杂，因此更难遵循和理解。

Answer 2

你可以在一个语句中通过

public String getPaymentReference(OrderContext orderContext) {
    List<PaymentTender> paymentTenders = getPaymentTenders(orderContext);
    return paymentTenders.stream()
        .filter(paymentTenders.size() == 1? pt -> true:
                pt -> pt.getAutoBill() == AutoBill.N || pt.getAutoBill() == null)
        .findFirst().map(PaymentTender::getId).orElse(DEFAULT_VALUE);
}

请注意，这不会对每个元素重复评估paymentTenders.size() == 1，而是根据状态使用不同的函数。当条件满足时，pt -> true 将接受任何元素，这将导致唯一的元素按预期被接受。否则，使用普通谓词pt -> pt.getAutoBill() == AutoBill.N || pt.getAutoBill() == null。