Remove specific strings from an Array in JavaScript

Question

I am solving a practice problem in Practice-it and there is a question that is saying to creating an empty array inside a function and pass the array of strings as an argument into the function and to remove a specific word from it.

Here is the question: Write a function named removeAll that accepts an array of Strings and a single String as parameters, and returns a new array. The returned array should be equivalent to the parameter array but with all occurrences of the String parameter removed, ignoring case. The array parameter should remain unchanged as a result of the function call. For example, if an array named words contains ["foo", "bar", "baz", "Foo", "FOO"], the call of removeAll(words, "foo") should return ["bar", "baz"].

My code: I am getting undefined

function removeAll(words, remove){
  let arr = [];
  remove = '';
  arr = arr.filter(words => words !== remove);
}

removeAll(["foo", "bar", "baz"], "foo");

Any help would be appreciated. Thank you!

function removeAll(words, remove){ words = words.filter(word => word !== remove); } — Jaromanda X
– Jaromanda X, Commented Dec 1, 2017 at 2:22
You do not return anything.... and your code does not handle cases — epascarello
– epascarello, Commented Dec 1, 2017 at 2:26

S. Walker · Accepted Answer · 2017-12-01 02:25:54Z

2

Your close, but you have 2 mistakes. Do not set remove to ''. And filter words not arr

function removeAll(words, remove){
  return words.filter(word => word.toLowerCase() !== remove.toLowerCase());
}

answered Dec 1, 2017 at 2:25

S. Walker

2,16214 silver badges31 bronze badges

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

Comments

Nandu Kalidindi · Accepted Answer · 2017-12-01 02:32:08Z

1

You are using the variables internally in the wrong way.

// toLowerCase to compare lowercase strings
function removeAll(words, remove){
  return words.filter(word => 
    word.toLowerCase() !== remove.toLowerCase()
  );
}

console.log(removeAll(["foo", "bar", "baz", "FOO"], "foo"));


// Using Arrow functions
removeAllES6 = (words, remove) => 
   words.filter(word => word.toLowerCase() !== remove.toLowerCase())

console.log(removeAllES6(["foo", "bar", "baz", "FOO"], "foo"));

edited Dec 1, 2017 at 2:32

answered Dec 1, 2017 at 2:23

Nandu Kalidindi

6,2901 gold badge28 silver badges38 bronze badges

Comments

miradham · Accepted Answer · 2017-12-01 03:00:22Z

1

Additionally to what is already mentioned, since you need a case insensitive comparision, use toLowerCase() or toUpperCase()

function removeAll(words, remove){
  return words.filter(words => words.toUpperCase() !== remove.toUpperCase());
}

arr = removeAll(["foo", "bar", "baz"], "foO");

edited Dec 1, 2017 at 3:00

answered Dec 1, 2017 at 2:28

miradham

2,36520 silver badges26 bronze badges

Collectives™ on Stack Overflow

Remove specific strings from an Array in JavaScript

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related