Java 8 Lambdas max()在流中使用Integer.max作为比较器[重复]

Question

我用示例值编写了简单的＜code＞列表，但事实证明，我也可以传递＜code＞Integer：：max

此外，程序打印出奇怪的结果，我检查了它的“内部”,它看起来没问题，但在我得到最终结果后-它们不准确。

例子：

@Test
public void testHowIntegerMaxWorksInStream() {
    List<Integer> list = Arrays.asList(5,3,8);
    Optional<Integer> op = list.stream().max((a, b) -> {
        System.out.println("Input arguments a=" + a + ", b=" + b);

        int max = Integer.max(a, b);
        System.out.println("Returning max(a,b)=" + max);
        return max;
    });
    System.out.println("Optional result=" + op.get());
}

输出：

Input arguments a=5, b=3
Returning max(a,b)=5
Input arguments a=5, b=8
Returning max(a,b)=8 // OK, Integer::max got 8.. but then ...
Optional result=5 // .. I got 5. WHY ???

我的问题是：

1. 为什么我可以传递整数：：max来代替比较器
2. 为什么我的函数返回5作为结果，而在内部它是8

Answer 1

因为您创建的比较器总是返回第一个最大的元素。

如果您比较A和B，如果A较小，则应返回-1。您返回A和B的Integer.max，在您的情况下，它始终是

(a,b) -> { 
   if (a>b) return 1;     // <-- you return a positive value always, so stream.max() thinks 5>8 :)
   if (a<b) return -1;
   if (a==b) return 0;
}

Answer 2

遗憾的是，< code>Integer.max代替比较器进行编译的原因是，它的返回类型符合stream的< code>max方法的预期:比较的结果是一个< code>int，比较两个< code>int的结果也是一个< code>int。但是，< code>Integer.max返回的结果与stream的< code>max预期的结果不一致，这解释了不正确的结果:例如，如果将< code>(5，8)传递给< code>Integer.max，它将返回< code>8，一个正数；然而，stream的< code>max将所有正数解释为第一个参数大于第二个参数的指示，这在这种情况下是不正确的。

此编译的唯一原因是您的比较器返回一个int。这不适用于Double或BigIntger。

Answer 3

你误会了。< code>max(comparator)采用比较整数的比较器。你所做的不是比较整数，即判断< code>a是否大于< code>b。你拿走了它们中的最大值，然后还回去。所以你说< code>a总是大于< code>b，因为你总是返回一个正数(你的列表只包含正数)。< code >比较器返回:

作为第一个参数的负整数、零或正整数小于、等于或大于第二个参数。

你需要做的是

public void testHowIntegerMaxWorksInStream() {
    List<Integer> list = Arrays.asList(5,3,8);
    Optional<Integer> op = list.stream().max((a, b) -> {
        int compare = Integer.compare(a, b);
        return compare;
    });
    System.out.println("Optional result=" + op.get());
}

即调用整数比较而不是最大值，这恰恰是这样做的：

比较其两个参数的顺序。当第一个参数小于、等于或大于第二个参数时，返回负整数、零或正整数。

对于问题的第二部分，请参阅：：（双冒号）运算符在Java 8中。它被称为方法引用。所以另一种写法是：

List<Integer> list = Arrays.asList(5,3,8);
Optional<Integer> op = list.stream().max(Integer::compare);
System.out.println("Optional result=" + op.get());