jquery removing comma and incrementing by 1?

Question

I have a span element which holds a number such as 1,568.

When a button is clicked, I need that number to increment by 1. So 1,568 becomes 1,569.

If I just get the value and add + 1 to it, the value becomes 2:

("#artist-fan-count").text(parseInt($("#artist-fan-count").text()) + 1);

Should I try to remove the comma and then run the code above? If I do the line below, nothing happens -- which leads me to believe I'm doing it wrong. Shrug.

$("#artist-fan-count").text().replace(',', '');

Any tips?

score 3 · Answer 1 · answered Mar 15 '13 at 22:53

3

Try the following to remove the comma and increment by 1:

var fanCountEl = document.getElementById('artist-fan-count');
fanCountEl.innerHTML = parseInt(fanCountEl.innerHTML.replace(',', '')) + 1;

answered Mar 15 '13 at 22:53

Julian H. Lam

25,501
13
46
73

this won't put the comma back. – Mohamed Nuur Mar 15 '13 at 22:55
This works great. is there a php number_format equivalent I can use to put the comma back? – Mike Mar 15 '13 at 22:56
Indeed, it won't. You'll have to add the comma back separately using another function. – Julian H. Lam Mar 15 '13 at 22:56
@Mike - I believe jQuery has a number format extension you can install. You could also take a look at [this article](http://www.mredkj.com/javascript/numberFormat.html) with working code examples for an "addCommas"-style function – Julian H. Lam Mar 15 '13 at 22:58

score 3 · Accepted Answer · edited May 23 '17 at 10:30

3

You can remove all the commas, parseInt, increment, and then reconstruct your number with the commas, like so:

var $e = $("#artist-fan-count");
var num = parseInt($e.text().replace(',','')) + 1;
$e.text($.fn.digits(num));

using digits from: Add comma to numbers every three digits

edited May 23 '17 at 10:30

Community

1
1

answered Mar 15 '13 at 22:59

Mohamed Nuur

5,536
6
39
55

1

There we go. Everything from the other four answers wrapped into one ;) – Julian H. Lam Mar 15 '13 at 23:03
yep, each answer was slightly off, and by combining them all, you get the whole store ;-) – Mohamed Nuur Mar 15 '13 at 23:06

Tobi · Answer 3 · 2013-03-15T23:10:05.743

2

Well, you have to write your replaced text back:

$("#artist-fan-count").text($("#artist-fan-count").text().replace(',', ''));

after that, you can parse your text as an integer, add your number and (if you want to) put that comma back in place.

If you want to reassemble your Text:

var number = $("#artist-fan-count").text();
$("#artist-fan-count").text(parseInt(number/1000)+","+(number%1000));

edited Mar 15 '13 at 23:10

answered Mar 15 '13 at 22:55

Tobi

1,175
1
19
44

This is... The same as my answer? – tckmn Mar 15 '13 at 22:56
2

hmm, seems that i just have slower Internet-Connection ;-) But yes, the point of that answer seems to be the same. – Tobi Mar 15 '13 at 22:58

Denys Séguret · Answer 4 · 2013-03-15T23:00:47.367

1

You could do that :

var $e = $("#artist-fan-count");
var str = ''+(parseInt($e.text().replace(/,/g,''), 10)+1);
var r = /(\d+)(\d{3})/;
while (r.test(str))  str = str.replace(r, '$1' + ',' + '$2');
$e.text(str);

edited Mar 15 '13 at 23:00

answered Mar 15 '13 at 22:53

Denys Séguret

372,613
87
782
758

2

this will fail for 2,999, it will display 2,1000 instead of 3,000. – Mohamed Nuur Mar 15 '13 at 22:55
@MohamedNuur I thought this was the goal. – Denys Séguret Mar 15 '13 at 22:57
@MohamedNuur If I get the goal (see the whole as a number), so this should do it. – Denys Séguret Mar 15 '13 at 23:01
think about it like this: it's like he's displaying money: `$1,762.34` and he needs to add some amount to it and re-format it. that's how i saw it. – Mohamed Nuur Mar 15 '13 at 23:02
@MohamedNuur Yes, I see it now, I'm just not used to those American numbers. I think my new code is correct. – Denys Séguret Mar 15 '13 at 23:02
in the question, he mentions that he has a number `1,568` and he wants to increment it by 1. Not two numbers and he only wants to increment the second number by 1. – Mohamed Nuur Mar 15 '13 at 23:03

score 1 · Answer 5 · answered Mar 15 '13 at 22:55

1

You're close, but replace doesn't do it in-place:

$("#artist-fan-count").text($("#artist-fan-count").text().replace(',', ''));

answered Mar 15 '13 at 22:55

tckmn

57,719
27
114
156

jquery removing comma and incrementing by 1?

5 Answers5