Iterate Through All an Array's Items Using For Loops, length is not defined

Tell us what’s happening:
In the line j===arr[i].length, length is not defined. What’s the reason and is there a way around it in the existing code?

Your code so far

function filteredArray(arr, elem) {
  let newArr = [];
  let j =0;
  // change code below this line
  for(let i=0;i<arr.length;i++){
  // change code above this line
  return newArr;

// change code here to test different cases:
console.log(filteredArray([[3, 2, 3], [1, 6, 3], [3, 13, 26], [19, 3, 9]], 3));

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36.

Link to the challenge:

It’s not length that’s not defined. It can’t read the length OF undefined.

The problem is on the line 12 and 13, you’re using [i] outside of the for loop scope.

The [i] is in the scope of for loop for i. Also globally declaring i doesn’t seem to work.

Whoops, my bad.

Well, I don’t know. I’ve checked the code for errors in different parts of the function and got none, but only an empty array is returned.

@nimrodcoder There are a few problems with this code. I suggest starting with let j =0;. Why did you add this?

Removing this gives me the error: j is undefined. Isn’t it defined in your nested for loop the same way i is? It’s not, so you can fix that.

Then, as you try to fix the code from there, pay attention to scoping as j will only be accessible from within the nested for loop.

I think you’ll be able to complete it once this structure is in place.

I looks to me that you are looping over arr while removing elements. What does that do to your index? arr.length?

Do you really need to remove these from arr?

1 Like

@nimrodcoder, this might make your life easier with this assignment:

Cheers :slight_smile:

1 Like

This solved it!!! thanks! i only needed to remove the splice line. :sweat_smile:

see…the first reply suggested scoping problems, that’s when i declared them globally. Scoping wasn’t as issue tho.