Java 8 编译器与重载方法混淆

Question

在将应用程序升级到 Java 8 时，我遇到了一个关于 google guava 的奇怪问题 newArrayList在几个地方。

看看这个例子:

import com.google.common.collect.UnmodifiableIterator;

import javax.naming.NamingException;
import javax.naming.directory.Attribute;
import javax.naming.directory.BasicAttribute;
import java.util.ArrayList;

import static com.google.common.collect.Iterators.forEnumeration;
import static com.google.common.collect.Lists.newArrayList;

public class NewArrayListIssue {
    public static void main(String[] args) throws NamingException {

        UnmodifiableIterator<?> elements = forEnumeration(getEnumeration().getAll());
        System.out.println("declarefirst = " + newArrayList(elements)); // calls newArrayList(Iterator<? extends E> elements)

        ArrayList directCopy = newArrayList(forEnumeration(getEnumeration().getAll()));
        System.out.println("useDirectly = " + directCopy); //calls newArrayList(E... elements)
    }

    public static Attribute getEnumeration(){
        return new BasicAttribute("foo",1);
    }
}

在第一个例子中，当我得到 UnmodifiableIterator首先进入它自己的变量，然后调用 newArrayList我得到了我所期望的，这是复制到新的 List 中的迭代器值.

在第二个例子中，forEnumeration直接进入 newArrayList方法我得到一个 List带有包含迭代器(包含值)的 a。

根据 Intellij，它认为两个方法调用都应该是 newArrayList(Iterator<? extends E> elements)但我在调试时发现第二次调用实际上转到了newArrayList(E... elements) .

只有当我使用针对 Java8 的 Oracle JDK8 进行编译时才会发生这种情况。如果我将目标设置为 7，它就可以正常工作。

Answer 1

问题是编译器认为 newArrayList(Iterator<? extends E>)不适用(可能是因为 this bug )，然后在您不使用特定元素类型时默默地选择始终适用的通用可变参数方法(这显示了这种过载的危险)为您的结果列表。

错误出现在通配符类型中，即在您的代码中它是 Attribute.getAll()返回 NamingEnumeration<?>因此 forEnumeration 的结果是UnmodifiableIterator<?>编译器拒绝分配给 Iterable<? extends E> , newArrayList 的参数类型.如果将内部调用的返回值转换为 Enumeration当您将外部调用的返回值转换为 Iterator 时，问题就消失了.

我看不到这个问题的简单短期解决方案。毕竟，我不明白你为什么不用 List<?> directCopy=Collections.list(getEnumeration().getAll());首先……

请注意，如果你想找到所有出现的这个问题，你可以简单地使用补丁版本的 Guava ，其中 newArrayList(E...)已被删除并检查所有编译器错误(假设您真正想要调用此重载的情况不多)。重写调用站点后，您可以返回到原来的 Guava 。