Question

 // Write a program to sum all the odd elements of an array

    a = Number(prompt("a:"));
    b = Number(prompt("b:"));
    c = Number(prompt("c:"));

    sum=Number(0);

    var test = [a,b,c];

    for (i=0; i<test.length;i++) {
        // All even numbers can be divided by 2
        if ((test[i]%2)>0) {
            alert(test[i] + ":odd");
        } else {
            alert(test[i] + ":even");
        }
        test[i]=0;
   }

   sum=0+test[i];
   alert(sum);

My program works brilliant up until its meant to add up all the numbers where it returns a NaN message! Any ideas on how I can sort this problem out?

Answer 1

I see three problems with your code. (Not counting the weird formatting of the curly brackets.)

1. Edit: looks like the question has been updated a couple of times since I started my answer.In the “if odd” test you are setting the number to zero with Perhaps you meant to zero out the even numbers so that you could later add up all the numbers and just get the odd ones, but test[i]=0 is not part of the if/else structure. It looks like it should be because of the indenting, but JavaScript ignores extra white-space.

2. You are not adding the numbers to the sum within your loop.

3. You are getting NaN because of this line:

   sum=0+test[i];
   

That statement is outside the end of the loop, so at that point i is equal to the length of the array and test[i] is undefined.

Try the following pseudo-code:

sum = 0
for (loop through the array) {
   if current number is odd
      sum = sum + current number
}