split an array without separate number of two digits

Question

I have the following variable with some number.

const string = "5,5,5,5";
const string = "5,5,5,5,10";

I pasted it twice to explain what I am trying to do.

I want to add all the numbers and get a total.

The first I did, is cleaning the variable to avoid the commas, when I split it

let stringWithoutSigns = string.replace(/,/g, "");

I paste the full code to better understanding.

const string = "5,5,5,5";

let stringWithoutSigns = string.replace(/,/g, "");

let itemString = stringWithoutSigns.length;

if (itemString > 2) {
  result = cleanString();
  console.log(result)
}

function cleanString() {
  let arrString = stringWithoutSigns.split("").map(function(index) {
    return parseInt(index, 10);
  });
  console.log(arrString);
  let sum = arrString.reduce(function(a, b) {
    return a + b;
  });
  return sum;
}

My problem, is that if I type a new number like 10, it doesn't works, because is splitted into 1, 0

Why are you removing the commas? Use `string.split(',')` to split the original string at the commas. — Barmar, Mar 12 '20 at 12:31
isn't just: `var sum = string.split(",").reduce((a,b) => +a + +b)`? — Calvin Nunes, Mar 12 '20 at 12:31
maybe `var sum = string.split(",").reduce((a,b) => (+a)+(+b))` — canbax, Mar 12 '20 at 12:32
if I don´t removed them, when I asplit it, I will have something like this ["5", ","...] — Sargentogato, Mar 12 '20 at 12:32
@Barmar - you don't need parseInt - calvin's code is correct — Jaromanda X, Mar 12 '20 at 12:34
ahashasdfa%$&$$... Thanks @CalvinNunes I was going on the bad way.... ahhh — Sargentogato, Mar 12 '20 at 12:36
@CalvinNunes your answer was the correct one Thanks a millions. — Sargentogato, Mar 12 '20 at 12:39

score 1 · Answer 1 · edited Mar 12 '20 at 12:35

1

Don't remove the commas, use them to split the string into numbers.

const string = "5,5,5,5";

result = cleanString(string);
console.log(result)


function cleanString(string) {
  let arrString = string.split(",").map(function(num) {
    return parseInt(num, 10);
  });
  console.log(arrString);
  let sum = arrString.reduce(function(a, b) {
    return a + b;
  });
  return sum;
}

edited Mar 12 '20 at 12:35

Cerbrus

70,800
18
132
147

answered Mar 12 '20 at 12:34

Barmar

741,623
53
500
612

1

Or `.map(Number)` – Cerbrus Mar 12 '20 at 12:36
2

@Cid No, because `map()` sends 3 arguments to the callback, and parseInt uses the second argument as the radix. – Barmar Mar 12 '20 at 12:37
See https://stackoverflow.com/questions/262427/why-does-parseint-yield-nan-with-arraymap – Barmar Mar 12 '20 at 12:38
@Barmar true about `parseInt`, I was sure the default value for radix was 10, but docs told me something else – Cid Mar 12 '20 at 12:39
@Cid The problem is that the argument passed by `map()` overrides the default. – Barmar Mar 12 '20 at 12:40

score 1 · Answer 2 · answered Mar 12 '20 at 12:36

Split the string by checking , rather that on empty string. Hope this is what you are looking for.

const string = "5,5,5,10";

let stringWithoutSigns = string.replace(/,/g, "");

let itemString = stringWithoutSigns.length;

if (itemString > 2) {
result = cleanString();
console.log(result)
}

function cleanString() {
  let arrString = string.split(",").map(function(index) {
    return parseInt(index, 10);
  });
  console.log(arrString);
  let sum = arrString.reduce(function(a, b) {
    return a + b;
  });
  return sum;
}

score 0 · Answer 3 · answered Mar 13 '20 at 09:37

I let one answer more. I can pass an empty string, 0, or as may numbers as I want(I tested it with 20 numbers untill now)

const string = "5,5,5,5,10"

console.log(add(string))

function add(string) {
  let splitted = string.split(",");
  let numbers = splitted.map(value => Number(value || 0));
  return numbers.reduce((a, b) => a + b);
}

score 0 · Answer 4 · answered Mar 13 '20 at 10:02

0

Basic

string.split(',').reduce((num,acc)=> parseInt(num) + parseInt(acc) )

answered Mar 13 '20 at 10:02

manish kumar

4,412
4
34
51

split an array without separate number of two digits

4 Answers4