Jquery sum on input boxes

Question

I am using following html form code...

<li><input type="text" class="paytax" name="payment[tax][]" /> <input type="text" class="payship" name="payment[shiping][]" /> <input type="text" class="paytotal" name="payment[total][]" /></li>
<li><input type="text" class="paytax" name="payment[tax][]" /> <input type="text" class="payship" name="payment[shiping][]" /> <input type="text" class="paytotal" name="payment[total][]" /></li>
<li><input type="text" class="paytax" name="payment[tax][]" /> <input type="text" class="payship" name="payment[shiping][]" /> <input type="text" class="paytotal" name="payment[total][]" /></li>
<li><input type="text" class="paytax" name="payment[tax][]" /> <input type="text" class="payship" name="payment[shiping][]" /> <input type="text" class="paytotal" name="payment[total][]" /></li>

I am trying to get total of .paytax and .paytax ($('.paytax').val()+$('.payship').val();) but its adding total of first li input to each .paytotal, please help me with proper code.

thanks for support.

jessegavin · Accepted Answer · 2011-04-19 19:48:37Z

2

My assumption (without seeing any javascript code) is that you're trying to show the total in each line item. Tax + Shipping = Total. This code accomplishes that.

$(".paytax,.payship").live("keyup", function() {
    var $li = $(this).closest("li");
    var tax = parseFloat($li.find(".paytax").val());
    var ship = parseFloat($li.find(".payship").val());
    if(isNaN(tax)) tax = 0;
    if(isNaN(ship)) ship = 0;
    $li.find(".paytotal").val(tax + ship);
});

edited Apr 19, 2011 at 19:48

answered Apr 19, 2011 at 19:39

jessegavin

75.8k28 gold badges141 silver badges165 bronze badges

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

5 Comments

seoppc Over a year ago

its not working, i think you have forgot to add .keyup or change function on input boxes.

Adam Terlson Over a year ago

He didn't forget anything. You forgot to actually say that that's what you wanted. Please be more clear in the future if you expect answers to be accurate to your requirements.

jessegavin Over a year ago

I think you forgot to mention that you wanted it on keyup!

seoppc Over a year ago

thats right, i am sorry if i couldn't provide proper information, what i want to do is, i am appending these fields using jquery and user can create upto 10 fields and each <li> total should be different according to passed value of .paytax and .payship

jessegavin Over a year ago

My answer should work fine for you. If you're going to ask people for help, PLEASE take the time to write a proper question with all the relevant information.

Adam Terlson · Accepted Answer · 2011-04-19 19:43:38Z

2

var total = 0;
$('input.paytax, input.payship').each(function() {
    total += parseFloat($(this).val());
});

This will total all the inputs of either of those classes.

UPDATE:

I see now what you're trying to do. You want line-by-line totals... Missed that. Go with the other posted answer (Jesse Gavin's). Mine will total the entire thing.

edited Apr 19, 2011 at 19:43

answered Apr 19, 2011 at 19:38

Adam Terlson

12.7k4 gold badges46 silver badges63 bronze badges

4 Comments

seoppc Over a year ago

would it total .paytax and .payship?

Adam Terlson Over a year ago

Wouldn't what be ".paytax" and ".payship"? Oh, you want to tally both? See update.

seoppc Over a year ago

are we still missing keyup or change function on inputs?

Adam Terlson Over a year ago

Your question made no mention of either of those things. See my comment on Jessegavin's post.

Collectives™ on Stack Overflow

Jquery sum on input boxes

2 Answers 2

5 Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related