adding two variables together

Question

I have been trying for... about 4 hours now lmao.

currentCalc returns 50 currentSum returns 0 when i alert them. Yet I cannot add them together with parseInt????

what am i doing wrong :'(

var identRow = $('tr.identRow');
identRow.each(function () {
    var getIdentClass = $(this).attr('class').split(' ').slice(1);
    $('tr.ohp' + getIdentClass + ' td.EURm').each(function (index) {
        var currentCalc = parseInt($(this).text().replace('.', ''), 10);
        var currentSum = $('tr.' + getIdentClass + ' td.totalEURm', this).text().replace('.', '');
        total = parseInt(currentCalc, 10) + parseInt(currentSum, 10);
        $('tr.' + getIdentClass + ' td.totalEURm').text(total);
        if (index == 6) {
            alert(total);
        }
    });
});

EDIT:

Oh goodness. Im completely confused now. I putr the break there. It says total = 50.

I want each iteration to add itself to the total. That is why I add currentCalc to the text of the field im plopping the currentCalc into.

$('tr.' + getIdentClass + ' td.totalEURm').text(total);

with my code now like this:

    var identRow = $('tr.identRow');
    identRow.each(function () {
      var getIdentClass = $(this).attr('class').split(' ').slice(1);
      $('tr.ohp' + getIdentClass + ' td.EURm').each(
        function (index) {
          var currentCalc = parseInt($(this).text().replace('.', ''), 10) || 0;
          var currentSum  = parseInt($('tr.' + getIdentClass + ' td.totalEURm', this).text().replace('.', ''), 10) || 0;
          var total = currentCalc + currentSum;
          $('tr.' + getIdentClass + ' td.totalEURm').text(total);
          if (index === 6) {
            alert(total);
          }
        });
    });

it alerts: 50, then 0, then 50, then 0.

EDIT:

How do I add currentCalc to its last value?

So first iteration its 10, seconds its 20. How do i make it so on the 2nd iteration it equals 30. currentCalc++ is just adding 1 to it.

Now you understand how crap i am :)

Answer 1

I am no expert in JS, but I saw that currentCalc is already an int:

var currentCalc = parseInt($(this).text().replace('.',''), 10);
//...
total = parseInt(currentCalc, 10) + parseInt(currentSum, 10);

so probably the parseInt on an int instead that on a string fails (?)

Answer 2

I do some code formatting and cleanup, try this:

var identRow = $('tr.identRow');
identRow.each(function () {
  var getIdentClass = $(this).attr('class').split(' ').slice(1);
  $('tr.ohp' + getIdentClass + ' td.EURm').each(
    function (index) {
      var currentCalc = parseInt($(this).text().replace('.', ''), 10) || 0;
      var currentSum  = parseInt($('tr.' + getIdentClass + ' td.totalEURm', this).text().replace('.', ''), 10) || 0;
      var total = currentCalc + currentSum;
      $('tr.' + getIdentClass + ' td.totalEURm').text(total);
      if (index === 6) {
        alert(total);
      }
    });
});

I added condition if parseInt fails the vars currentCalc and currentSum will be 0. Also, like in answer above i'm avoiding double parseInt

Can you give an example html page to try out?

Answer 3

If you get two alerts, that likely means either your outer or inner .each statements is matching two entries.

If you're using firebug, use console.debug(total); instead of alert(). I recommend using console.debug(this) at some point to make sure it has what you think it has, too. Put it above the alert(). That information would be useful to see.

Answer 4

    //SUM OF COLUMNS
    var total = 0;
    var identRow = $('tr.identRow');
    identRow.each(function () {
      var getIdentClass = $(this).attr('class').split(' ').slice(1);
      $('tr.ohp' + getIdentClass + ' td.EURm').each(
        function (index) {
            var currentCalc = $(this).text().replace('.', '');
            total = parseInt(currentCalc)+total;
            $('tr.' + getIdentClass + ' td.totalEURm').text(total);
        });
    });

did the trick.

Now i just gotta set the total to 0 when it gets to the second category because at the moment it keeps adding from where it left off. Progress though. Thanks for everything