convert jquery each function to pure javascript

Question

I have script to show dropdown in select box. The script currently i am using is

jQuery.each( dslr, function( index, dslrgp) {
    var aslrp= dslrgp.aslrp;
    jQuery.each( aslrp, function(index2, pslrp) {
        var found = 0;
        jQuery.each( dropdown, function(index3, dditem) {
            if (dditem.countryname == pslrp.countryname)
            {
                foundit = 1;
            }
        });
        if (foundit == 0)
            dropdown.push(pslrp);

    });
});

How can i convert this to pure javascript. Because if i am using this

dslr.forEach(function( index, dslrgp) {
    var aslrp= dslrgp.aslrp;
    aslrp.forEach(function(index2, pslrp) {
        var found = 0;
        dropdown.forEach(function(index3, dditem) {
            if (dditem.countryname == pslrp.countryname)
            {
                foundit = 1;
            }
        });
        if (foundit == 0)
            dropdown.push(pslrp);

    });
});

it is not working.

"it is not working" — Define "not working". Give a clear problem statement. How does the behaviour you get differ from the behaviour you expect? — Quentin
– Quentin, Commented Feb 10, 2017 at 10:37
"convert jquery each function to pure javascript" — jQuery is pure JavaScript. There are perfectly valid reasons to want to remove a dependency on jQuery, but don't mistake jQuery for being anything other than JavaScript. — Quentin
– Quentin, Commented Feb 10, 2017 at 10:38

Bartek Fryzowicz · Accepted Answer · 2017-02-10 10:45:14Z

2

Note the difference in order of arguments in native forEach - first is the value of item, second is index. So instead of:

aslrp.forEach(function(index2, pslrp) {
...
dropdown.forEach(function(index3, dditem) {

use this:

aslrp.forEach(function(pslrp, index2) {
...
dropdown.forEach(function(dditem,index3) {

edited Feb 10, 2017 at 10:45

answered Feb 10, 2017 at 10:38

Bartek Fryzowicz

6,70420 silver badges29 bronze badges

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

Comments

ManuKaracho · Accepted Answer · 2017-02-10 10:41:13Z

2

You are using the .forEach() method wrong. forEach docs

You don't need to pass the array in as first argument. Just pass the callback.

dslr.forEach(function(dslrgp) {
  // do something..
}

or with key / value iteration

dslr.forEach(function(value, index) {
  // do something..
}

answered Feb 10, 2017 at 10:41

ManuKaracho

1,2282 gold badges15 silver badges32 bronze badges

Comments

magritte · Accepted Answer · 2017-02-10 10:39:06Z

1

Your method signature is wrong. It's:

arr.forEach(function callback(currentValue, index, array) {
    //your iterator
}[, thisArg]);

See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach

answered Feb 10, 2017 at 10:39

magritte

7,71610 gold badges65 silver badges81 bronze badges

1 Comment

Upendra Sharma Over a year ago

what are arr, currentValue and array here ?

Collectives™ on Stack Overflow

convert jquery each function to pure javascript

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related