CODEWITHSUNDEEP
javascriptarraysfor-loop
U.P.
U.P.

Reputation: 81

Array loop in javascript

I try to develop a simple program that prints all numbers in between 1 and 100 that divide by 3 without any residual and calculate the total sum

I did it with for loop:

var sum = 0;
for (var i = 3; i <= 100; i = i + 3) {
    document.write("<br/>" + i);
    sum = sum + i;
}
document.write("<br/>sum = " + sum); //1683

But I failed when I wanted to do it with array:

var numbers = [];
var sum = 0;
for (var i = 0; i <= 100; i = i + 3) {
    numbers[i - 1] = i;
}

for (var index = 0; index < 100; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + i;
}

document.write("<br/>sum = " + sum);

Upvotes: 0

Views: 121

Answers (7)

Mehravish Temkar
Mehravish Temkar

Reputation: 4365

This should work using a single loop

var numbers = [];
var sum = 0;
for (var i = 3; i <= 100; i = i + 3) {
    numbers[i] = i;
    document.write("<br/>" + i);
    sum = sum + i;
}

document.write("<br/>sum = " + sum);

Upvotes: 0

Azad
Azad

Reputation: 5272

adding number to array 

var numbers = [];
for(var i = 3; i <= 100; i = i +3){
  numbers.push(i);
}

summation and printing values

var sum = 0;
for (var i = 0; i < numbers.length; i++) {
    document.write("<br/>" + numbers[i]);
    sum = sum + numbers[i];
}
document.write("<br/>sum = " + sum); //1683

Upvotes: 1

Flying Gambit
Flying Gambit

Reputation: 1276

Use it like this,

Array indexes should start from 0, that is why I have introduced another variable j=0

var numbers = [];
var sum = 0;
for (var i = 0, j = 0; i <= 100; i = i + 3, ++j) {
    numbers[j] = i;
}

Update

First Issue: In your code, ie. below code of yours, 

for (var i = 0; i <= 100; i = i + 3) {
    numbers[i - 1] = i;
}

In the first iteration, 

i = 0;
numbers[0-1] = i // i.e numbers[-1] = 0;

and in your second loop, you are starting the index from 0

for (var index = 0; index < 100; index++) {

Second issue:

Also, if you don't use a sequential counter to fill the Array, you will end with undefined values for the ones you did not fill.

enter image description here

If you notice, the output after the loop, it says numbers.length = 99 which is wrong it will not have that many items in it.

Third Issue:

In below code, even if you introduce a sequential counter, this is still wrong 

for (var i = 0; i <= 100; i = i + 3) {
    numbers[i - 1] = i;
}

because i should start with 3 instead of 0, otherwise you will end up with 34 elements in the array because numbers[0] will be 0;

Fourth Issue:

In this code, 

for (var index = 0; index < 100; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + i;
}

You don't actually have to loop it till 100, you already have the numbers array filled, so you just need to use numbers.length, like this 

var len = numbers.length;
for (var index = 0; index < len; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + i;
}

A better way to write this

var numbers = [];
for (var i = 3, j=0; i <= 100; i = i + 3, j++) {
    numbers[j] = i;
}

var sum = numbers.reduce((a, b) => a+b);
console.log(sum);

The line var sum = numbers.reduce((a, b) => a+b); uses Array.reduce() method.

Upvotes: 2

Deep
Deep

Reputation: 9804

There are few issues in your code.

for (var i = 0; i <= 100; i = i + 3) {
    numbers[i - 1] = i;
}

1: array is 0 based. so first insertion into the array goes for a toss.

2: the number array created will have skipping index like 3, 6 ,9 

for (var index = 0; index < 100; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + i;
}

3: Here you are iterating index till 100 , you should iterate it till the length of the numbers array only.

when index is 1,2 number[index] will become undefined.

4: sum = sum + i (i ??????)

You should try like this or you can also use push()

var numbers = [];
var sum = 0;
for (var i = 0,j=0; i <= 100; i = i + 3, j= j+1) {
    numbers[j] = i; // array is 0 based.
  
}



for (var index = 0; index < numbers.length; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + numbers[index];
}

document.write("<br/>sum = " + sum);

Upvotes: 1

M14
M14

Reputation: 1810

Try this

var numbers = [];
var sum = 0;
for (var i = 0; i <= 100; i = i + 3) {
    numbers[(i-3)/3] = i;
}

for (var index = 0; index <  numbers.length; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + numbers[index];
}

document.write("<br/>sum = " + sum);

Here is the fiddle i tried https://jsfiddle.net/4ncgnd7c/

Upvotes: 0

MadDocNC
MadDocNC

Reputation: 820

You have several issues i suppose.

var numbers = [];
var sum = 0;
for (var i = 0; i <= 100; i = i + 3) {
    numbers.push(i);
}

for (var index = 0; index < numbers.length; index++) {
    document.write("<br/>" + numbers[index]);
    sum = sum + i;
}

document.write("<br/>sum = " + sum);

Also for array you can use:

for (var i in array) {
console.log(array[i]);
}

And I'm pretty sure, that array of number sequence is absolutely useless, if there is no other information in it.

Upvotes: 0

Vostrugin
Vostrugin

Reputation: 160

Indexes in an array begin with zero.

for (var i = 0; i <= 100; i = i + 3) {
    numbers[i - 1] = i; // In the first iteration, there will be numbers[-1] = i;
}

Upvotes: 0

Related Questions