使用java 8在BigDecimal列表中查找max

Question

让我们假设我有一个这样的类：

public class A {
    private int id;
    private BigDecimal amount;
}

我有一个列表

这种方法：

List<A> brol = new ArrayList<>();
BigDecimal max = brol.stream().max(Comparator.comparing(A -> A.getAmount())).get().getAmount();
        System.out.println("MAX = " + max);

给出了NoTouchElementException，所以我应该为此创建一个特定的比较器？




Answer 1

你不需要一个特定的比较器。比较器不是你的问题。

流畅的调用风格很好、时髦，但它有一个许多人似乎忽视的主要缺点：当函数调用序列中的某个地方发生错误时，你不知道它发生在哪里。

因此，为了解决此类问题，最好将流畅的火车分解成单独的货车。（至少对于故障排除，如果你想的话，把它放回火车后，所有的故障排除都完成了。）

在这种情况下，您将看到max（）返回一个可选

因此，您需要提供ArrayList为空的情况。可以在调用stream（）之前执行此操作。max（），或者您可以按照Peter Lawrey的建议，通过调用可选的isPresent（）方法来实现。

顺便说一句，你犯下这个错误的事实表明：

>

您正在使用一些过时的构建工具，这些工具无法对您的代码发出警告，或者

您没有启用所有警告，或者

你收到警告，但你忽略了它们。

不要这样做，这样你永远不会走得太远。我的IntelliJ IDEA在您的代码上给我一个警告（JetBrains称之为“检查”），说. get（）是在没有. isPresent（）的情况下调用的。因此，我可以在编译代码之前、运行代码之前、故障排除之前以及询问stackoverflow问题之前立即判断代码的问题。

Answer 2

我要做的是检查可选的

Optional<BigDecimal> max = brol.stream()
                               .map(a -> a.amount)
                               .max(Comparator.naturalOrder());
if (max.isPresent()) {
   // have a max
}

而不是比较器。naturalOrder（），如果您觉得这更清楚，可以使用BigDecimal:：compare。

Optional.get（）正在抛出NoSuchElementException，因为没有元素。当获得最大值的对象不是可比较的时，您只需要提供比较器

在这种情况下，您不需要最大A，只需要最大量，它是BigDecimal，它是可比。

如果你真的想要最小值，你可以提供一个比较器，尽管在这方面使用min（）是一个更简单的选择。