jquery string comparison variable

Question

I have the following code:

arr = ["abc", "def", "ghi", "jkl"]
arr2 = ["abc", "def", "ghi", "jkl"]

$(arr).each(function(){
    thiselem = this;

    $(arr2).each(function(){
        if(thiselem == "abc" && this == "abc")
            alert("case 1");
        if(thiselem == this)
            alert('case 2');
    });
});

When I run this only "case 1" pops up. Logically this statement should be true by the transitive property, so I am guessing it is some JavaScript string syntactic issue or a jQuery scope thing is messing this up. Any advice is appreciated.

Scoping your variables will save your butt one day.

Diodeus - James MacFarlane
– Diodeus - James MacFarlane

2012-06-05 16:09:04 +00:00
Commented Jun 5, 2012 at 16:09 — Diodeus - James MacFarlane
– Diodeus - James MacFarlane, Commented Jun 5, 2012 at 16:09

georg · Accepted Answer · 2012-06-05 16:55:46Z

Other posters have suggested workarounds, I'll try to answer the question why your original code doesn't work. The answer is rather non-trivial and reveals some good-to-know javascript gotchas.

jQuery.each uses apply to pass this to its callback. When apply's argument is a primitive value, like string, it will be "boxed", i.e. converted to an object (specifically, the String object):

console.log(typeof("cat"))  // string

logger = function() { console.log(typeof(this)) }
logger.apply("cat")  // object

Now consider the following:

a = ["abc"]
b = ["abc"]

$(a).each(function() {
   var that = this
   $(b).each(function() {
       console.log(this == that) // false!
   })
})

Although a[0] and b[0] are "obviously" equal, the == operator returns false because both are objects, and two object variables are only equal if they are physically the same object. On the other side, this works as expected:

a = ["abc"]
b = ["abc"]

$(a).each(function() {
     console.log(this == "abc") // true
     console.log(this == b[0]) // true
})

When JS compares an object to a string, the object is converted to a string primitive using toString. Since this is a String object, its toString returns the primitive string it was made of, and two primitive strings are equal if their characters are equal.

Selvakumar Arumugam · Accepted Answer · 2012-06-05 16:10:03Z

1

Try like below,

var arr = ["abc", "def", "ghi", "jkl"]
var arr2 = ["abc", "def", "ghi", "jkl"]

$(arr).each(function(idx, el){
    $(arr2).each(function(idx, iEl){
        if(el == "abc" && iEl == "abc")
            alert("case 1");       
        if(el == iEl)
            alert('case 2');
    });
 });

Note: I assume the above is just a pseudo code. We can help better if you let us know what you are trying to do.

DEMO

answered Jun 5, 2012 at 16:10

Selvakumar Arumugam

79.9k15 gold badges123 silver badges139 bronze badges

Comments

thecodeparadox · Accepted Answer · 2012-06-05 16:11:31Z

0

arr = ["abc", "def", "ghi", "jkl"]
arr2 = ["abc", "def", "ghi", "jkl"]

$(arr).each(function(index, val1) {
    $(arr2).each(function(i, val2) {
        if (val1 == "abc" && val2 == "abc") alert("case 1");
        if (val1 == val2) alert('case 2');
    });
});

Sample workout

edited Jun 5, 2012 at 16:11

answered Jun 5, 2012 at 16:06

thecodeparadox

87.2k22 gold badges143 silver badges164 bronze badges

Comments

Nishu Tayal · Accepted Answer · 2012-06-05 16:24:36Z

0

Try this:

var arr = ["abc", "def", "ghi", "jkl"]
var arr2 = ["abc", "def", "ghi", "jkl"]

jQuery.each(arr,function(key1, val1){
  jQuery.each(arr2,function(key2, val2 ){
    if(val1 == "abc" && val2 == "abc")
        alert("case 1");       
    if(val1 == val2)
        alert('case 2');
   });
});

Here is Demo

answered Jun 5, 2012 at 16:24

Nishu Tayal

20.9k8 gold badges53 silver badges105 bronze badges

Collectives™ on Stack Overflow

jquery string comparison variable

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related