Class9 higherorder #10
Conversation
reneemeyer
left a comment
reneemeyer
left a comment
There was a problem hiding this comment.
Eric, good start, but you need to work out some of the kinks in these functions. Remember, the purpose of functions in JS is re-usability and the ability to return data. We don't want to hard code any data into a function.
| function forEach(arr, callback) { | ||
| // Your code here | ||
| // create an array with whatever you want in it | ||
| const arrProblemOne = [1, 2, 3] |
|
|
||
| //make a for loop that runs the number of items in the array amount of times. | ||
| //make a variable that prints a phrase the amount of times defined by the for loop | ||
| for (let i = 0; i < arrProblemOne.length; i++) { |
There was a problem hiding this comment.
Your for loop should loop around the arr that the user passes in, not a hard coded array.
| //make a for loop that runs the number of items in the array amount of times. | ||
| //make a variable that prints a phrase the amount of times defined by the for loop | ||
| for (let i = 0; i < arrProblemOne.length; i++) { | ||
| callback(); |
There was a problem hiding this comment.
Each item needs to be passed into the callback.
| @@ -3,15 +3,45 @@ | |||
| const assert = require('assert'); | |||
|
|
|||
| function forEach(arr, callback) { | |||
| function map(arr, callback) { | ||
| // Your code here | ||
| // Create an array, put whatever you want in it | ||
| const test = ['1', '2', '3', '4', '5', '6']; |
| //doThis(), a function that creates a new array, then does whatever function you want when you call doThis to the original array | ||
| //pushes the result of the called function to the new array | ||
| //print out the new array | ||
| const doThis = (arr, callBackFunction) => { |
There was a problem hiding this comment.
what is the purpose of this internal function?
| //print out the new array | ||
| const doThis = (arr, callBackFunction) => { | ||
| const newArr = []; | ||
| test.forEach((i) => { |
There was a problem hiding this comment.
your forEach should loop around the array passed, not a hardcoded array.
| test.forEach((i) => { | ||
| newArr.push(callBackFunction(i)); | ||
| }); | ||
| console.log(newArr); |
There was a problem hiding this comment.
you should return the newArr, not console log it
| //a new, empty array accepts the results of the function | ||
| const filtered = (arrTwo, callbackTwo) => { | ||
| const arrResultThree = []; | ||
| if (callbackTwo(arrTwo[i])) arrResultThree.push(arrTwo[i]) |
There was a problem hiding this comment.
Where is i defined? This doesn't make any sense. Also where did longestWords come from? Please follow the exact directions. This function should take an array and a function as arguments. It should return an array with only the items that return true in the function. Don't use hard coded arrays or functions.
Checkpoint Rubric
This is the rubric that your instructor will use to grade your checkpoints. Please do not edit.
Checkpoint 1
Checkpoint 2
Checkpoint 3