@Test
public void test() {
    MyProperties props = new MyProperties();
    props.setProperty("value", new Date());

    StringUtils.isNullOrEmpty(props.getProperty("value"));
}

public class MyProperties {
    private Map<String, Object> properties = new HashMap<String, Object>();

    public void setProperty(String name, Object value) {
        properties.put(name, value);
    }

    @SuppressWarnings("unchecked")
    public <T> T getProperty(String name) {
        return (T) properties.get(name);
    }
}

public class StringUtils {

    public static boolean isNullOrEmpty(Object string) {
        return isNullOrEmpty(valueOf(string));
    }

    public static String valueOf(Object string) {
        if (string == null) {
            return "";
        }
        return string.toString();
    }

    public static boolean isNullOrEmpty(String string) {
        if (string == null || string.length() == 0) {
            return false;
        }
        int strLength = string.length();
        for (int i = 0; i < strLength; i++) {
            char charAt = string.charAt(i);
            if (charAt > ' ') {
                return true;
            }
        }
        return false;
    }

}

多年来，这个单元测试一直通过。然后升级到 Java 8 后，在某些环境下，当通过 javac 编译代码时，它会选择 StringUtils.isNullOrEmpty(String) 重载。这会导致单元测试失败并显示以下错误消息：

java.lang.ClassCastException: java.util.Date cannot be cast to java.lang.String at com.foo.bar.StringUtils_UT.test(StringUtils_UT.java:35)

通过 ant（ant 1.9.6、jdk_8_u60、Windows 7 64 位）在我的机器上编译和运行时单元测试通过，但在具有相同版本的 ant 和 java 的另一台机器上失败（ant 1.9.6 jdk_8_u60、Ubuntu 12.04.4 32 位） 。

Java's 类型推断 https://docs.oracle.com/javase/tutorial/java/generics/genTypeInference.html，它在编译时从所有适用的重载中选择最具体的重载，已在 Java 8 中进行了更改。我认为我的问题与此有关。

我知道编译器将 MyProperties.getProperty(...) 方法的返回类型视为 T，而不是 Date。由于编译器不知道 getProperty(...) 方法的返回类型，为什么它选择 StringUtils.isNullorEmpty(String) 而不是 StringUtils.isNullorEmpty(Object) - 后者应该始终有效？

这是 Java 中的错误还是只是 Java 8 类型推断更改的结果？另外，为什么使用相同版本的java的不同环境会以不同的方式编译此代码？

这段代码smells。是的，这在 Java 7 下通过了，是的，它在 Java 7 中运行正常，但是有一些东西绝对错误 here.

首先，我们来谈谈这个泛型类型。

@SuppressWarnings("unchecked")
public <T> T getProperty(String name) {
    return (T) properties.get(name);
}

你能一眼看出什么T should是？如果我在 Java 7 合规模式下使用 IntelliJ 在该行运行这些转换，我会得到这非常有帮助的信息ClassCastException:

Cannot cast java.util.Date to T

因此，这意味着在某种程度上，Java 知道这里有问题，但它选择更改该转换，而不是从(T) to (Object).

@SuppressWarnings("unchecked")
public <T> Object getProperty(String name) {
    return (Object) properties.get(name);
}

在这种情况下，演员阵容是多余的，你会得到一个Object正如您所期望的，从地图上可以看到。然后，调用正确的重载。

现在，在 Java 8 中，事情变得更加理智了；因为你并没有真正提供类型getProperty方法，它爆炸了，因为它really无法投射java.util.Date to T.

最后，我掩盖了要点：

泛型的这种使用是错误的并且是错误的。

你甚至不need这里泛型。您的代码可以处理String or an Object，而你的地图只包含Object无论如何。

你应该只返回Object来自getProperty方法，因为无论如何你只能从地图返回。

public Object getProperty(String name) {
    return properties.get(name);
}

这确实意味着您不再能够拨打电话directly进入签名为的方法String（因为你正在通过Object现在），但这确实意味着您损坏的泛型代码最终可以得到解决。

If you really不过，想要保留这种行为，您必须在函数中引入一个新参数，该参数实际上允许您指定想要从映射返回哪种类型的对象。

@SuppressWarnings("unchecked")
public <T> T getProperty(String name, Class<T> clazz) {
    return (T) properties.get(name);
}

然后你可以这样调用你的方法：

StringUtils.isNullOrEmpty(props.getProperty("value", Date.class));

现在我们绝对确定什么T是的，Java 8 对此代码很满意。这仍然有点气味，因为你把东西存放在一个Map<String, Object>;如果你有Object重写方法，您可以保证该映射中的所有对象都具有有意义的toString，那么我个人会避免使用上面的代码。