jQuery get data from each loop and store in global variable

Question

I am writing some code to grab product id's and quantities before adding to my cart. I am looping through each product where the quantity is greater than 0, and then getting some information stored in the data attributes.

The issue i am having is outputting all of the data. here is my code:

jQuery( ".productList .catQuantity" ).each(function( index ) {
    var productID = jQuery(this).attr('data-product_id');
    var productQu = jQuery(this).attr('data-quantity');   
    if(productQu > 0){
        //console.log(productID+':'+productQu+',');
        window.allProducts = (productID+':'+productQu+',');
    }
});
console.log(allProducts);

So the above grabs the product id and quantity and stores it in a global variable. When i console.log the variable, only the last product added is included in the variable.

Any ideas why this is?

Zakaria Acharki · Accepted Answer · 2017-11-10 15:29:17Z

2

You should append them to the variable instead using +=, else the variable allProducts will be overwritten in every iteration of the loop :

window.allProducts += (productID+':'+productQu+',');

Hope this helps.

answered Nov 10, 2017 at 15:29

Zakaria Acharki

67.5k15 gold badges79 silver badges106 bronze badges

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

Comments

user6748331 · Accepted Answer · 2017-11-10 15:37:20Z

0

Use array for storing all products, as you are appending unnecessary comma after last item. Then, if you need, just convert / show this array as string:

var allProducts = []

jQuery( ".productList .catQuantity" ).each(function( index ) {
  var productID = jQuery(this).attr('data-product_id')
  var productQu = jQuery(this).attr('data-quantity')  
  if (productQu > 0) {
    window.allProducts.push(productID + ':' + productQu)
  }
})

console.log(allProducts.toString())

answered Nov 10, 2017 at 15:37

user6748331

Collectives™ on Stack Overflow

jQuery get data from each loop and store in global variable

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related