jquery adding input val together

Question

For some reason my code returns "0111" when the returned value should be 3 adding the numbers together.

Is there a better way to write this so it adds the value of the input text?

var p = $(".p").val(); 
var r = $(".r").val();
var d = $(".d").val();
var s = $(".s").val();

var checkability = p + r + d + s;

alert(checkability)

PSL · Accepted Answer · 2013-09-25 16:55:03Z

3

You are concatenating strings you need to cast it to numeric. val() return data as string, or explicitly use parseInt(var, 10) or parseFloat based on your type.

Simple way is t use unary + operator prefixing the variable:

var checkability = +p + +r + +d + +s;

Fiddle

answered Sep 25, 2013 at 16:55

PSL

124k21 gold badges257 silver badges243 bronze badges

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

Comments

j08691 · Accepted Answer · 2013-09-25 16:55:19Z

0

Sure, the easiest thing is to coerce the string values into integers like:

var p = +$(".p").val(); 
var r = +$(".r").val();
var d = +$(".d").val();
var s = +$(".s").val();

jsFiddle example

You could also use the longer parseInt() function like var p = parseInt( $(".p").val(), 10);

answered Sep 25, 2013 at 16:55

j08691

209k33 gold badges269 silver badges281 bronze badges

Comments

Barbara Laird · Accepted Answer · 2013-09-25 17:15:44Z

0

Use parseInt to make them integers

var p = parseInt($(".p").val(),10); 
var r = parseInt($(".r").val(),10);
var d = parseInt($(".d").val(),10);
var s = parseInt($(".s").val(),10);

var checkability = p + r + d + s;

alert(checkability)

edited Sep 25, 2013 at 17:15

answered Sep 25, 2013 at 16:56

Barbara Laird

12.7k2 gold badges48 silver badges58 bronze badges

3 Comments

Quentin Over a year ago

Don't use parseInt without a radix, doubly so if you are dealing with user input.

Barbara Laird Over a year ago

@Quentin What if you want to support user input of both hex and dec? :)

Quentin Over a year ago

@BarbaraLaird — Then that is best handled with a clear, radio button based, user interface :)

Collectives™ on Stack Overflow

jquery adding input val together

3 Answers 3

Comments

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related