为什么这段代码会抛出ClassCastException以及如何避免它

Question

考虑以下代码:

import java.util.*;



class jm45 implements Comparator<jm45>
{
   private int x;
   jm45(int input) { x = input; }
   public static void main( String args[] )
   {
      List list = new ArrayList();
      list.add(new jm45(2));
      list.add(new jm45(2));
      Collections.sort(list); //faulty line
   }
   public int compare( jm45 t1 , jm45 t2 )
   {
      return t1.x - t2.x;
   }
}

Answer 1

你的类实现Comparator<jm45>而不是Comparable<jm45>.

A Comparator知道如何比较两个对象 - 一个Comparable知道如何比较另一个对象.

您需要传入比较器以sort()供使用(作为第二个参数),或者值必须具有可比性.

这是一个使用Comparable界面的版本:

import java.util.*;

class Test implements Comparable<Test>
{
    private int x;

    Test(int input)
    { 
        x = input;
    }

    public static void main(String args[])
    {
        List<Test> list = new ArrayList<Test>();
        list.add(new Test(2));
        list.add(new Test(2));
        Collections.sort(list);
    }

    public int compareTo(Test other)
    {
      return x - other.x;
    }
}

这是一个使用Comparator界面的版本:

import java.util.*;

class TestComparator implements Comparator<Test>
{
   public int compare(Test t1, Test t2)
   {
      return t1.getX() - t2.getX();
   }
}

class Test
{
    private int x;

    Test(int input)
    { 
        x = input;
    }

    int getX()
    {
        return x;
    }

    public static void main(String args[])
    {
        List<Test> list = new ArrayList<Test>();
        list.add(new Test(2));
        list.add(new Test(2));
        Collections.sort(list, new TestComparator());
    }
}

没有什么可以阻止一个类实现Comparator<T>自己,但这样做有点奇怪.例如,你通常不会要求一个字符串将另外两个字符串相互比较 - 它与原始字符串本身无关.