How to compare item of set collection to current Date?

Question

I have the following set and need to compare its date instance with the current date. Although both dates are the same but the comparison returns false !!

MyClass.java

import java.util.Date;
public class MyClass {
   private Date date;

   ...

}

My Code

 ....
 Set <MyClass> myclass = new HashSet();

 I populate it with some data here...

 for(MyClass m : myclass)
 {
   System.err.println("date>>:" + trim(m.getDate()));  //returns 2013-08-08
   System.err.println("date>>:" + trim(getCurrentDate()));  //returns 2013-08-08
   System.err.println("boolean:" +                            
               trim(m.getDate()).equals(trim(getCurrentDate()))); //returns false
 }
}

 public Date getCurrentDate() {
    SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd");
    Date date = new Date();
    dateFormat.format(date));
    return date;
}

public Date trim(Date date){
    Calendar calendar = Calendar.getInstance();
    calendar.setTime(date);
    calendar.set(Calendar.MILLISECOND, 0);
    calendar.set(Calendar.SECOND, 0);
    calendar.set(Calendar.MINUTE, 0);
    calendar.set(Calendar.HOUR, 0);
    return calendar.getTime();
}

Joni · Answer 1 · 2013-08-08T06:44:01.133

1

The Date class includes the time of the day to millisecond precision, and the time counts when comparing for equality.

To compare only the "date" part you can do one of several things, for example format the dates as year-month-day and compare the resulting strings, or create Calendar objects from the dates and compare the year, month and day individually. Another option is to make sure the Dates you compare have the same hour of the day, for example 12:00, that way you can use the equals method.

edited Aug 08 '13 at 06:44

answered Aug 08 '13 at 06:38

Joni

108,737
14
143
193

1

so how to solve the problem and compare them based on date only ? – Tim Norman Aug 08 '13 at 06:39
I am interested in year-month-day solution but do not know how to implement that do you have any sample ? – Tim Norman Aug 08 '13 at 06:45

score 1 · Answer 2 · edited May 23 '17 at 10:31

1

Each time you call getCurrentDate, you might receive a new date. Formatting it the way you do is essentially a no-op and the date still carries its hours, minutes, seconds and milliseconds.

So they are actually proably different for real.

You could remove this extra information to get the desired behaviour.

edited May 23 '17 at 10:31

Community

1
1

answered Aug 08 '13 at 06:39

assylias

321,522
82
660
783

@TimNorman Try printing `date.getTime()` of both to see if they return the same value (they probably don't). – assylias Aug 08 '13 at 07:02
I got you but even after using the trim method (provided in the question) their comparison returns false. – Tim Norman Aug 08 '13 at 07:06

harsh · Accepted Answer · 2013-08-08T07:00:05.153

Dates are not same, they may differ by millis/sec. Date equals doesn't depend upon format of date but compares value. Below code would return false as well:

Date d1 = new Date();
        SimpleDateFormat f = new SimpleDateFormat("yyyy-mm-dd");
        Date d2 = new Date();
        f.format(d2);
        System.out.println(d1);//e.g. Thu Aug 08 12:09:24 IST 2013
        System.out.println(d2);//e.g. Thu Aug 08 12:09:26 IST 2013
        System.out.println(d1.equals(d2));//false

Date.equals compares time (Date.getTime()), equals will return true only if they matches:

public boolean equals(Object obj) {
        return obj instanceof Date && getTime() == ((Date) obj).getTime();
    }

Per javadoc:

 The result is true if and only if the argument is not null and is a Date object that represents the same point in time, to the millisecond, as this object. 

Thus, two Date objects are equal if and only if the getTime method returns the same long value for both.

Date.getTime return the number of milliseconds since January 1, 1970, 00:00:00 GMT

So in you updated question with trim, consider you are comparing two long values of time in millis.

If you require to compare yyyy-MM-dd values of two different date instances, consider using String.equals instead (hack way):

SimpleDateFormat f = new SimpleDateFormat("yyyy-MM-dd");
        String date1 = f.format(new Date());//2013-08-08
        String date2 = f.format(new Date());//2013-08-08
        System.out.println(date1.equals(date2));

score 1 · Answer 4 · answered Aug 08 '13 at 06:46

You can use GregorianCalendar and Calendar#get(..) to only compare year, month, and day.
There is a perfect sample from the javadoc :

 // get the supported ids for GMT-08:00 (Pacific Standard Time)
 String[] ids = TimeZone.getAvailableIDs(-8 * 60 * 60 * 1000);
 // if no ids were returned, something is wrong. get out.
 if (ids.length == 0)
     System.exit(0);

  // begin output
 System.out.println("Current Time");

 // create a Pacific Standard Time time zone
 SimpleTimeZone pdt = new SimpleTimeZone(-8 * 60 * 60 * 1000, ids[0]);

 // set up rules for Daylight Saving Time
 pdt.setStartRule(Calendar.APRIL, 1, Calendar.SUNDAY, 2 * 60 * 60 * 1000);
 pdt.setEndRule(Calendar.OCTOBER, -1, Calendar.SUNDAY, 2 * 60 * 60 * 1000);

 // create a GregorianCalendar with the Pacific Daylight time zone
 // and the current date and time
 Calendar calendar = new GregorianCalendar(pdt);
 Date trialTime = new Date();
 calendar.setTime(trialTime);

 // print out a bunch of interesting things
 System.out.println("ERA: " + calendar.get(Calendar.ERA));
 System.out.println("YEAR: " + calendar.get(Calendar.YEAR));
 System.out.println("MONTH: " + calendar.get(Calendar.MONTH));
 System.out.println("WEEK_OF_YEAR: " + calendar.get(Calendar.WEEK_OF_YEAR));
 System.out.println("WEEK_OF_MONTH: " + calendar.get(Calendar.WEEK_OF_MONTH));
 System.out.println("DATE: " + calendar.get(Calendar.DATE));
 System.out.println("DAY_OF_MONTH: " + calendar.get(Calendar.DAY_OF_MONTH));
 System.out.println("DAY_OF_YEAR: " + calendar.get(Calendar.DAY_OF_YEAR));
 System.out.println("DAY_OF_WEEK: " + calendar.get(Calendar.DAY_OF_WEEK));
 System.out.println("DAY_OF_WEEK_IN_MONTH: "
                    + calendar.get(Calendar.DAY_OF_WEEK_IN_MONTH));
 System.out.println("AM_PM: " + calendar.get(Calendar.AM_PM));
 System.out.println("HOUR: " + calendar.get(Calendar.HOUR));
 System.out.println("HOUR_OF_DAY: " + calendar.get(Calendar.HOUR_OF_DAY));
 System.out.println("MINUTE: " + calendar.get(Calendar.MINUTE));
 System.out.println("SECOND: " + calendar.get(Calendar.SECOND));
 System.out.println("MILLISECOND: " + calendar.get(Calendar.MILLISECOND));
 System.out.println("ZONE_OFFSET: "
                    + (calendar.get(Calendar.ZONE_OFFSET)/(60*60*1000)));
 System.out.println("DST_OFFSET: "
                    + (calendar.get(Calendar.DST_OFFSET)/(60*60*1000)));

You don't have to convert `Date` to `SimpleTimeZone`, your `Date` will replace the `Date trialTime` :) — NiziL, Aug 08 '13 at 07:01
I am wondering if I replace this code with the body of my trim method how to return the date after formatting it ? — Tim Norman, Aug 08 '13 at 07:05

Deepu--Java · Answer 5 · 2013-08-12T12:23:24.313

1

Actually there is a problem in your method..

 public Date getCurrentDate() {
    SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd");
    Date date = new Date();
    dateFormat.format(date);
    return date;
}

dateFormat.format(date) will return a String date in yyyy-MM-dd format but you are returning date from this method which will return the Date in 'Thu Aug 08 12:21:34 IST 2013' this format not in '2013-08-08' this. So you should take the String as return from this method and then compare it by equals. Try this, I think this should help you.

edited Aug 12 '13 at 12:23

answered Aug 08 '13 at 06:58

Deepu--Java

3,742
3
19
30

what do you suggest ? would you provide me with a sample code, I do not get you – Tim Norman Aug 08 '13 at 07:01
Just see your getCurrentDate() method, you are returning Date from this method, just put a System.out.print() before 'return date;' and see the format you are returning.. – Deepu--Java Aug 08 '13 at 07:15
public String getCurrentDate() { SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd"); Date date = new Date(); String dt=dateFormat.format(date); return dt; } Try this method.. – Deepu--Java Aug 08 '13 at 07:17
Tell me if it helps you... :) – Deepu--Java Aug 08 '13 at 07:20
no I had to trim the date, I have accepted the correct answer – Tim Norman Aug 08 '13 at 23:51

How to compare item of set collection to current Date?

5 Answers5