jQuery.inArray not behaving as expected

Question

I'm trying to loop through a list of numbers and check if that number is part of another list of numbers using jQuery.each and jQuery.inArray. jQuery.inArray does not seem to be behaving as expected.

Here is my code:

var some_numbers = [1, 2];
var more_numbers = [0, 1, 2];

$.each(more_numbers, function(index, value) {
  if($.inArray(value, some_numbers)) {
    console.log(value);
  }
});

console.log('Some Numbers:');
console.log(some_numbers);

Here is the resulting console output:

0
2
Some Numbers:
[1, 2]

Will someone please help? This is maddening.

Edit: Problem solved! Changed my condition to this:

if($.inArray(value, some_numbers) !== -1)

Thanks everyone!

I can't accept the answer for another 7 minutes, according to the dialog that appeared when I attempted to do so. — Jon Mooring
– Jon Mooring, Commented Nov 29, 2010 at 14:19
Yes, but you should accept answers to your other questions. EDIT Looking through them, maybe you shouldn't; they don't have great answers. — SLaks
– SLaks, Commented Nov 29, 2010 at 14:21
Yeah, I just went back through and accepted what I felt qualified as a decent answer or at least what was closest to the solution that I ended up going with. Thanks again for your quick response to this question. — Jon Mooring
– Jon Mooring, Commented Nov 29, 2010 at 14:27

SLaks · Accepted Answer · 2010-11-29 14:14:01Z

3

$.inArray returns an index, not a boolean.

To check whether the element was found, you need to check whether it's >= 0.

answered Nov 29, 2010 at 14:14

SLaks

891k182 gold badges1.9k silver badges2k bronze badges

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

1 Comment

Jon Mooring Over a year ago

Beautiful. Added to the condition "!== -1" and this solved my problem. Thank you!

Collectives™ on Stack Overflow

jQuery.inArray not behaving as expected

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related