jQuery $.inArray() not working properly with array made with jQuery makeArray()

Question

I created an date array using this:

 var holidays = ["7/24/2010","7/25/2010"];
 var holidaysArray = jQuery.makeArray(holidays);

and then testing to see if myDate (a date object) exists in the array:

if ($.inArray(myDate, holidaysArray ) == -1) {....}

However the test always return -1 even though myDate is one of the two days. I was trying to avoid using date strings to do the test.

How can I use inArray function with date objects and array of date objects? (I am not sure if holidaysArray is actually an array of date objects and maybe that's why the test is failing.)

Wondering why you are using jQuery.makeArray() when holidays is already an actual Array. — user113716
– user113716, Commented Jul 23, 2010 at 22:02
Patrick is right, you use makeArray for things that look like an array but are not (like the arguments variable inside a function) — Pablo Fernandez
– Pablo Fernandez, Commented Jul 23, 2010 at 22:06
My intention was to convert an array of strings to an array of date objects. I am not sure how to use makeArray to accomplish this. myDate is a date object. It's not a string. I assume inArray() works on the same type for the two arguments. — Tony_Henrich
– Tony_Henrich, Commented Jul 23, 2010 at 22:17

Pablo Fernandez · Accepted Answer · 2010-07-23 22:14:13Z

4

I think you are comparing Date and String objects, that's why you'll get always false.

see:

new Date("12/12/2000") == "12/12/2000" // this is false

EDIT:

Also! note that:

new Date("12/12/2000") == new Date("12/12/2000") // this is false too!

You should compare dates using their epoch time value like this

new Date("12/12/2000").valueOf() == new Date("12/12/2000").valueOf() // this is TRUE

edited Jul 23, 2010 at 22:14

answered Jul 23, 2010 at 21:56

Pablo Fernandez

106k60 gold badges196 silver badges234 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

nicolaskruchten · Accepted Answer · 2010-07-23 22:20:21Z

0

To summarize others' answers:

You already have an array, you don't need to use makeArray().
Your array doesn't contain dates, it contains strings.
Dates can't be compared directly in JavaScript.

answered Jul 23, 2010 at 22:20

nicolaskruchten

27.7k8 gold badges91 silver badges103 bronze badges

Comments

Dan Stocker · Accepted Answer · 2010-07-23 22:51:11Z

0

Basically you want to perform a search (lookup), right?

If you formulate your array like this:

var holidays = [{ date: "7/24/2010" }, { date: "7/25/2010" }];

And create a jOrder table out of it, indexed by date (unique):

var table = jOrder(holidays)
    .index('date', ['date']);

Then you can check the presence of a certain date in the table like this:

if (!table.where([{ date: myDate }]).length) {....}

answered Jul 23, 2010 at 22:51

Dan Stocker

7027 silver badges5 bronze badges

Collectives™ on Stack Overflow

jQuery $.inArray() not working properly with array made with jQuery makeArray()

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related