Javascript forEach Loop

Question

Trying to make this function true. I think the problem is in my for loop.

function forEach(array, callback){    
    console.log(array, callback);
    for(var i = 0; i < array.length; i++) {

    }
}

 // testing your code with console.assert
var total = 1;
var myArray = [1, 2, 3, 4];
function multiplyTotal(a) {
    total *= a;
}
forEach(myArray, multiplyTotal);
// and finally assert; if this fails, the program stops
console.assert(total === 24);

You don't actually call you callback? What did you expect to see happen? Add callback(array[i]); in your for loop. Or just use the existing forEach functions in Javascript. — Matt Burland
– Matt Burland, Commented Mar 10, 2016 at 20:25
Your problem is the empty body of that for loop. You'll need to execute the things that you expect your function to do in there. — Bergi
– Bergi, Commented Mar 10, 2016 at 20:29

madox2 · Accepted Answer · 2016-03-10 20:32:25Z

2

function forEach(array, callback){    
    console.log(array, callback);
    for(var i = 0; i < array.length; i++) {
       callback(array[i]); // you need to call callback function
    }
}

edited Mar 10, 2016 at 20:32

answered Mar 10, 2016 at 20:25

madox2

52.3k21 gold badges106 silver badges101 bronze badges

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

Comments

Smuj Em · Accepted Answer · 2016-03-10 20:33:56Z

0

Additionally, Javascript already has a built in function for this:

myArray.forEach(multiplyTotal);

http://www.w3schools.com/jsref/jsref_forEach.asp

answered Mar 10, 2016 at 20:33

Smuj Em

6427 silver badges10 bronze badges

Collectives™ on Stack Overflow

Javascript forEach Loop

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related