Adding values of multiple text box values

Question

I have multiple text boxes with a class name .paymentamount.i want to add all values of text boxes of the same class but i am getting error nan.Code is this

function paymentData()
{
    $('.paymentamount').keyup(function(){
        var amountSum=0;
        $('.paymentamount').each(function(){
            amountSum+=parseInt(this.value);
        //console.log(this.value);
        });
        console.log(amountSum);

        });
}

suggest modifying amountSum+=parseInt(this.value); as amountSum+=parseInt($(this).val()); — optimusprime619
– optimusprime619, Commented May 15, 2012 at 8:07

Pranay Rana · Accepted Answer · 2012-05-15 08:10:08Z

2

   1. $.trim($(this).val())
   2. make use of $(this).val()
   3. check before adding var match = /^[0-9]$/.test(value);

Final code for you after this modification to your code

 $('.paymentamount').each(function(){
           var value = $.trim($(this).val()); 
           var match = /^[0-9]$/.test(value); 
           if(match)
             amountSum+=parseInt(value);
         //console.log(this.value);
         });

edited May 15, 2012 at 8:10

answered May 15, 2012 at 8:04

Pranay Rana

177k37 gold badges244 silver badges266 bronze badges

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

Comments

Tats_innit · Accepted Answer · 2012-05-15 08:21:02Z

1

Hiya Demo http://jsfiddle.net/MaK2k/1/ or using $.isNumeric http://jsfiddle.net/AKnvr/

**Code**


$('.paymentamount').keyup(function(){
   var amountSum=0;
    $('.paymentamount').each(function(){
        if (this.value != "")
           amountSum+=parseInt(this.value);

    });

    $(".paymentamountlable").text(amountSum);


    });

edited May 15, 2012 at 8:21

answered May 15, 2012 at 8:09

Tats_innit

34.1k10 gold badges74 silver badges77 bronze badges

1 Comment

Tats_innit Over a year ago

No probs, @Hira - Glad it helped you can use; extra .isNumeric and isNan if you want to make sure that it only sum up the numeric values! :) Another demo here: jsfiddle.net/AKnvr

Thulasiram · Accepted Answer · 2012-05-15 08:09:52Z

1

function paymentData()
{
    $('.paymentamount').keyup(function(){
        var amountSum=0;
        $('.paymentamount').each(function(){
            amountSum+=parseFloat($.trim($(this).val()));

        });
        console.log(amountSum);

        });
}

answered May 15, 2012 at 8:09

Thulasiram

8,5628 gold badges48 silver badges54 bronze badges

Comments

yogeswaran K · Accepted Answer · 2012-05-15 08:20:25Z

0

$('.paymentamount').keyup(function(){
    var amountSum=0;
    $('.paymentamount').each(function(){
        if(parseInt(this.value) > 0)
        {
        amountSum = parseInt(this.value) + parseInt(amountSum);
        }
    }); 

 console.log(amountSum);
});



Try this

answered May 15, 2012 at 8:20

yogeswaran K

2,2988 gold badges29 silver badges45 bronze badges

Comments

bluish · Accepted Answer · 2012-05-15 14:04:45Z

-1

You should try:

amountSum+=parseInt($(this).val());

edited May 15, 2012 at 14:04

bluish

27.5k28 gold badges126 silver badges185 bronze badges

answered May 15, 2012 at 8:08

jbduzan

1,1262 gold badges14 silver badges30 bronze badges

1 Comment

Spudley Over a year ago

-1 because parseInt() can return NaN if the value is not numeric, and also because you missed the radix argument for parseInt(). This argument is optional, but should be specified, because otherwise it can default to octal, which can give some unexpected results.

Collectives™ on Stack Overflow

Adding values of multiple text box values

5 Answers 5

Comments

1 Comment

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

1 Comment

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related