Java：删除“Comparable 是原始类型”警告

Question

假设我有一个名为 foo 的方法，以 2 个对象作为参数。两个对象具有相同的类型并且都实现类似的接口。

void foo(Object first, Object second){

    if (!first.getClass().isInstance(second))   //first and second of the same type
        return;

    Comparable firstComparable = (Comparable)first;  //WARNING
    Comparable secondComparable = (Comparable)second;  //WARNING

    int diff = firstComparable.compareTo(secondComparable);  //WARNING
}

前 2 个警告是：

Comparable 是原始类型。对泛型类型 Comparable 的引用 应该参数化

最后一个警告：

类型安全：compareTo(Object)方法属于原始类型 可比。对泛型类型 Comparable 的引用应该是 参数化

如何重构我的代码以删除这些警告？

编辑： 我可以在不更改 foo 方法的签名的情况下做到这一点吗？

Answer 1

您必须告诉编译器它们是相同类型并且具有可比性。如果您无法更改签名，您可以添加向后兼容的方法。

@SuppressWarnings("unchecked")
static void foo(Object first, Object second) {
    foo((Comparable) first, (Comparable) second);
}

static <T extends Comparable<T>> void foo(T first, T second){
    int diff = first.compareTo(second); // no warning.
}

Answer 2

无需更改签名，您可以

    void foo(Object first, Object second){

        if (!first.getClass().isInstance(second)) 
            return;

        Comparable<Object> firstComparable = (Comparable<Object>)first;  
        Comparable<Object> secondComparable = (Comparable<Object>)second; 

        int diff = firstComparable.compareTo(secondComparable);  
    }

但是您仍然得到：
类型安全：未检查从 Object 到 Comparable

但 Comparable 不是原始类型。对泛型类型 Comparable的引用应该参数化
并且没有类型安全性：compareTo(Object) 方法属于原始类型 Comparable。对泛型类型 Comparable的引用应该参数化

Answer 3

你必须使用
Comparable，其中 Type 是实现 Comparable 的对象。

首先，为什么你的方法参数是Objects的实例？如果确定参数类型相同，则应使用特定的类作为参数。如果可以有类的层次结构，则将类放在层次结构中最高的类。使用Object来实现一般功能从来都不是一个好主意。

Answer 4

编辑：既然你说你不能改变方法的签名，那么如果没有不安全的（对编译器）强制转换和@SuppressWarnings，你真的无法逃脱：

@SuppressWarnings("unchecked")
public void foo(final Object first, final Object second) {
    if (!first.getClass().isInstance(second)) // first and second of the
        return;

    Comparable<Object> firstComparable = (Comparable<Object>) first;
    Comparable<Object> secondComparable = (Comparable<Object>) second;
    int diff = firstComparable.compareTo(secondComparable);
}

Answer 5

添加@SuppressWarnings注释。

@SuppressWarnings("unchecked")
void foo(Object first, Object second){

    if (!first.getClass().isInstance(second))   //first and second of the same type
        return;

    Comparable firstComparable = (Comparable)first;  //WARNING
    Comparable secondComparable = (Comparable)second;  //WARNING

    @SuppressWarnings("unused")
    int diff = firstComparable.compareTo(secondComparable);  //WARNING
}