Accumulating an array's values - Javascript

Question

I'm trying to accumulate a series of numbers in an array e6

Here is the relevant code.

 e3 = prompt(e1 + ", Please enter few numbers (maximum of 6) separated by commas", "1,2,3,4,5");
 e6 = e3.split(',');

for(var a=0;a <= e6.length ;a++) {

       e9=e9 + +e6[a];

    }   
document.write(e9)  ;

However, what get's printed is NaN instead of the default sum of 15. Any ideas how to fix? Thank you.

Edit: Forgot to mention that i already had declared all my variables earlier.

var e1,e2,e3,e4,e5,e6,e7,e8,e9,e10,e11;

Edit2: Here is my entire work in action. https://jsfiddle.net/nhz0Lnx8/

Declare your variables with var and initialize e9 to 0. — Pointy
– Pointy, Commented Jun 22, 2016 at 15:58
@andirew1990 From your fiddle (edit #2), it looks like you've solved this problem. Is that correct? — Mike Zavarello
– Mike Zavarello, Commented Jun 22, 2016 at 16:49

Folashade · Accepted Answer · 2016-06-22 16:09:46Z

1

You should be looking only as far as e6.length-1, but the best solution is to avoid the off by one errors.

var e3 = prompt("Please enter few numbers (maximum of 6) separated by commas", "1,2,3,4,5");
var e6 = e3.split(',');

var e9 = 0;
e3.split(',').map((x)=>{e9 += +x})
document.write(e9)

edited Jun 22, 2016 at 16:09

answered Jun 22, 2016 at 16:08

Folashade

514 bronze badges

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

1 Comment

andirew1990 Over a year ago

This is for a class, i dont think it would be appropriate to use some of the tools you suggested such as map or the => as we have not discussed them yet. Thanks for your effort though.

Rafal Zajac · Accepted Answer · 2016-06-22 16:21:49Z

1

The error is in the for loop declaration: "a <= e6.length" should be "a < e6.length" ("less than equal or equal" should be changed to "less than")

answered Jun 22, 2016 at 16:21

Rafal Zajac

1,6931 gold badge16 silver badges14 bronze badges

Comments

Mohit Bhardwaj · Accepted Answer · 2016-06-22 16:30:17Z

0

var e3 = prompt("Please enter few numbers (maximum of 6) separated by commas", "1,2,3,4,5");
var e6 = e3.split(',');

var e9 = 0;

for(var a=0;a < e6.length ;a++) {

       e9 += parseInt( e6[a] );

    }   
document.write(e9)  ;

edited Jun 22, 2016 at 16:30

answered Jun 22, 2016 at 15:59

Mohit Bhardwaj

10.1k7 gold badges41 silver badges66 bronze badges

6 Comments

andirew1990 Over a year ago

Didn't change anything. Still display NaN

Mohit Bhardwaj Over a year ago

Please check now. Working fine for me now.

andirew1990 Over a year ago

Displays nothing at all now in document.write(e9). i forgot to mention that i already declared all my variables earlier. will edit main post.

Pointy Over a year ago

@andirew1990 this code does work just fine. Perhaps the surrounding HTML is incorrect.

andirew1990 Over a year ago

I managed to get it to work by removing ur trim() part. Now it displays it properly.

|

Collectives™ on Stack Overflow

Accumulating an array's values - Javascript

3 Answers 3

1 Comment

Comments

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related