Intermediate Algorithm Scripting: Drop it-help me!

my code isn’t working like it should . can someone tell what’s wrong with it? :

here is my code:


function dropElements(arr, func) {
   for(let i = 0; i < arr.length; i++){
     if(func(arr[0]) === true){
       return arr.slice(0);
     } else {
         arr.shift();
         
     }
     
   }
   return arr;
}

console.log(dropElements([1, 2, 3, 4], function(n) {return n > 5;}));

Challenge: Drop it

Link to the challenge:

you are checking if func is strictly equal to true, but func is a function, not a boolean

1 Like

i meant if func returns true but i don’t know how to write it in Javascript . can you help me

here look i tried to fix i’m almost there but still something i wrong :slightly_smiling_face:

function dropElements(arr, func) {
   for(let i = 0; i < arr.length; i++){
     if(func(arr[0]) === true){
       return arr.slice(0);
     } else {
         arr.splice(0,1);
         
     }
     
   }
   return arr;
}

console.log(dropElements([1, 2, 3, 4], function(n) {return n > 5;}));

So consider this: you’re incrementing over the array, and you have this loop variable you’ve defined (you’re calling it i). Inside your loop, do you want to refer to arr[0], or arr[i]? So, for example,

if(func(arr[i]) === true){
  // what do you want to do here?
}

And then, if it’s NOT true, you simply jump to the next iteration of the loop. Then you just need to figure how you’ll handle the case of NO matches.

1 Like

im using arr[0] on purpose becasue every time i’m removing the first element of the array that doesn’t match the condition so i’m always staying at arr[0]

It isn’t the best of ideas to mutate arrays as you are looping over them like that. Picture a woody woodpecker cartoon, where he’s sawing a tree limb while standing on it.

Here’s a pseudocode version of the cleanest, easiest-to-understand way to handle this:

for (loop over every element of array, using an index){
  if (running func with array[index] is true) {
    return the sliced array from that index to the end
  }
} End the for loop
If we get here, then we had no true elements, return an empty array

Note that that isn’t the only solution, simply the cleanest. What you’re doing is potentially mutating that array each iteration, leaving the original array in an unpredictable state.

The reason you’re getting unexpected results is, you keep incrementing i, and comparing it at each iteration to the length of arr… which is now changing. So say, at the third iteration, my array has been reduced to two members that I still need to compare, but i is now at 3, thus exceeding my array length. The loop stops.

For this logic to work, you’ll need to define a variable before the loop happens, like this:

const numberOfTimesToLoop = arr.length;
for(let i=0; i<numberOfTimesToLoop; i++){
  ...
}

that makes numberOfTimesToLoop a static, fixed value, as opposed to arr.length which you are deliberately changing each time.

2 Likes

wow ! thank you man i really aprecciate the help !! god bless you and thanks for all this great informations . problem solved !

Glad it worked out. Best of luck!

1 Like

good luck to you !!!