How add object to array jQuery

Question

I'm try add my objects to array, but on finish I have all identical objects

$('#Create').click(function(event) {
  event.preventDefault();

  var categoryId = $('#CatId').val();
  var enteredDate = $('#datepicker').val();
  var empId = $('#employeeID').text();
  var systemDate = $.datepicker.formatDate('dd.mm.yy', new Date());
  var obj = {
    CategoryId: categoryId,
    EnteredDate: enteredDate,
    SystemDate: systemDate,
    EmpId: empId
  };
  var arrToServer = [];
  var list = new Array();
  $("input[type=checkbox]:checked").each(function() {
    var productId = $(this).attr('id');
    obj.ProductId = productId;
    arrToServer.push(obj);                
  });
  arrToServer = JSON.stringify(arrToServer);
}

My arrToServer have 2 identical objetcs, why?

Here's a fiddle that may make the behavior clearer: jsfiddle.net/FcHpK — Jason P
– Jason P, Commented Feb 25, 2014 at 18:10

Satpal · Accepted Answer · 2014-02-25 18:10:01Z

4

create a copy of object then set project id.

$("input[type=checkbox]:checked").each(function () {
    var productId = $(this).attr('id');
    var clonedobj = jQuery.extend({}, obj); //create a shallow
    clonedobj.ProductId = productId;
    arrToServer.push(clonedobj);
});

edited Feb 25, 2014 at 18:10

answered Feb 25, 2014 at 18:04

Satpal

134k13 gold badges168 silver badges171 bronze badges

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

Comments

Felix Kling · Accepted Answer · 2014-02-25 18:13:18Z

3

My arrToServer have 2 identical objetcs, why?

Because you are only creating one object, outside the loop, and push the same object to the array multiple times. Passing an object to a function or assigning it to a different variable does not create a copy of the object.

Create the object inside the loop (and with loop I mean the .each callback).

Here is an example using .map, which is much cleaner IMO:

var arrToServer = $("input[type=checkbox]:checked").map(function() {
    return {
        CategoryId: categoryId,
        EnteredDate: enteredDate,
        SystemDate: systemDate,
        EmpId: empId,
        ProductId: $(this).attr('id')
    };
}).get();

edited Feb 25, 2014 at 18:13

answered Feb 25, 2014 at 18:08

Felix Kling

820k181 gold badges1.1k silver badges1.2k bronze badges

3 Comments

SpaceNinja Over a year ago

Can you please explain why there is a .get() at the end of your example?

Felix Kling Over a year ago

.get returns an array. Without it, you would be working with a jQuery object.

SpaceNinja Over a year ago

Ah thank you. When I searched, I saw this which referred to Ajax: api.jquery.com/jquery.get. Thanks!

Collectives™ on Stack Overflow

How add object to array jQuery

2 Answers 2

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related