18

我正在将应用程序转换为使用 Java 1.5 并找到以下方法：

  /**
   * Compare two Comparables, treat nulls as -infinity.
   * @param o1
   * @param o2
   * @return -1 if o1<o2, 0 if o1==o2, 1 if o1>o2
   */
  protected static int nullCompare(Comparable o1, Comparable o2) {
    if (o1 == null) {
      if (o2 == null) {
        return 0;
      } else {
        return -1;
      }
    } else if (o2 == null) {
      return 1;
    } else {
      return o1.compareTo(o2);
    }
  }

理想情况下，我想让该方法采用两个相同类型的 Comparables，是否可以转换它以及如何转换？

我认为以下方法可以解决问题：

protected static <T extends Comparable> int nullCompare(T o1, T o2) {

但它未能消除 IntelliJ 中的警告“未检查调用 'compareTo(T)' 作为原始类型 'java.lang.Comparable' 的成员”：

return o1.compareTo(o2);
4

5 回答 5

21

将其更改为：

protected static <T extends Comparable<T>> int nullCompare(T o1, T o2) {

您需要它，因为 Comparable 本身就是一个泛型类型。

于 2008-09-12T14:24:53.267 回答
5

这是一个奇怪的案例：

static class A {
    ...
}

static class B extends A implements Comparable<A> {
    public int compareTo(A o) {
        return ...;
    }
}

幸运的是，像上面这样的代码很少见，但是 nullCompare() 将不支持 Bs 的比较，除非声明 Comparable 可以应用于 T或其任何超类：

protected static <T extends Comparable<? super T>> int nullCompare(T o1, T o2) {

尽管大多数人永远不会从上述调整中受益，但在为导出的库设计 API 时它可能会派上用场。

于 2008-09-17T17:42:56.277 回答
2

无法编辑，所以我必须发布我的答案。

您需要声明嵌套类型参数，因为 Comparable 是通用的。

protected static <T extends Comparable<? super T>> int nullCompare(T o1, T o2) {

请注意Comparable< ? super T >，这使得更灵活。您将在 Collections.sort 上看到相同的方法定义

public static <T extends Comparable<? super T>> void sort(List<T> list) {
于 2008-10-22T23:18:25.597 回答
0

我不确定泛化这种方法是否有意义。目前该方法适用于任何类型的 Comparable；如果您对其进行泛化，您将不得不多次实现它（使用完全相同的代码）。有时可以比较两个没有共同祖先的对象，但任何通用版本都不允许这样做。

通过添加泛型，您不会给代码增加任何安全性；在调用 compareTo 时会出现任何安全问题。我的建议只是压制警告。它并没有真正警告你任何有用的东西。

于 2008-09-22T15:30:53.343 回答
0

为了使其更通用，您甚至可以让它适用于两种不同的类型。=P

  /**
   * Compare two Comparables, treat nulls as -infinity.
   * @param o1
   * @param o2
   * @return -1 if o1&lt;o2, 0 if o1==o2, 1 if o1&gt;o2
   */
  protected static <T> int nullCompare(Comparable<? super T> o1, T o2) {
    if (o1 == null) {
      if (o2 == null) {
        return 0;
      } else {
        return -1;
      }
    } else if (o2 == null) {
      return 1;
    } else {
      return o1.compareTo(o2);
    }
  }
于 2009-05-02T19:51:12.673 回答