CODEWITHSUNDEEP
phpmysqlhtmljquery
MGB
MGB

Reputation: 37

jQuery AJAX POST to mysql table for dynamic table data- Am I doing this in anything like the right way?

I have a table which I can dynamically add and delete any number of rows to. Once the data is all entered by the user I am using the jQuery AJAX functionality to POST it to a mysql database so there is no page redirect or refresh.

The only way I could think of getting it to work was using this for loop in my jQuery, effectively posting each row to the database separately. How dumb is this? Should I be getting all the table data and posting it once? There could be any number of rows varying on user and the user could add and delete rows as much as he wants before submitting.

The strange i variable counting is due to there being two th rows at the top of the table. I couldn't work out a smart way of doing this.

I was a bit paranoid about the data always being associated with the correct row.

Thankyou for your time. 

jQuery(function() {  
  jQuery(".button1").click(function() {  
    // process form here
    var rowCount = jQuery('#dataTable tr').length;

    for (var i = 2; i < rowCount; i++){
    // the four elements of each row I am storing to the mysql
    var txtRow1 = jQuery('#txtRow' + (i-1)).val();
    var tickerRow1 = jQuery('#tickerRow' + (i-1)).val();
    var quantRow1 = jQuery('#quantRow' + (i-1)).val();
    var valueRow1 = jQuery('#valueRow' + (i-1)).val();

    var dataString = 'txtRow1='+ txtRow1 + '&tickerRow1=' + tickerRow1 + '&quantRow1=' + quantRow1 + '&valueRow1=' + valueRow1;  
    //alert (dataString);return false;  
    jQuery.ajax({  
    type: "POST",  
    url: "http://rccl.co.uk/form_action1.php",  
    data: dataString
    });  
    };
return false;  
});  
});

Upvotes: 0

Views: 1424

Answers (2)

Elf Sternberg
Elf Sternberg

Reputation: 16381

It looks very clearly to me as if you have a well-established index of the row in question, using your variable i. Most form handlers server-side will unpack repeated keys of the form into a list, for stuff like many checkboxes with the same name. You could exploit that here.

datastring = '';
for(var i=2; i<rowCount; i++) {
    var txtRow1 = jQuery('#txtRow' + (i-1)).val();
    var tickerRow1 = jQuery('#tickerRow' + (i-1)).val();
    var quantRow1 = jQuery('#quantRow' + (i-1)).val();
    var valueRow1 = jQuery('#valueRow' + (i-1)).val();
    dataString = datastring + 'index[]=' + (i-1) + 'txtRow1[]='+ txtRow1 + '&tickerRow1[]=' + tickerRow1 + '&quantRow1[]=' + quantRow1 + '&valueRow1[]=' + valueRow1;
}

Then make the ajax call with the whole string.

On the server-side, you should get arrays for each of these, the first of each of which corresponds to the first row, the second of each of which corresponds to the second row, and so on.

It's been a long time since I used PHP, but I believe the [] symbols for each key item are necessary to clue PHP's $_POST that it should convert the various keys' contents into arrays.

Upvotes: 1

Francis Yaconiello
Francis Yaconiello

Reputation: 10939

I would try posting it all at once.

Reasons

  1. fewer calls, less chance of your message getting lost in transit (your server rejecting requests because of flood)
  2. you don't have to worry about requests responding out of order (maybe not an issue, but having an ass load of jumbled responses could potentially be an issue)
  3. it will be faster for large numbers of rows getting saved at once

Upvotes: 0

Related Questions