Adding multiple strings as numbers in Javascript

Question

I have the following piece of code that converts multiple strings into number, which works fine, but I don't like the way it looks and want to know if I could write it in a better way.

  Number(result.channels.electricity.chan[0].day[0]._)
+ Number(result.channels.electricity.chan[1].day[0]._)
+ Number(result.channels.electricity.chan[2].day[0]._)
+ Number(result.channels.electricity.chan[3].day[0]._)
+ Number(result.channels.electricity.chan[4].day[0]._)
+ Number(result.channels.electricity.chan[5].day[0]._)

Is there a cleaner way of writing this?

numerous ways to wrap a loop around this

charlietfl
– charlietfl

2016-01-23 18:31:38 +00:00
Commented Jan 23, 2016 at 18:31 — charlietfl
– charlietfl, Commented Jan 23, 2016 at 18:31

MinusFour · Accepted Answer · 2016-01-23 18:30:16Z

3

With reduce is pretty simple:

var sum = results.channels.electricity.chan.reduce(function(a, b){
    return a + Number(b.day[0]._);
}, 0);

answered Jan 23, 2016 at 18:30

MinusFour

14.5k4 gold badges32 silver badges39 bronze badges

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

Comments

guest271314 · Accepted Answer · 2016-01-23 18:36:31Z

1

Try using while loop

var channels = ["1", "2", "3", "4", "5"], i = -1, res = 0;
while (++i < channels.length ) res += Number(channels[i]);
console.log(res)

answered Jan 23, 2016 at 18:36

guest271314

1

3 Comments

Bazinga777 Over a year ago

Thanks, but my data structure doesn't allow me to use this.

guest271314 Over a year ago

@Bazinga777 "my data structure doesn't allow me to use this" Can describe why data structure does not allow approach ? Is data structure an array ?

Bazinga777 Over a year ago

I am parsing an xml file parsed to json which looks pretty bad, so this would create a bit of a mess

Microfed · Accepted Answer · 2016-01-23 18:55:15Z

1

Just for fun. With some ES6-7 features:

let chans = results.channels.electricity.chan;
let total = chans.reduce((sum, { day: [ { _ } ] }) => (sum + Number(_)), 0);

answered Jan 23, 2016 at 18:55

Microfed

2,89824 silver badges26 bronze badges

Comments

Peter Roehlen · Accepted Answer · 2016-01-23 18:34:00Z

0

A few things you can do.

Small fix to improve readability would be to assign the chan property to a variable so you don't have to repeat the whole path. E.g. var chan = result.channels.electricity.chan
Use forEach to iterate over the array elements and increment a variable inside the loop.

answered Jan 23, 2016 at 18:34

Peter Roehlen

1282 silver badges8 bronze badges

Collectives™ on Stack Overflow

Adding multiple strings as numbers in Javascript

4 Answers 4

Comments

3 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related