Question

我正在尝试使用下面的代码按日期/时间（最新的第一个）订购日期对象。订购不正确。我认为compareTo方法需要以不同方式实现以实现所需的排序？

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


public class DateSorter {

    public static void main(String args[]){

        List<DateObject> list = new ArrayList<DateObject>();

        DateObject d1 = new DateObject("2012-12-05" , "11:21:19");
        list.add(d1);

        d1 = new DateObject("2012-12-05" , "11:20:19");
        list.add(d1);

        d1 = new DateObject("2012-12-05" , "11:20:19");
        list.add(d1);

        d1 = new DateObject("2012-12-04" , "10:20:19");
        list.add(d1);

        d1 = new DateObject("2010-12-07" , "13:20:19");
        list.add(d1);

        d1 = new DateObject("2012-12-05" , "11:20:19");
        list.add(d1);

        Collections.sort(list);

        for(DateObject d : list){
            System.out.println(d);
        }
    }

}

import java.text.DateFormat;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Date;

public class DateObject implements Comparable<Object> {

    private String date;
    private String time;

    public DateObject(String date, String time) {
        this.date = date;
        this.time = time;
    }

    public int compareTo(Object o) {

        DateFormat formatter;
        Date date1 = null;
        Date date2 = null;
        formatter = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
        try {
            date1 = (Date) formatter.parse(this.date + " " + this.time);
            date2 = (Date) formatter.parse(this.date + " " + this.time);
        } catch (ParseException e) {
            e.printStackTrace();
        }
        catch(NullPointerException npe){
            System.out.println("Exception thrown "+npe.getMessage()+" date1 is "+date1+" date2 is "+date2);
        }

         return date1.compareTo(date2);

    }

    @Override
    public String toString(){       
        return this.date+" "+this.time;
    }

}

这是显示的输出：

2012-12-05 11:21:19
2012-12-05 11:20:19
2012-12-05 11:20:19
2012-12-04 10:20:19
2010-12-07 13:20:19
2012-12-05 11:20:19

此输出应为：

2010-12-07 13:20:19
2012-12-05 11:20:19
2012-12-05 11:20:19
2012-12-05 11:20:19
2012-12-05 11:21:19
2012-12-04 10:20:19

更新：当我将compareTo方法修改为：

时

public int compareTo(Object o) {


    DateFormat formatter;
    Date date1 = null;
    Date date2 = null;  
    DateObject other = (DateObject) o;

    formatter = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
    try {
        date1 = (Date) formatter.parse(this.date + " " + this.time);
        date2 = (Date) formatter.parse(other.date + " " + other.time);
    } catch (ParseException e) {
        e.printStackTrace();
    }
    catch(NullPointerException npe){
        System.out.println("Exception thrown "+npe.getMessage()+" date1 is "+date1+" date2 is "+date2);
    }

     return date1.compareTo(date2);

}

输出结果为：

2010-12-07 13:20:19
2012-12-04 10:20:19
2012-12-05 11:20:19
2012-12-05 11:20:19
2012-12-05 11:20:19
2012-12-05 11:21:19

2012-12-05之前应该出现哪个仍然不正确

Answer 1

您正在解析相同的日期两次：

date1 = (Date) formatter.parse(this.date + " " + this.time);
date2 = (Date) formatter.parse(this.date + " " + this.time);

应该是：

date1 = (Date) formatter.parse(this.date + " " + this.time);
DateObject other = (DateObject) o;
date2 = (Date) formatter.parse(other.date + " " + other.time);

你应该测试o是一个DateObject而不是null，然后再尝试使用它并使用它。

修改

刚刚使用return date2.compareTo(date1);尝试了更新后的代码，我得到了：

2012-12-05 11:21:19 2012-12-05 11:20:19 2012-12-05 11:20:19 2012-12-05 11:20:19 2012-12-04 10:20:19 2010-12-07 13:20:19

按降序排列（请注意，最后一个日期是2010年）。

Answer 2

您将日期与自身进行比较，而不是

中传递的对象

 date1 = (Date) formatter.parse(this.date + " " + this.time);
 date2 = (Date) formatter.parse(this.date + " " + this.time);

使用o的地方？

DateObject给你什么，顺便说一下？它看起来像Date个对象。请注意java.util.Date容纳日期和时间信息，令人困惑。如果使用标准日期对象，则不必编写比较器。更好的是，您可以调查Joda-Time，它具有更好，更直观和线程安全的API。

Answer 3

我认为错误是：

 date2 = (Date) formatter.parse(this.date + " " + this.time);

应该是

 date2 = (Date) formatter.parse(((DateObject)o).date + " " + ((DateObject)o).time);

你也可以这样做摆脱演员：

public class DateObject implements Comparable<DateObject> {
...
public int compareTo(DateObject o) {
    date2 = (Date) formatter.parse(o.date + " " + (o.time);

Answer 4

你应该这样做：

DateObject other = (DateObject) o;

date1 = (Date) formatter.parse(this.date + " " + this.time);
date2 = (Date) formatter.parse(other.date + " " + other.time);

date2.compareTo(date1);

注意最后一行。你说你想要最新成为第一个。在这种情况下，您应该将date2与date1进行比较，反之亦然，因为date1.compareTo（date2）将按升序排序。

Answer 5

DateObject应该实现Comparable。在compareTo方法中，date2需要从传递给方法的对象派生。

排序日期/时间对象（最新的）

5 个答案: