Hello everyone! I’m currently working on the “Intermediate Algorithm Scripting: Steamroller” lesson from the JS course. By verifying the tests manually, it seems to me that everything is working fine, but apparently FCC doesn’t agree with me, and I don’t know why. Here is my code:
let newArr = [ ];
function steamrollArray(arr) {
for(let i = 0; i < arr.length; i++) {
if(Array.isArray(arr[i])) {
steamrollArray(arr[i]);
} else {
newArr.push(arr[i]);
}
}
return newArr;
}
steamrollArray([1, [2], [3, [[4]]]]);
Your code contains global variables that are changed each time the function is run. This means that after each test completes, subsequent tests start with the previous value. To fix this, make sure your function doesn’t change any global variables, and declare/assign variables within the function if they need to be changed.
Example:
var myGlobal = [1];
function returnGlobal(arg) {
myGlobal.push(arg);
return myGlobal;
} // unreliable - array gets longer each time the function is run
function returnLocal(arg) {
var myLocal = [1];
myLocal.push(arg);
return myLocal;
} // reliable - always returns an array of length 2
I’ve edited your post for readability. When you enter a code block into a forum post, please precede it with a separate line of three backticks and follow it with a separate line of three backticks to make it easier to read.
You can also use the “preformatted text” tool in the editor (</>) to add backticks around text.
Thank you very much! But I don’t really understand, when I was doing a console.log on the function, it showed the correct results. What happens in the background?
