如何按日期对对象列表进行排序（java 集合，List）

Question

private List<Movie> movieItems = null;
public List<Movie> getMovieItems() {
    final int first = 0;
    if (movieItems == null) {
        getPagingInfo();
        movieItems = jpaController.findRange(new int[]{pagingInfo.getFirstItem(), pagingInfo.getFirstItem() + pagingInfo.getBatchSize()});
        Collections.sort(movieItems, new Comparator(){
           public int compare (Object o1, Object o2){
               Date d1 = movieItems.get(((Movie)o1).getMovieId()).getDate();
               Date d2 = movieItems.get(((Movie)o2).getMovieId()).getDate();
               if(d1.before(d2)){
                   movieItems.set(1, (Movie)o1);
                   movieItems.set(2, (Movie)o2);
               }
               return first;
           }
       });
    }
    return movieItems;
}

jpaController 正在带回 4 部电影并给我以下内容

java.lang.ArrayIndexOutOfBoundsException：数组索引超出范围：4 在 java.util.Vector.get(Vector.java:694) 处 entitybeans.jsf.PeliculaController$1.compare(PeliculaController.java:260) 在 java.util.Arrays.mergeSort(Arrays.java:1270) 处 java.util.Arrays.sort(Arrays.java:1210) 在 java.util.Collections.sort(Collections.java:159) at entitybeans.jsf.PeliculaController.getPeliculaItems(PeliculaController.java:257) 在 sun.reflect.NativeMethodAccessorImpl.invoke0(本机方法) 处 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) 在 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) 在 java.lang.reflect.Method.invoke(Method.java:597) 处 javax.el.BeanELResolver.getValue(BeanELResolver.java:302) 在 javax.el.CompositeELResolver.getValue(CompositeELResolver.java:175) 在 com.sun.faces.el.FacesCompositeELResolver.getValue(FacesCompositeELResolver.java:72) 在 com.sun.el.parser.AstValue.getValue(AstValue.java:116) 处 com.sun.el.parser.AstValue.getValue(AstValue.java:163)....

Answer 1

在您的 compare 方法中，o1 和 o2 已经是 movieItems 列表中的元素。所以，你应该这样做：

Collections.sort(movieItems, new Comparator<Movie>() {
    public int compare(Movie m1, Movie m2) {
        return m1.getDate().compareTo(m2.getDate());
    }
});

Answer 2

不要访问或修改Comparator 中的集合。比较器应该仅用于确定哪个对象位于另一个对象之前。要比较的两个对象作为参数提供。

Date 本身是可比较的，因此，使用泛型：

class MovieComparator implements Comparator<Movie> {
    public int compare(Movie m1, Movie m2) {
       //possibly check for nulls to avoid NullPointerException
       return m1.getDate().compareTo(m2.getDate());
    }
}

并且不要在每次排序时实例化比较器。使用：

private static final MovieComparator comparator = new MovieComparator();

Answer 3

在 Java 8+ 中，现在很简单：

movieItems.sort(Comparator.comparing(Movie::getDate));

Answer 4

您错误地使用了比较器。

 Collections.sort(movieItems, new Comparator<Movie>(){
           public int compare (Movie m1, Movie m2){
               return m1.getDate().compareTo(m2.getDate());
           }
       });

Answer 5

我会添加 Commons NullComparator 来避免一些问题......

Answer 6

你可以使用这个：

Collections.sort(list, org.joda.time.DateTimeComparator.getInstance());