Como usar o comparador em Java para ordenar

Aprendi a usar o comparável, mas estou a ter dificuldade com o comparador. Estou tendo um erro no meu código:

Exception in thread "main" java.lang.ClassCastException: New.People cannot be cast to java.lang.Comparable
 at java.util.Arrays.mergeSort(Unknown Source)
 at java.util.Arrays.sort(Unknown Source)
 at java.util.Collections.sort(Unknown Source)
 at New.TestPeople.main(TestPeople.java:18)
Aqui está o meu código:
import java.util.Comparator;

public class People implements Comparator {
   private int id;
   private String info;
   private double price;

   public People(int newid, String newinfo, double newprice) {
       setid(newid);
       setinfo(newinfo);
       setprice(newprice);
   }

   public int getid() {
       return id;
   }

   public void setid(int id) {
       this.id = id;
   }

   public String getinfo() {
       return info;
   }

   public void setinfo(String info) {
       this.info = info;
   }

   public double getprice() {
       return price;
   }

   public void setprice(double price) {
       this.price = price;
   }

   public int compare(Object obj1, Object obj2) {
       Integer p1 = ((People) obj1).getid();
       Integer p2 = ((People) obj2).getid();

       if (p1 > p2) {
           return 1;
       } else if (p1 < p2){
           return -1;
       } else {
           return 0;
       }
    }
}
import java.util.ArrayList;
import java.util.Collections;

public class TestPeople {
    public static void main(String[] args) {
        ArrayList peps = new ArrayList();

        peps.add(new People(123, "M", 14.25));
        peps.add(new People(234, "M", 6.21));
        peps.add(new People(362, "F", 9.23));
        peps.add(new People(111, "M", 65.99));
        peps.add(new People(535, "F", 9.23));

        Collections.sort(peps);

        for (int i = 0; i < peps.size(); i++){
            System.out.println(peps.get(i));
        }
    }
}

Eu acredito que tem que fazer algo com a fundição no método de comparação, mas eu estava brincando com ele e ainda não conseguia encontrar a solução

Author: Mr. Polywhirl, 2010-05-15

10 answers

Há algumas coisas estranhas na tua aula de exemplo.
  • chama-se povo enquanto tem um price e info (mais algo para objectos, Não pessoas);
  • Ao nomear uma classe como plural de algo, sugere que é uma abstração de mais de uma coisa.
De qualquer forma, aqui está uma demonstração de como usar um Comparator<T>:
public class ComparatorDemo {

    public static void main(String[] args) {
        List<Person> people = Arrays.asList(
                new Person("Joe", 24),
                new Person("Pete", 18),
                new Person("Chris", 21)
        );
        Collections.sort(people, new LexicographicComparator());
        System.out.println(people);
        Collections.sort(people, new AgeComparator());
        System.out.println(people);
    }
}

class LexicographicComparator implements Comparator<Person> {
    @Override
    public int compare(Person a, Person b) {
        return a.name.compareToIgnoreCase(b.name);
    }
}

class AgeComparator implements Comparator<Person> {
    @Override
    public int compare(Person a, Person b) {
        return a.age < b.age ? -1 : a.age == b.age ? 0 : 1;
    }
}

class Person {

    String name;
    int age;

    Person(String n, int a) {
        name = n;
        age = a;
    }

    @Override
    public String toString() {
        return String.format("{name=%s, age=%d}", name, age);
    }
}

Editar

E uma demonstração Java 8 equivalente seria assim:

public class ComparatorDemo {

    public static void main(String[] args) {
        List<Person> people = Arrays.asList(
                new Person("Joe", 24),
                new Person("Pete", 18),
                new Person("Chris", 21)
        );
        Collections.sort(people, (a, b) -> a.name.compareToIgnoreCase(b.name));
        System.out.println(people);
        Collections.sort(people, (a, b) -> a.age < b.age ? -1 : a.age == b.age ? 0 : 1);
        System.out.println(people);
    }
}
 187
Author: Bart Kiers, 2016-02-27 21:20:57

Aqui está um modelo super curto para fazer a ordenação imediatamente:

Collections.sort(people,new Comparator<Person>(){
   @Override
   public int compare(final Person lhs,Person rhs) {
     //TODO return 1 if rhs should be before lhs 
     //     return -1 if lhs should be before rhs
     //     return 0 otherwise
     }
 });

Se é difícil de lembrar, tente apenas lembrar que é semelhante (em termos do sinal do número) a:

 lhs-rhs 

Isso é no caso de você querer ordenar em ordem ascendente : de menor número para maior número.

 118
Author: android developer, 2017-01-19 08:19:29

Utilize People implements Comparable<People> em vez disso; isto define a ordenação natural para People.

A Comparator<People> também pode ser definido em adição, mas People implements Comparator<People> não é a maneira certa de fazer as coisas.

As duas sobrecargas para {[6] } são diferentes:

  • <T extends Comparable<? super T>> void sort(List<T> list)
    • ordena objectos Comparable usando a sua ordenação natural
  • <T> void sort(List<T> list, Comparator<? super T> c)
    • ordena qualquer coisa usando um Compatível Comparator
Estás a confundir os dois ao tentar separar um Comparator (que é novamente por isso que não faz sentido que Person implements Comparator<Person>). Mais uma vez, para usar Collections.sort, você precisa de um destes para ser verdadeiro:
  • o tipo deve ser Comparable (utilizar o 1-arg sort)
  • a Comparator for the type must be provided (use the 2-args sort)

Perguntas relacionadas


Além disso, não utilize tipos brutos em novos código Os tipos crus não são seguros, e são fornecidos apenas para compatibilidade.

Isto é, em vez disto:
ArrayList peps = new ArrayList(); // BAD!!! No generic safety!

Devias ter usado a declaração genérica typesafe como esta:

List<People> peps = new ArrayList<People>(); // GOOD!!!

Você vai descobrir que o seu código nem sequer compila!!Isso seria bom, porque há algo de errado com o código.implements Comparable<Person>), mas porque você usou o tipo raw, o compilador não verificou para isso , e em vez disso você tem um ClassCastException at corre!!!

Isto deve convencê-lo a usar sempre os tipos genéricos typesafe no novo código. Sempre.

Ver também

 32
Author: polygenelubricants, 2017-05-23 12:34:50
Por uma questão de exaustividade, aqui está um método simples de um liner compare:
Collections.sort(people,new Comparator<Person>()
{
    @Override
    public int compare(Person lhs,Person rhs) 
    {  
      return Integer.signum(lhs.getId()-rhs.getId());  
    }
}
 9
Author: NumberFour, 2014-12-10 19:30:57

Java 8 adicionou uma nova forma de fazer comparadores que reduz a quantidade de código que você tem que escrever, comparador.comparando. Verifique também O comparador.invertida

Aqui está uma amostra.
import org.junit.Test;

import java.util.ArrayList;
import java.util.Comparator;
import java.util.List;

import static org.junit.Assert.assertTrue;

public class ComparatorTest {

    @Test
    public void test() {
        List<Person> peopleList = new ArrayList<>();
        peopleList.add(new Person("A", 1000));
        peopleList.add(new Person("B", 1));
        peopleList.add(new Person("C", 50));
        peopleList.add(new Person("Z", 500));
        //sort by name, ascending
        peopleList.sort(Comparator.comparing(Person::getName));
        assertTrue(peopleList.get(0).getName().equals("A"));
        assertTrue(peopleList.get(peopleList.size() - 1).getName().equals("Z"));
        //sort by name, descending
        peopleList.sort(Comparator.comparing(Person::getName).reversed());
        assertTrue(peopleList.get(0).getName().equals("Z"));
        assertTrue(peopleList.get(peopleList.size() - 1).getName().equals("A"));
        //sort by age, ascending
        peopleList.sort(Comparator.comparing(Person::getAge));
        assertTrue(peopleList.get(0).getAge() == 1);
        assertTrue(peopleList.get(peopleList.size() - 1).getAge() == 1000);
        //sort by age, descending
        peopleList.sort(Comparator.comparing(Person::getAge).reversed());
        assertTrue(peopleList.get(0).getAge() == 1000);
        assertTrue(peopleList.get(peopleList.size() - 1).getAge() == 1);
    }

    class Person {

        String name;
        int age;

        Person(String n, int a) {
            name = n;
            age = a;
        }

        public String getName() {
            return name;
        }

        public int getAge() {
            return age;
        }

        public void setName(String name) {
            this.name = name;
        }

        public void setAge(int age) {
            this.age = age;
        }
    }



}
 6
Author: rince, 2017-03-09 19:44:28

Você quer implementar comparável, não comparador. Você precisa implementar o método do compareTo. Mas estás perto. O comparador é uma rotina de comparação de" terceiros". Comparável é que este objeto pode ser comparado com outro.

public int compareTo(Object obj1) {
  People that = (People)obj1;
  Integer p1 = this.getId();
  Integer p2 = that.getid();

  if (p1 > p2 ){
   return 1;
  }
  else if (p1 < p2){
   return -1;
  }
  else
   return 0;
 }
Nota, é melhor procurares nulls aqui para o getId..só por precaução.
 4
Author: Will Hartung, 2010-05-15 06:29:45

Aqui está um exemplo de um comparador que irá funcionar para qualquer método ARG zero que retorne um comparável. Existe algo assim num jdk ou numa biblioteca?

import java.lang.reflect.Method;
import java.util.Comparator;

public class NamedMethodComparator implements Comparator<Object> {

    //
    // instance variables
    //

    private String methodName;

    private boolean isAsc;

    //
    // constructor
    //

    public NamedMethodComparator(String methodName, boolean isAsc) {
        this.methodName = methodName;
        this.isAsc = isAsc;
    }

    /**
     * Method to compare two objects using the method named in the constructor.
     */
    @Override
    public int compare(Object obj1, Object obj2) {
        Comparable comp1 = getValue(obj1, methodName);
        Comparable comp2 = getValue(obj2, methodName);
        if (isAsc) {
            return comp1.compareTo(comp2);
        } else {
            return comp2.compareTo(comp1);
        }
    }

    //
    // implementation
    //

    private Comparable getValue(Object obj, String methodName) {
        Method method = getMethod(obj, methodName);
        Comparable comp = getValue(obj, method);
        return comp;
    }

    private Method getMethod(Object obj, String methodName) {
        try {
            Class[] signature = {};
            Method method = obj.getClass().getMethod(methodName, signature);
            return method;
        } catch (Exception exp) {
            throw new RuntimeException(exp);
        }
    }

    private Comparable getValue(Object obj, Method method) {
        Object[] args = {};
        try {
            Object rtn = method.invoke(obj, args);
            Comparable comp = (Comparable) rtn;
            return comp;
        } catch (Exception exp) {
            throw new RuntimeException(exp);
        }
    }

}
 2
Author: John, 2017-03-10 14:41:12
public static Comparator<JobSet> JobEndTimeComparator = new Comparator<JobSet>() {
            public int compare(JobSet j1, JobSet j2) {
                int cost1 = j1.cost;
                int cost2 = j2.cost;
                return cost1-cost2;
            }
        };
 1
Author: QuadBiker, 2015-10-06 06:08:34

A solução pode ser optimizada da seguinte forma: Em primeiro lugar, usar uma classe interna privada como o escopo para os campos é ser o grupo de teste de classe envolvente, de modo que a implementação de pessoas de classe não será exposta ao mundo exterior. Isto pode ser entendido em termos de criação de um APIthat espera uma lista ordenada de pessoas Em segundo lugar, usando a expressão Lamba (java 8) que reduz o código, daí o esforço de desenvolvimento

Por isso, o código seria o seguinte:

import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;

public class TestPeople {
    public static void main(String[] args) {
        ArrayList<People> peps = new ArrayList<>();// Be specific, to avoid
                                                    // classCast Exception

        TestPeople test = new TestPeople();

        peps.add(test.new People(123, "M", 14.25));
        peps.add(test.new People(234, "M", 6.21));
        peps.add(test.new People(362, "F", 9.23));
        peps.add(test.new People(111, "M", 65.99));
        peps.add(test.new People(535, "F", 9.23));

        /*
         * Collections.sort(peps);
         * 
         * for (int i = 0; i < peps.size(); i++){
         * System.out.println(peps.get(i)); }
         */

        // The above code can be replaced by followin:

        peps.sort((People p1, People p2) -> p1.getid() - p2.getid());

        peps.forEach((p) -> System.out.println(" " + p.toString()));

    }

    private class People {
        private int id;

        @Override
        public String toString() {
            return "People [id=" + id + ", info=" + info + ", price=" + price + "]";
        }

        private String info;
        private double price;

        public People(int newid, String newinfo, double newprice) {
            setid(newid);
            setinfo(newinfo);
            setprice(newprice);
        }

        public int getid() {
            return id;
        }

        public void setid(int id) {
            this.id = id;
        }

        public String getinfo() {
            return info;
        }

        public void setinfo(String info) {
            this.info = info;
        }

        public double getprice() {
            return price;
        }

        public void setprice(double price) {
            this.price = price;
        }
    }
}
 1
Author: Akhil Gupta, 2016-11-24 15:39:38

Deve usar o método sort sobrecarregado(peps, new People ())

import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;

public class Test 
{
    public static void main(String[] args) 
    {
        List<People> peps = new ArrayList<>();

        peps.add(new People(123, "M", 14.25));
        peps.add(new People(234, "M", 6.21));
        peps.add(new People(362, "F", 9.23));
        peps.add(new People(111, "M", 65.99));
        peps.add(new People(535, "F", 9.23));

        Collections.sort(peps, new People().new ComparatorId());

        for (int i = 0; i < peps.size(); i++)
        {
            System.out.println(peps.get(i));
        }
    }
}

class People
{
       private int id;
       private String info;
       private double price;

       public People()
       {

       }

       public People(int newid, String newinfo, double newprice) {
           setid(newid);
           setinfo(newinfo);
           setprice(newprice);
       }

       public int getid() {
           return id;
       }

       public void setid(int id) {
           this.id = id;
       }

       public String getinfo() {
           return info;
       }

       public void setinfo(String info) {
           this.info = info;
       }

       public double getprice() {
           return price;
       }

       public void setprice(double price) {
           this.price = price;
       }

       class ComparatorId implements Comparator<People>
       {

        @Override
        public int compare(People obj1, People obj2) {
               Integer p1 = obj1.getid();
               Integer p2 = obj2.getid();

               if (p1 > p2) {
                   return 1;
               } else if (p1 < p2){
                   return -1;
               } else {
                   return 0;
               }
            }
       }
    }
 0
Author: michal, 2015-07-26 13:45:03