警告：[重载] 方法 m1 可能与方法 m2 不明确

Question

import java.util.function.*;

class Test { 
    void test(int    foo, Consumer<Integer> bar) { }
    void test(long   foo, Consumer<Long>    bar) { }
    void test(float  foo, Consumer<Float>   bar) { }
    void test(double foo, Consumer<Double>  bar) { }
}

当我用javac -Xlint Test.java 编译它时，我收到了一些警告：

Test.java:4: warning: [overloads] test(int,Consumer<Integer>) in Test is potentially ambiguous with test(long,Consumer<Long>) in Test
    void test(int    foo, Consumer<Integer> bar) { }
         ^
Test.java:6: warning: [overloads] test(float,Consumer<Float>) in Test is potentially ambiguous with test(double,Consumer<Double>) in Test
    void test(float  foo, Consumer<Float>   bar) { }
         ^
2 warnings

如果我将Consumer 更改为Supplier，警告就会消失。这个程序没有警告：

import java.util.function.*;

class Test { 
    void test(int    foo, Supplier<Integer> bar) { }
    void test(long   foo, Supplier<Long>    bar) { }
    void test(float  foo, Supplier<Float>   bar) { }
    void test(double foo, Supplier<Double>  bar) { }
}

这是为什么呢？这个警告是什么意思？这些方法如何模棱两可？抑制警告是否安全？

Answer 1

这些警告的出现是因为重载解析、目标类型和类型推断之间有趣的交集。编译器会提前为您考虑并警告您，因为大多数 lambda 表达式都是在没有显式声明的类型的情况下编写的。例如，考虑这个调用：

    test(1, i -> { });

i 的类型是什么？编译器在完成重载解析之前无法推断它......但值 1 匹配所有四个重载。无论选择哪个重载都会影响第二个参数的目标类型，进而影响为i 推断的类型。这里确实没有足够的信息让编译器决定调用哪个方法，所以这行实际上会导致编译时错误：

    error: reference to test is ambiguous
           both method test(float,Consumer<Float>) in Test and
           method test(double,Consumer<Double>) in Test match

（有趣的是，它提到了 float 和 double 重载，但是如果您将其中一个注释掉，您会得到与 long 重载相同的错误。）

可以想象一种策略，其中编译器使用最具体的规则完成重载解析，从而选择带有 int 参数的重载。然后它将有一个明确的目标类型应用于 lambda。编译器设计者认为这太微妙了，在某些情况下程序员会惊讶于最终调用了哪个重载。与其以可能出乎意料的方式编译程序，他们认为将其作为错误并迫使程序员消除歧义会更安全。

编译器在方法声明处发出警告，表明程序员为调用这些方法之一而编写的可能代码（如上所示）将导致编译时错误。

为了消除通话的歧义，人们不得不写

    test(1, (Integer i) -> { });

或为i 参数声明一些其他显式类型。另一种方法是在 lambda 之前添加强制转换：

    test(1, (Consumer<Integer>)i -> { });

但这可以说是更糟。您可能不希望 API 的调用者在每次调用时都必须处理这种事情。

Supplier 的情况不会出现这些警告，因为可以通过本地推理确定供应商的类型，而无需任何类型推断。

您可能需要重新考虑将这个 API 组合在一起的方式。如果您真的想要具有这些参数类型的方法，最好将方法重命名为testInt、testLong 等，并完全避免重载。请注意，Java SE API 在类似的情况下已经做到了这一点，例如Comparator 上的comparingInt、comparingLong 和comparingDouble；还有mapToInt、mapToLong和mapToDoubleStream。