Send multidimensional Array via $.post

Question

I have a multi-dimensional array which I fill via $.each

$('.gBook').click(function(){
        var values = [];
        var valueToPush = { }; // or "var valueToPush = new Object();" which is the same
        var i = 0;
        $('td[data-check="true"]').each(function(){
                valueToPush["price"] = $(this).attr("data-price");
                valueToPush["id"] = $(this).attr("data-id");
                values.push(valueToPush);
                i++;
        });

        var arrayToSend = {values};

        $.post( '<?php echo PATH;?>ajax/updateRoom.php',values, function(data){
                if(data != "ERROR"){
                    $('#all-content').html(data).css("overflow-y","auto");
                }else{
                    alert("ERROR");
                }
        });
    });

But the outcome is the same for all POST vars:

Array (
  [values] => Array (
   [0] => Array ( [price] => 15 [id] => 3380 )
   [1] => Array ( [price] => 15 [id] => 3380 )
   [2] => Array ( [price] => 15 [id] => 3380 )
  ) 
)

Where is my mistake? It seems that "push" overrides everytime?

you should initialize the var valueToPush inside the each function, you are pushing the same object "pointer" in values each time the function is executed (and updating these values) — Hacketo
– Hacketo, Commented Feb 24, 2015 at 15:38
That's not a multidimensional array, that's an array of objects. — Bergi
– Bergi, Commented Feb 24, 2015 at 15:47

Quentin · Accepted Answer · 2015-02-24 15:41:45Z

4

This line:

var valueToPush = { };

is outside your loop.

So you:

create one object
put a reference to it in valueToPush
put values in it
copy the reference to it onto the array
go to 3

You only ever have one object, you just have multiple references to it in the array.

You need to create a new object each time you go around the loop.

$('td[data-check="true"]').each(function(){
    var valueToPush = {};
    valueToPush["price"] = $(this).attr("data-price");

answered Feb 24, 2015 at 15:41

Quentin

949k136 gold badges1.3k silver badges1.4k bronze badges

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

2 Comments

PrimuS Over a year ago

I understand that, but however this results in Array ( [undefined] => )

PrimuS Over a year ago

Got it, had to use $.post( '<?php echo PATH;?>ajax/updateRoom.php',[b]arrayToSend[/b], function(data){ instead of values...

Arthur · Accepted Answer · 2015-02-24 15:52:35Z

0

Your each loop should simply look like this:

$('td[data-check="true"]').each(function(){
     values.push({
        price: $(this).attr("data-price"),
        id: $(this).attr("data-id")
     });
 });

You need to insert a new object in each iteration. Currently you are inserting references to a single object to into your array.

answered Feb 24, 2015 at 15:52

Arthur

1,4431 gold badge18 silver badges35 bronze badges

1 Comment

PrimuS Over a year ago

also "undefined" as above, starting to believe it's something else?

Collectives™ on Stack Overflow

Send multidimensional Array via $.post

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related